I dont know why this dosent work?????

ssj2 · 2003-05-02T14:45:09

Ok first Im VERY new to c++. Now ive been working on a simple DOS die rolling program (basicly you would type in a number and it would give you one between one and the numer you typed) it took forever for me to get it to compile. When I finly got it to do that it ddent even do what I wanted it to do. I hope you can help me. #include <iostream> #include <stdlib.h> int main(int argc, char *argv[]) { int roll = 1; int no = 0; int sides = 1; std::cout << "select the number of sides you want to roll!"; std::cin >> sides; if(sides > no) { roll = rand() % sides; std::cout << roll; } else if(sides = no) { std::cout << "try again buddy!:/n"; } return 0; } And there is the code please help me.

For Beginners

Started by ssj2 May 01, 2003 02:19 PM

16 comments, last by ssj2 21 years ago

Wrudyn

122

May 01, 2003 03:01 PM

It closes because like Pipo said you need to add system("PAUSE") to the end. This will add a "Press any key to continue..." prompt at the end of your program. To make the program ask again after you enter a 0, you need tu use a loop. I would do something like this:

int sides = 0std::cout << "select the number of sides you want to roll!";  while (sides <= 0){   std::cin >> sides;   if(sides > no)   {      ...   }   else if(sides <= no)   {      std::cout << "try again buddy!:/n";   }}system("PAUSE");

[edited by - Wrudyn on May 1, 2003 4:06:01 PM]

ssj2

122

Author

May 01, 2003 03:07 PM

Thanks much everyone it works pretty well now.

NewbieGamer

122

May 01, 2003 04:31 PM

Sorry... didn't notice that you are done with it...

[edited by - newbiegamer on May 1, 2003 5:35:21 PM]

Cheers!!!V!

Betrayer_of_Code

130

May 01, 2003 06:07 PM

Ummm...Why dont you add "using namespace std;" so you dont have to add std:: before all you cin''s and cout''s? It takes a heck of a lot less time

BoC HomepageLuck is a Horse to ride like any other...Luckily im not a gambler, I dont know how to ride.

ph33r

380

May 01, 2003 09:50 PM

quote:Original post by Betrayer_of_Code
Ummm...Why dont you add "using namespace std;" so you dont have to add std:: before all you cin''s and cout''s? It takes a heck of a lot less time

You shouldn''t use the line

using namespace std;

When you do that all function, classes, and variables that are within std namespace are now avaialable. You will eventually run into the same problem that was resolved by putting the standard in the std namespace! Your just opening up the doors to global name polution.

Get into the habit of doing this

using std::string;