errors.. errrors.. more errors

line 10 is this line

Code:

---

class item1, item2, item3, item4, item5, item6, item7, item8, item9, item10, item11, item12, item13, item14, item15, item16, item17, item18, item19, item20, item21, item22, item23, item24, item25, item26, item27, item28, item29, item30;

---

the error is

Quote:

---

parse error before `,'

---

line 376 is this one

Code:

---

  gold=75, health=100, magic=60, attack=3, aim=7, defence=9, item1=wizardybook, item2=basicstaff, item3=robe, item4=manapotion, item5=hppotion, item6=hppotion, item7=hppotion, item8=hppotion, item9=hppotion, item10=empty, item11=empty, item12=empty, item13=empty, item14=empty, item15=empty, item16=empty, item17=empty, item18=empty, item19=empty, item20=empty, item21=empty, item22=empty, item23=empty, item24=empty, item25=empty, item26=empty, item27=empty, item28=empty, item29=empty, item30=empty;

---

with the error

Quote:

---

parse error before `='

---

line 398 is

Code:

---

fout >> "Inventory Item 1: " << item1 << endl;

---

with error

Quote:

---

no match for `ofstream & >> const char[19]'

---

i havnt gotta clue why this errors ocure, if u neeed to see more lines say so

First error:

You're declaring multiple classes like multiple variables. You can't do that; I have a feeling you didn't mean to use "class".

Second error:

You either missed a variable type, or you're trying to set the values of multiple predefined variables all in one line using commas. If it's the first, add the variable type. If it's the second, you need to separate them using semicolons, not commas.

Third error:

Assuming "fout" is just a typo specific to here, the first thing is an input operator; it should be an output operator.

1. class is a keyword, it looks here like you've used it to identify a class. Try using something similar, eg

Code:

---

class MyClass
{
//definition
};

MyClass item1, item2;  //etc, etc.

---

2. not so sure about the 2nd error, it may be related to the first though.

3. Assuming you have declared fout, the only problem is the confusion between insertion and extraction operators. Use this instead

Code:

---

fout << "Inventory Item 1: " << item1 << endl;

---

ofstream fout;

meaning fout is ofstream. i declare my own commands sometimes

and i used class because i get this error when i made it char

Quote:

---

`class string' used where a `char' was expected

---

then with clas si get the other error

------

for the

Code:

---

class MyClass
{
//definition
};

MyClass item1, item2;  //etc, etc.

---

i never used classes before i was just trying to get the errors fixed, but wut would take place of //definition

Quote:

---

Originally posted by Gabu
First error:

You're declaring multiple classes like multiple variables. You can't do that; I have a feeling you didn't mean to use "class".

Second error:

You either missed a variable type, or you're trying to set the values of multiple predefined variables all in one line using commas. If it's the first, add the variable type. If it's the second, you need to separate them using semicolons, not commas.

Third error:

Assuming "fout" is just a typo specific to here, the first thing is an input operator; it should be an output operator.

---

they are all alreayd named at the top where it says class item1 and so fourth

fout is ofstream

heres the entire area around error 2

Code:

---

cout << "Choose your characters name." << endl ;
cin >> player ;
cout << "Choose what kind of character to be." << endl ;
cout << "1. Mage 2. Barbarian 3. Warrior 4. Archer" << endl ;
cin >> charactertype ;
  if (charactertype == 1)
      gold=75, health=100, magic=60, attack=3, aim=7, defence=9, item1=wizardybook, item2=basicstaff, item3=robe, item4=manapotion, item5=hppotion, item6=hppotion, item7=hppotion, item8=hppotion, item9=hppotion, item10=empty, item11=empty, item12=empty, item13=empty, item14=empty, item15=empty, item16=empty, item17=empty, item18=empty, item19=empty, item20=empty, item21=empty, item22=empty, item23=empty, item24=empty, item25=empty, item26=empty, item27=empty, item28=empty, item29=empty, item30=empty;
  else if(charactertype == 2)
      gold=75, health=100, magic=0, attack=15, aim=10, defence=15, item1=hatchet, item2=axe, item3=stonesmallshield, item4=leathercloths, item5=hppotion, item6=hppotion, item7=hppotion, item8=hppotion, item9=hppotion, item10=empty, item11=empty, item12=empty, item13=empty, item14=empty, item15=empty, item16=empty, item17=empty, item18=empty, item19=empty, item20=empty, item21=empty, item22=empty, item23=empty, item24=empty, item25=empty, item26=empty, item27=empty, item28=empty, item29=empty, item30=empty;
  else if(charactertype == 3)
      gold=75, health=100, magic=9, attack=13, aim=13, defence=11, item1=dagger, item2=mediumsword, item3=bronzesmallshield, item4=bronzeplatewh, item5=hppotion, item6=hppotion, item7=hppotion, item8=hppotion, item9=hppotion, item10=empty, item11=empty, item12=empty, item13=empty, item14=empty, item15=empty, item16=empty, item17=empty, item18=empty, item19=empty, item20=empty, item21=empty, item22=empty, item23=empty, item24=empty, item25=empty, item26=empty, item27=empty, item28=empty, item29=empty, item30=empty;
  else if(charactertype == 4)
      gold=75, health=100, magic=9, attack=12, aim=16, defence=11, item1=dagger, item2=arrows, item3=leathercloths, item4=hppotion, item5=hppotion, item6=hppotion, item7=hppotion, item8=hppotion, item9=hppotion, item10=empty, item11=empty, item12=empty, item13=empty, item14=empty, item15=empty, item16=empty, item17=empty, item18=empty, item19=empty, item20=empty, item21=empty, item22=empty, item23=empty, item24=empty, item25=empty, item26=empty, item27=empty, item28=empty, item29=empty, item30=empty;

---

PHP Code:

---

if (charactertype == 1)
{
   gold=75;
   health=100; 
  magic=60;
   ...;
} 


---

You nee to add { and } and each item should be ended with ; instead of ,

Quote:

---

Originally posted by Klinerr1
for the

Code:

---

class MyClass
{
//definition
};

MyClass item1, item2;  //etc, etc.

---

i never used classes before i was just trying to get the errors fixed, but wut would take place of //definition

---

ok think of a class as an object, the types of data you need to describe that data and the different things that the object can do. The things it can do are member function and the way it is described is through data members. Something whihc might be appropriate for you might be:

Code:

---

class Barbarian
{
public:
  Barbarian( );    //default constructor
  ~Barbarian( );  //destructor

  //some typical member functions
  void setName( char* name );  //change the name
  char* getName( );  //get the Barbarians name
  void draw( );  //if it is graphical

private:
  char* name;
  int age;
  int experiencePoints;
  int magicPoints
  Weapon currentWeapon;
};


//now you can declare a Barbarian like you would declare an int, eg

Barbarian me;
me.setName( "endo" )

---

There are many other things that could go in here, the possibilities are endless - depends on you really.


One other thing, change this:

Code:

---

cout << "Choose your characters name." << endl ;
cin >> player ;
cout << "Choose what kind of character to be." << endl ;
cout << "1. Mage 2. Barbarian 3. Warrior 4. Archer" << endl ;
cin >> charactertype ;
  if (charactertype == 1)
      gold=75, health=100, magic=60, attack=3, aim=7, defence=9, item1=wizardybook, item2=basicstaff, item3=robe, item4=manapotion, item5=hppotion, item6=hppotion, item7=hppotion, item8=hppotion, item9=hppotion, item10=empty, item11=empty, item12=empty, item13=empty, item14=empty, item15=empty, item16=empty, item17=empty, item18=empty, item19=empty, item20=empty, item21=empty, item22=empty, item23=empty, item24=empty, item25=empty, item26=empty, item27=empty, item28=empty, item29=empty, item30=empty;
  else if(charactertype == 2)
      gold=75, health=100, magic=0, attack=15, aim=10, defence=15, item1=hatchet, item2=axe, item3=stonesmallshield, item4=leathercloths, item5=hppotion, item6=hppotion, item7=hppotion, item8=hppotion, item9=hppotion, item10=empty, item11=empty, item12=empty, item13=empty, item14=empty, item15=empty, item16=empty, item17=empty, item18=empty, item19=empty, item20=empty, item21=empty, item22=empty, item23=empty, item24=empty, item25=empty, item26=empty, item27=empty, item28=empty, item29=empty, item30=empty;
  else if(charactertype == 3)
      gold=75, health=100, magic=9, attack=13, aim=13, defence=11, item1=dagger, item2=mediumsword, item3=bronzesmallshield, item4=bronzeplatewh, item5=hppotion, item6=hppotion, item7=hppotion, item8=hppotion, item9=hppotion, item10=empty, item11=empty, item12=empty, item13=empty, item14=empty, item15=empty, item16=empty, item17=empty, item18=empty, item19=empty, item20=empty, item21=empty, item22=empty, item23=empty, item24=empty, item25=empty, item26=empty, item27=empty, item28=empty, item29=empty, item30=empty;
  else if(charactertype == 4)
      gold=75, health=100, magic=9, attack=12, aim=16, defence=11, item1=dagger, item2=arrows, item3=leathercloths, item4=hppotion, item5=hppotion, item6=hppotion, item7=hppotion, item8=hppotion, item9=hppotion, item10=empty, item11=empty, item12=empty, item13=empty, item14=empty, item15=empty, item16=empty, item17=empty, item18=empty, item19=empty, item20=empty, item21=empty, item22=empty, item23=empty, item24=empty, item25=empty, item26=empty, item27=empty, item28=empty, item29=empty, item30=empty;

---

to a switch statement:

Code:

---

switch( charactertype )
{
case 1:
  //assign all that stuff for a Mage.
  break;

case 2:
  //assign all that stuff for a Barbarian.
  break;

case 3:
  //assign all that stuff for a Warrior.
  break;

case 4:
  //assign all that stuff for an Archer.
  break;

default:
  break;
}

---

after seeing i ahve screwed up in a lot of places i am going to rewrite the code. I want d to avoid having more than one line in the if tree but oh well. im gonna do that set to thing.

thanks for your allls support + examples

wust the difference between the public: and the private: in your class example

i need the varibles in that class thing to be noticed before int main(); will it work the same way

Quote:

---

void draw( ); //if it is graphical

---

wut does if it is grpahical mean?

error= no match for `ofstream & >> const char[9]'
line= fout >> "Item 1: " << item1 << endl;

fout is ofstream

item 1 is a string type varible holding this data

item1="dagger";

You have a ">>" where it should be a "<<" there.

oh **** ur right i hate those out/input operators

wust the difference between the public: and the private: in your class example

public - Anything has access to the data.
private - Only member functions of the class have access.
protected - Only the base class and classes derived from it have acces.

Eg.[CODE]
class DOG
{

Eg.

Code:

---


class DOG
{
public:
        int GetAge();
        virtual ~Dog();
private:
        int Age;
};

---

Nothing can access 'Age' except the member function GetAge().
Even derived classes won't have access to 'Age'.
To give acces to derived classes, 'private' should be changed to 'protected'

Quote:

---

Originally posted by Klinerr1
i need the varibles in that class thing to be noticed before int main(); will it work the same way
wut does if it is grpahical mean?

---

If you are planning on making your program with graphics, then you will need a draw( ) function to draw the object to the screen. I wouldn't worry about it, it doesn't seem like your planning on doing this.

so dont use that line?