Thread: loop problem

This code works for like one iteration and then it quits working-I donno why.

Code:

#include <iostream.h>
#include <stdlib.h>
#include <time.h>
#include <ctype.h>

int commandmain();

void funcroll6(void)
{
int roll6=0, num6roll=0, temproll;

cout<<"Enter a number of 6-sided dice to roll:";
cin>>num6roll;

while (num6roll>0)
{
temproll=(rand() % 6+1);
roll6=temproll+roll6;
num6roll = num6roll - 1;
}

cout<<"\nYou rolled for a total of "<<(roll6)<<"!";

}


int main()
{

time_t seconds;

time(&seconds);

srand((unsigned int) seconds);


while (commandmain()==0)
commandmain();

return 0;
}

int commandmain()
{

char command[256], commain=0;

cout<<"\nEnter a command:";
cin.getline(command, 256, '\n');

if(!strcmpi("dice", command))
{
funcroll6();
}
else if(!strcmpi("roll dice", command))
{
funcroll6();
}
else if(!strcmpi("roll them bones", command))
{
funcroll6();
}
else if(!strcmpi("help", command))
{
cout<<"\nSince this is a dice program, try ''roll dice'' or ''dice'' or ''roll them bones''...And yes, the dragon is out sick.";
}
else if(!strcmpi("quit", command))
{
commain=-1;
}
else
{
cout<<"\nEnter help for assistance:";
commain=0;
}

return (commain);
}

Use breakpoints and determine where the loop fails. Variable commain is declared as a char, not int. Make it an int.

Again, determine where the loop fails first then post.

Kuphryn

its failing right after it rolls the dice

Do you mean this line?

Code:

else if(!strcmpi("roll dice", command))
   funcroll6();
   // Failure

Kuphryn

are u implying that something is wrong with that?

No. I need to know where the program fails, i.e. line #.

Kuphryn

It runs once, I can do any command, but after I roll dice it gives me Enter a command: Enter help for help.. Enter a command: and then none of the commands work anymore

hello?

Originally posted by Noobie
hello?

Please don't post one liners, specially when they are not needed. Patience is a virtue.

Run it to where it stops, right click in the window and select "select all". When it highlights press enter, then come here and paste in code tags. It'll show use the state of the screen at programs error.

First of all, responding to your own thread saying 'hello' is just plain annoying. People will respond when they feel they should, not when you feel they should.

Secondly. Look at the logic of your code here. commain is default set to 0 on function call. You do your input, it does it, then it exist's your while loop in main, and calls your function once again, takes input, and then quits.

Learn how to make a flow chart.

The current code is below

[code]
Microsoft Windows XP [Version 5.1.2600]
(C) Copyright 1985-2001 Microsoft Corp.

C:\BC5\BIN\stuff>num6roll.exe

Enter a command:h

Enter help for assistance:
Enter a command:help

Since this is a dice program, try ''roll dice'' or ''dice'' or ''roll them bones
''
Enter a command:dice
Enter a number of 6-sided dice to roll:36

You rolled for a total of 114!
Enter a command:
Enter help for assistance:
Enter a command:dice
Enter a number of 6-sided dice to roll:36

You rolled for a total of 119!
Enter a command:
Enter help for assistance:
Enter a command:
[code]

Code:

#include <iostream.h>
#include <stdlib.h>
#include <time.h>
#include <ctype.h>

int commandmain();

void funcroll6(void)
{
int roll6=0, num6roll=0, temproll;

cout<<"Enter a number of 6-sided dice to roll:";
cin>>num6roll;

while (num6roll>0)
{
temproll=(rand() % 6+1);
roll6=temproll+roll6;
num6roll = num6roll - 1;
}

cout<<"\nYou rolled for a total of "<<(roll6)<<"!";

}


int main()
{

time_t seconds;

time(&seconds);

srand((unsigned int) seconds);


while (commandmain()==0)
commandmain();

return 0;
}

int commandmain()
{

char command[256], commain=0;

cout<<"\nEnter a command:";
cin.getline(command, 256, '\n');

if(!strcmpi("dice", command))
{
funcroll6();
}
else if(!strcmpi("roll dice", command))
{
funcroll6();
}
else if(!strcmpi("roll them bones", command))
{
funcroll6();
}
else if(!strcmpi("help", command))
{
cout<<"\nSince this is a dice program, try ''roll dice'' or ''dice'' or ''roll them bones''";
}
else if(!strcmpi("quit", command))
{
commain=-1;
}
else
{
cout<<"\nEnter help for assistance:";
commain=0;
}

return (commain);
}