Function that takes a string and character and returns number of occurrences of character in the string

[anieves001]

Function that takes a string and character and returns number of occurrences of the character in the string.

  int countChars(const string& s,char c){
int count=0;
for(int i=0;i<s.length();i++){
if(s[i]==c)
count++;
}
}
int main(){
string s;
char c;
cout<<"Enter a string of text\n"
cin>>s;
cout<<"Enter char\n";
cin>>c;
countChars(s,c);
cout<<countChars<<"\n";
return 0;
}

Trying to figure out why it keeps returning 1 no matter what I put in. Any help will be greatly appreciated.

[keskiverto]

Your compiler might say something like:

warning: no return statement in function returning non-void [-Wreturn-type]

[anieves001]

Sorry forgot to include that in my source code. I actually have return count; at the end of the function

[keskiverto]

Ok, what do you do in main() with the value that the function returns?

[anieves001]

int main(){

string s;
char c;
cout<<"Enter a string of test\n";
cin>>s;
cout<<"Enter char\n";
cin>>c;

countChars(s,c);

cout<<countChars<<"\n";


return 0;

}
Thanks for your help.

[cire]

We can see that code in the OP.

The question is "What are you doing with the value countChars returns?"

The answer is: "You discard it without using it."

Don't ignore the return value in main.