C++ Value Return Function that produces a true/false

[zamaam0728]

I am at the end of my first semester of C++, and I'm not sure what I should do to make this program meet the requested specifications. The assignment is as follows:

Write a function called ignoreCaseCompa re() that has two character (char) parameters. The function should return true if the two characters received represent the same letter, even if the case does not agree. Otherwise, the function should return false. Then, write a simple main() function that uses your ignoreCaseCompa re().

Additional instructions:

Here is the function heading:
bool ignoreCaseCompa re(char c1, char c2)

{

//Write the code to compare c1 with c2 and return true/false

}


Here is what I have:

#include <iostream>
 
using std::cout;
using std::cin;
using std::endl;
 
//function prototype
bool ignoreCaseCompare (char first, char second);
 
int main ()
{
    char first = ' ';
    char second = ' ';
 
    cout << "Enter a letter: ";
    cin >> first;
    cout << "Enter another letter: ";
    cin >> second;
 
    first = toupper(first);
    second = toupper(second);
 
    cout << "Are the letters the same? " <<
 
    return 0;
}
//******function defintions*****
bool ignoreCaseCompare (char one, char two)
 
{if (one == two)
return true;
else
return false;
}
 
    //end main function

This programs is much easier without the value-return function. Can anyone offer any advice or direction?

[newb16]

move toupper inside ignoreCaseCompa re

[donbock]

Your main function doesn't call ignoreCaseCompa re().

[jkmyoung]

You want to call ignoreCaseCompa re() on the original characters, not the toupper equivalents. Move the toupper and the comparison parts into the function itself.

[zamaam0728]

I am getting error messages still, but here are the changes I made.

#include <iostream> 
 
using std::cout; 
using std::cin; 
using std::endl; 
 
//function prototype 
bool ignoreCaseCompare (char first, char second); 
 
int main () 
{ 
    char first = ' '; 
    char second = ' ';
    char answer = ' ';
 
    cout << "Enter a letter: "; 
    cin >> first; 
    cout << "Enter another letter: "; 
    cin >> second; 
 
    answer = bool ignoreCaseCompare (char first, char second)
    cout << "The letters the same: " answer << endl;
 
    return 0; 
} 
//******function defintions***** 
bool ignoreCaseCompare (char one, char two) 
 
one = toupper(one); 
two = toupper(two); 
 
 
{if (one == two) 
return true; 
else 
return false; 
} 
 
    //end main function

[Frinavale]

You should really try to read and look at what the error message has to say. If you still can't make heads or tails of what it's telling you...you should post the error message so that we can see it and can help you further with your problem.

The error in this case is very obvious...the error(s) occurred on lines 29 and 30 of your above posted code....


You did not move the following lines inside a method:

 one = toupper(one); 
 two = toupper(two); 

Please note that in order for code to be within a method it must be inside the method's opening and closing curly braces {}

For example:

bool ignoreCaseCompare (char one, char two) 
{
  //code in here belongs to the ignoreCaseCompare method.
}

-Frinny

[zamaam0728]

I moved

one = toupper(one);  
two = toupper(two);

inside the function.

Here are the error messages:

error C2062: type 'bool' unexpected
error C3646: 'one' : unknown override specifier
error C2072: 'ignoreCaseComp are' : initialization of a function
error C2065: 'one' : undeclared identifier
error C4430: missing type specifier - int assumed. Note: C++ does not support default-int
error C2447: '{' : missing function header (old-style formal list?)

I have never had any difficulty with this until now. Ugh!

#include <iostream> 
 
using std::cout; 
using std::cin; 
using std::endl; 
 
//function prototype 
bool ignoreCaseCompare (char first, char second); 
 
int main () 
{ 
    char first = ' '; 
    char second = ' ';
    char answer = ' ';
 
    cout << "Enter a letter: "; 
    cin >> first; 
    cout << "Enter another letter: "; 
    cin >> second; 
 
    answer = bool ignoreCaseCompare (char first, char second)
    cout << "The letters the same: " answer << endl;
 
    return 0; 
} 
//******function defintions***** 
bool ignoreCaseCompare (char one, char two) 
 
one = toupper(one); 
two = toupper(two); 
 
 
{if (one == two) 
return true; 
else 
return false; 
} 
 
    //end main function

[Frinavale]

Add an open curly brace "{" to line 28 in your above posted code.
Then delete the open curly brace on line 33 (that is in front of your if statement).

In C, C++, Java, JavaScript, C#...and may other programming languages...
The open curly braces "{" indicate the start of a block of code and the close curly brace "}" indicates the end of a block of code.

A function is a block of code and it requires an open curly brace "{" and a close curly brace "}" to indicate where the function stops and where it ends. Anything in side the curly braces is used by the method.

Your function has to look like:

void myMethod()
{
  bool a = true;
  bool b = false;
}

Notice how the bool a and the bool b are declared inside the method?
Notice how the open curly brace "{" comes before these variables?


If I had put the following it would not compile because it doesn't make sense to the compiler:

void myMethod()
 a = true;
{
 
  bool b = false;
}

That is what you have going on right now in your code.


Once you've fixed that, you should address the problems that you are causing on line 21 in your above posted code. This is not how you call a method....remov e the "bool" on that line or indicate that you are casting to a bool by placing the bool within open and close parentheses like this: (bool).....also you need to add a semicolon ";" to the end of line 21 to indicate that you are finished.

-Frinny

[newb16]

Calling your function as
answer = bool ignoreCaseCompa re (char first, char second)
is wrong. I don't post the correct version - one semester of C/C++ should be enough to learn how to call C functions and how to read error messages.
Hint - toupper() is a function too.

[zamaam0728]

I thought about using the void, but we can not because that is inthe next chapter, and we can not include commands that haven't been covered yet.