GCC: function was not declared in this scope

[gcampton]

Hello, been a while since I've done any coding, went on holidays for 4months and deferred.
Anyway I have this error and can not figure out what I'm doing wrong any help is appriciated :)

wordfinder.cpp: In function 'bool parseSize(std::string, int&, int&)':
wordfinder.cpp:120: error: 'isInteger' was not declared in this scope

wordfinder.cpp

bool parseSize(string line, int& row, int& col)
{
	int size_pos[3]= { 0 };
	string strRow, strCol;
	// get characters from string

	size_pos[0] = line.find('=');
	size_pos[1] = line.find(',');
	size_pos[2] = line.find_first_of(" #\t\n");

	int len1 = size_pos[1] - size_pos[0] - 1;
	strRow = line.substr(size_pos[0]+1,len1);
	int len2 = size_pos[2] - size_pos[1] - 1;
	strCol = line.substr(size_pos[1]+1,len2);

	// check characters are valid numbers
	// and puzzle size is ok
	if (isInteger(strRow,row))
	{
		if (isInteger(strCol,col))
		{
			if (row>1 && col>1)
				return true;
			cout << "Puzzle size is too small: "
				<< row << " * " << col << endl;
		}
		else
			cout << "Column is not a valid number." << endl;
	}
	else
		cout << "Row is not a valid number." << endl;
	return false;
}

//=============================================//

void wordfinder::convert2Upper(string& str)
{
	for (unsigned i=0; i<str.length(); i++)
		str[i]=toupper(str[i]);
}

//=============================================//

bool wordfinder::isInteger(string& str, int& i)
{
	istringstream ss(str);
	return ss >> i ? true : false;
}

wordfinder.h

#ifndef WORDFINDER_H
#define WORDFINDER_H

#include <string>
#include <iostream>
#include <cstdlib>
#include <new>
#include <fstream>
#include <sstream>


class wordfinder
{
   public:
      wordfinder();
      wordfinder(std::string);
      ~wordfinder();
      void parseFile();
      void processString(std::string& line);
      bool parseWord(std::string line,std::string& word);
      bool parseSize(std::string line,int& row,int& col);
      void convert2Upper(std::string& str);
      bool isInteger(std::string& str,int& i);
      void resetBools();
      bool isComment(std::string& str);
      bool isSize(std::string& str);
      bool isSearch(std::string& str);
   private:
      std::string filename;
      bool gotSize;
      bool gotWord;
      bool gotPuzzle;
};

#endif 

[Vexer]

In the first code snippet, move the IsInteger() function definition to above the parseSize() function definition.

[Galik]

You need to qualify the function definition with the name of the class it belongs to like this:

bool wordfinder::parseSize(string line, int& row, int& col)
{
    // ...
}

[gcampton]

edit: heh thnx Galik, I found it as well. :)

so annoying when simple mistakes take 3+ hours to find...
wasn't until I saw the small snippet I posted that I saw the problem.

[gcampton]

Vexer wrote:

In the first code snippet, move the IsInteger() function definition to above the parseSize() function definition.

Thought I'd comment on this:
Function definitions are already included by the header file which is included by the pre-processor before compile time.
Where the actual function is placed in the .cpp file is irrelevant.

edit:
Unless of course your not using headers, simply functions as part of a file.
eg won't work:

int main()
{
   somestuff();
   return 0;
}

void somestuff()
{
   cout << "Hello world";
}

eg will work:

void somestuff();

int main()
{
   somestuff();
   return 0;
}

void somestuff()
{
   cout << "World Hello";
}

the second code snippet is similar to what the preprocessor does for header files.

[RyanCaywood]

void somestuff()
{
   cout << "World Hello";
}

int main()
{
   somestuff();
   return 0;
}

Will also work... though this is probably irrelevant. Also take note that I really didn't read much into this thread, and may be saying something stupid...

[Vexer]

Thought I'd comment on this:
Function definitions are already included by the header file which is included by the pre-processor before compile time.
Where the actual function is placed in the .cpp file is irrelevant.

Ah...

Thanks, I knew this was true for regular functions, wasn't sure about methods.