13 replies to this topic

[xunhandmex]

if it was any language OTHER then C++ i would be able to tell you :-\ ... though all i can say is maybe check your logic... sometimes all it is is that the logic might be a little off and you might be double adding/subtracting when you don't mean too thats what i run across 1/2 the time. ... i could try and help if you post saying what the Error states.

[Jordan]

D34DM347, on Feb 28 2005, 04:33 PM, said:

Well, it doesn't matter that no one helped me, because I fixed it myself.

I took the time to edit the code you posted above and it compiles with some warnings but no errors and runs fine. You should really check into some of the warnings and try fixing them.

I would like to show you a few improvements I made to your code.

I noticed you had a variables hp1, hp2, hp3 all equal 100. You had it like this,

 int hp1, hp2, hp3, hpmain, answer;
hp1 = 100;
hp2 = 100;
hp3 = 100;

If they are all going to be the same value just set them to the value, "100" when you defined them like so,

 int hp1, hp2, hp3, hpmain, answer = 100;

I also saw in your switch statement that you used hp1, hp2, hp3. Why not just use one variable for this? hp1 for example. Unless you plan on putting this in a while loop you should just use one variable for the whole process.

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

using namespace std; 

int main() { 
     
     struct tm *local; 
     time_t t; 
     t = time(NULL); 
     local = localtime(&t); 
     
     int hp1, hp2, hp3, hpmain, answer; 
     hp1, hp2, hp3 = 100; 
     hpmain = 250;
     
     
     cout << "Do you attack skeleton 1, 2, or 3?" << endl; 
     cin >> answer; 
     
     switch(answer) { 
                    case 1: 
                         cout << "You attack skeleton 1." << endl; 
                         cout << local->tm_sec << " Damage was dealt to monster 1." << endl; 
                         hp1 = hp1 - local->tm_sec; 
                         if(local->tm_sec=0) { cout << "You missed!" << endl; }
                         break; 
                         
                    case 2: 
                         cout << "You attack monster 2." << endl; 
                         hp2= hp2 - local->tm_sec; 
                         cout << hp2 << " hit points left." << endl;
                         if(local->tm_sec = 0) { cout << "You missed!" << endl; } 
                         break; 
                              
                    case 3:
                         cout << "You attack monster 3." << endl; 
                         hp3 = hp3 - local->tm_sec; 
                         cout << hp3 << " hit points left." << endl; 
                         if(local->tm_sec =0 ){ cout << "You missed!" << endl; }
                         break; 
                              
                    }
          
     }

I

[.iCY7.]

D34DM347, on Mar 10 2005, 04:33 PM, said:

I'm increasing my post count!

i think this is gonna end up in a ban

[Faken]

Hellzdev, on Mar 10 2005, 12:40 PM, said:

D34DM347, on Mar 10 2005, 04:33 PM, said:

I'm increasing my post count!

i think this is gonna end up in a ban

It sure did!

Faken

[Indigo]

Quote

Well, it doesn't matter that no one helped me, because I fixed it myself.

Could you post your final code too? I'm curious how you did it..!

[Jaymz]

Indigo, on Jul 17 2005, 09:30 AM, said:

Quote

Well, it doesn't matter that no one helped me, because I fixed it myself.

Could you post your final code too? I'm curious how you did it..!

No, he can't, he's been banned

[Indigo]

I saw that... But can somebody please delete ALL the "I'm increasing my post count!" posts? Man, they're anoying!