Back to For Beginners

C++ Text Rpg Script Error

For Beginners

Started by Thermalwolf January 11, 2011 06:29 AM

6 comments, last by Thermalwolf 13 years, 3 months ago

Thermalwolf

100

Author

January 11, 2011 06:29 AM

So my friend and I are starting our first game, a text based rpg. Now it's only a few hours in but we are stuck with this problem.

#include "load.h"

#include <string>

#include <iostream>



static enum StringValue(choice)

{

	choice["newgame"] = choiceNewGame;

	choice["loadgame"] = choiceLoadGame;

}

When you try and compile this is the error:
load.cpp(6): error C2470: 'choice' : looks like a function definition, but there is no parameter list; skipping apparent body

I don't understand. I'm sure I have everything in the right place

BeerNutts

4,410

January 11, 2011 06:42 AM

So my friend and I are starting our first game, a text based rpg. Now it's only a few hours in but we are stuck with this problem.

#include "load.h" #include <string> #include <iostream> static enum StringValue(choice) { choice["newgame"] = choiceNewGame; choice["loadgame"] = choiceLoadGame; }

When you try and compile this is the error:
load.cpp(6): error C2470: 'choice' : looks like a function definition, but there is no parameter list; skipping apparent body

I don't understand. I'm sure I have everything in the right place

[font="calibri, verdana, tahoma, sans-serif"]static enum StringValue(choice) [font="calibri, verdana, tahoma, sans-serif"] [font="calibri, verdana, tahoma, sans-serif"]The definition above doesn't make sense. You're defining a function as a static enum (what kind of enum?) And the parameter is choice (what type of variable is choice?). [font="calibri, verdana, tahoma, sans-serif"] [font="Courier New"]I don't even think you're meaning to define a function, as it looks like you're trying to define an enum named StringValue, but you've gone about it incorrectly as well. [font="Courier New"] [font="Courier New"]Regardless, I don't think C++ supports using strings as enum indexes, so the code inside the function (or the variable declaration) won't be proper either. [font="Courier New"] [font="Courier New"]You also need to provide what "choiceNewGame" and "choiceLoadGame" mean and how they're defined (are they variables? Functions? Objects?) [font="Courier New"] [font="Courier New"]You better go back and read the chapters again as I think you've missed alot here. [font="Courier New"] [font="Courier New"]<blockquote>I'm sure I have everything in the right place [font="Courier New"]Maybe not.

My Gamedev Journal: 2D Game Making, the Easy Way

---(Old Blog, still has good info): 2dGameMaking
-----
"No one ever posts on that message board; it's too crowded." - Yoga Berra (sorta)

Thermalwolf

100

Author

January 11, 2011 06:54 AM

Ok I'm sorry I didn't explain exactly what I was doing. This file is separate to keep things clean. I'm using it in a case.

#include <iostream>

#include <string>

#include "load.h"

#include "NewGame.h"



int main()

{

	using namespace std;

	int choice;

	cout << "****************************************\n";

	cout << "* Welcome to TEXT BASED ADVENTURES!!!! *\n";

	cout << "****************************************\n";

	cout << "* NewGame  *\n* LoadGame *\n************" << endl;

	cin >> choice;



	switch (choice)

	{

	case choiceNewGame:

		newgame();

		break;

	case choiceLoadGame:

		cout << "No load game detected!" << endl;

		break;

	default:

		cout << "Not a valid entry..." << endl;

		break;

	}

	system("pause");

	return 0;

}

Theirs the main code we have so far. See if this helps in figuring out my issue unless it's still all the same as from what you've said above.

levijgraham

100

January 11, 2011 07:09 AM

We should probably show the load.h also



#ifndef LOAD_H

#define LOAD_H

static enum StringValue

{ 

	choiceNewGame,

    choiceLoadGame,

};

#endif

BitMaster

8,652

January 11, 2011 12:08 PM

If this is meant to be C++ it might help if you said what you want to do. Because at the time being I'm only sure of two things:

1) This is not legal C++

2) I have no clue what you want to do

Sappharos

140

January 11, 2011 02:41 PM

From what I can see the file containing your main function is valid C++, but please explain what you're trying to do in the other two files.

Matthew Rule
http://matthyr.wordpress.com/projects/

The Communist Duck

154

January 11, 2011 05:54 PM

I can see what you want to do. You to take user input, and then go to either new game or load game. However, you cannot take input from cin directly to an enum. Instead, try to implement something like:

*Press 'n' for a new game
*Press 'l' to load an existing game

Then:



char choice;

cin >> choice; //Read in the user input into choice



if(choice == 'n' || choice == 'N') //Lower case OR upper case

{

/*new game*/

} else if(choice == 'l' || choice == 'L')

{

/*load*/

} else { //invalid input

std::cout << "Error, choose a valid option please.\n";

}

Of course, this lacks error handling and querying for more input if error'd, and there's a function to uppercase characters (so you wouldn't have to check 'n' AND 'N') but my C++ is rusty.

Thermalwolf

100

Author

January 11, 2011 06:35 PM

Yes dude that is exactly what we wanted to do, I thought I explained and showed enough but you got it. I'll give that code a try thank you very much

C++ Text Rpg Script Error

This topic is closed to new replies.

Popular Topics

Recommended Tutorials

C++ Text Rpg Script Error

This topic is closed to new replies.

Popular Topics

Recommended Tutorials

Reticulating splines