cant find silly mistake

[heyyouyesyouiloveyou]

Ok, so I'm obviously making some stupid mistake, but I've been trying to find it for 20 minutes so...

Here is my class declaration file "game.h"

  //class Game declaration file
#ifndef GAME_H
#define GAME_H
#include "bead.h"
#include "player.h"
#include <vector>
using namespace std;
class Game {
private:
	char userInput;
	vector<Bead> beads;
	int boardWidth;
	int boardHeight;
	Bead* beadPtr;
	int cursorXPos;
	int cursorYPos;
public:
	Game();
	void render(); 
	bool isOver(Player p);
	void takeUserInput();
	void makeBeads();
	void deleteBeads();
	void movePlayer();
	void updateGame();
};
#endif 

You'll notice I have declared the char userInput.

In my game.cpp file, I wrote the following function:

void takeUserInput() {
	userInput = getch();
}

I have included "game.h" in game.cpp, but for some reason I am getting the "use of undeclared identifier" error.

What am I doing wrong?

Thanks!

[Duthomhas]

void Game::takeUserInput() {
  ...
}

I presume you have <conio.h> included somewhere also?

Hope this helps. :)

[heyyouyesyouiloveyou]

Yup, super silly mistake! Thanks for catching it! I am actually using ncurses. Is there an advantage to conio, or is it the same?

[giblit]

conio.h is non-standard so I wouldn't assume there be any advantages also according to microsoft you should use _getch() instead of getch() as it is deprecated.

[Duthomhas]

"Deprecated" according to Microsoft. It's still part of the old (ancient) MS DOS terminal functions. That particular function lives on for its simple utility, and I assumed (incorrectly) that you were using the <conio.h> library. Sorry.

[edit] That is, there is a commonly-known function with the same name... Most people won't think of NCurses's getch() first...[/edit]

NCurses is superior, so keep using it. (Good job!)

[heyyouyesyouiloveyou]

Thank you Duoas and giblit!