Identifier not found error on function call

C++Compiler Errors

C++ Problem Overview


I have a program here where I invert the case of an entered string. This is the code in my .cpp file and I am using Visual Studio C++ IDE. I am not sure what I need in a header file or if I need one to make this work.

Error with my function call swapCase. Main does not see swapCase for some reason that I'm not sure of.

#include <cctype>
#include <iostream>
#include <conio.h>

using namespace std;

int main()
{
	char name[30];
	cout<<"Enter a name: ";
	cin.getline(name, 30);
	swapCase(name);
	cout<<"Changed case is: "<< name <<endl;
	_getch();
	return 0;
}

void swapCase (char* name)
{
	for(int i=0;name[i];i++)
	{
		if ( name[i] >= 'A' && name[i] <= 'Z' )
			name[i] += 32; //changing upper to lower
		else if( name[i] >= 'a' && name[i] <= 'z')
			name[i] -= 32; //changing lower to upper
	}
}

Any other tips for syntax or semantics is appreciated.

C++ Solutions


Solution 1 - C++

Add this line before main function:

void swapCase (char* name);

int main()
{
   ...
   swapCase(name);    // swapCase prototype should be known at this point
   ...
}

This is called forward declaration: compiler needs to know function prototype when function call is compiled.

Solution 2 - C++

Unlike other languages you may be used to, everything in C++ has to be declared before it can be used. The compiler will read your source file from top to bottom, so when it gets to the call to swapCase, it doesn't know what it is so you get an error. You can declare your function ahead of main with a line like this:

void swapCase(char *name);

or you can simply move the entirety of that function ahead of main in the file. Don't worry about having the seemingly most important function (main) at the bottom of the file. It is very common in C or C++ to do that.

Solution 3 - C++

At the time the compiler encounters the call to swapCase in main(), it does not know about the function swapCase, so it reports an error. You can either move the definition of swapCase above main, or declare swap case above main:

void swapCase(char* name);

Also, the 32 in swapCase causes the reader to pause and wonder. The comment helps! In this context, it would add clarity to write

if ('A' <= name[i] && name[i] <= 'Z')
    name[i] += 'a' - 'A';
else if ('a' <= name[i] && name[i] <= 'z')
    name[i] += 'A' - 'a';

The construction in my if-tests is a matter of personal style. Yours were just fine. The main thing is the way to modify name[i] -- using the difference in 'a' vs. 'A' makes it more obvious what is going on, and nobody has to wonder if the '32' is actually correct.

Good luck learning!

Solution 4 - C++

You have to define void swapCase before the main definition.

Attributions

All content for this solution is sourced from the original question on Stackoverflow.

The content on this page is licensed under the Attribution-ShareAlike 4.0 International (CC BY-SA 4.0) license.

Content TypeOriginal AuthorOriginal Content on Stackoverflow
QuestionKRBView Question on Stackoverflow
Solution 1 - C++Alex FView Answer on Stackoverflow
Solution 2 - C++Michael KristofikView Answer on Stackoverflow
Solution 3 - C++Kevin HoppsView Answer on Stackoverflow
Solution 4 - C++v01dView Answer on Stackoverflow