Thread: returning mode value from array

I have an idea of how I want to tackle this but its hard for me to implement in code. I want the function to check each element and if it spots another one just like it, to do an incrementation. I need a nudge. I'm not sure how to take that first step. Any help as always is appreciated.

Code:

      #include <iostream>
using namespace std;
int mode(int a[],int size );//prototype
int main()
{

 
 const int size=10;
int a[size]={1,2,-3,-4,-5,-6,7,8,-9,10};


 mode(a,size);//call

  return 0;

}


 int mode(int a[], int size) //returns mode element of array
 {
    int m=0;
	 for(int i=0;i<size;i++){
      if(i                               //????????
		 a[i];
	 }
	 
}

I just mean I want the function to count how many times each element is present in the array. The element that has the highest count would be the mode which is what I want the function to return.

Couldn't another way also involve using a switch statement?

Dantestwin - you seem to be posting a lot of threads like this. If I may teach a quick lesson in debugging:

mode.cpp(17) - the file in which the error occurred (which you will need to know when you start building larger projects consisting of multiple files), and the line number. Look at the location specified here, as you will oftentimes notice that you just made a simple typo. It's really useful to concentrate on one area, and since the compiler tells you exactly where there's a problem, it's made easy.

error C2065 - a lot of compilers distribute a list of error codes and what they mean. In this case, you've got a bit of a summary at the end.

'freq' : undeclared identifier - just try and think about what this is trying to tell you, although it can be hard since it uses a lot of PC buzz words that newbies aren't likely to know. In this case, let me talk you through how I would do it. First of all, it's talking about 'freq', which you know is a variable that you made. "Undeclared identifier" - it's simply a variable that hasn't been declared properly. The actual cause for this could be that the variable has gone out of scope or wasn't declared.

edit: To be specific, in this case you simply didn't declare or initialize freq, like you did for size, etc...

thanks sean, i fixed the bug now. It compiles fine and everything but there is one logical error. The biggest element in the array is 10 and it the program prints out the current frequency for every other element but 10. for some reason, it always adds 1. like if 10 occurred 3 times, it would print out 4. and so forth. any help as to what glitch in my code is sparking this error would be greatly appreciated.

Code:

#include <iostream>
using namespace std;
int mode(int freq[],int a[],int size );//prototype
int main()
{

 
 const int size=10;
int a[size]={1,10,3,5,5,10,7,7,7,10};
int freq[size]={0};

cout<< mode(freq,a,size);//call

  return 0;

}


 int mode(int freq[], int a[], int size) //reverse order of elements in array
 {

	 int largest=0;
	 int modeValue=0;
	 
	 for(int i=0;i<size;i++)
		 freq[i]=0; //initializing frequency to 0
 
	  //summarizing frequencies

	 for (int j=0; j<size; j++)
		 ++freq[a[j]];

	 //output results

	 for (int rating=1;rating<=size;rating++){
		 cout<<rating<<' '<<freq[rating]<<endl;

		 if(freq[rating]>largest) {
			 largest=freq[rating];
			 modeValue=rating;
		 }
      
 
	 }
       return modeValue;
    
 }