Thread: A question about for loops

your program is a bit messed up, but the answer to your question is you have an infinite for loop.

Try outputting i after the outer if statement closing brace within the loop and see what happens

OP's program is supposed to represent a game where there is a 'winning number' and the user has ten attempts to guess it correctly. The output is attempting to count down the number of tries remaining / used after each guess

@rogster001: Thanks

@OP:

1)
> intnumber[2];
> cin >> number[0];

Why would you use an array under such circunstances? Considering the input is supposed to be solely 1 number, you'd be better off doing it this way:
int number;
cin>>number;

2)
>if(number[0] == 1)

I don't understand this piece of code. The program is supposed to close if the user inserts a number other than 1?

3)
Your loop itself is kinda messy. You'd be better off doing something like this:

Code:

//why from 0 to 9(Or from 1 to 10)? i equals to the number of tries and should beggin with the 1st try, which is i=1 or i = 0 + 1
for (int i = 0; i < 10; i++)
    {
        //Logicly the number of attempts goes from the lowest to the highest, ain't that right?
        //eg. 1st attempt, then 2nd attempt, etc etc.
        switch (i)
        {
        case 0:
            std::cout << "This is your 1st attempt\n";
            break;
        case 1:
            std::cout << "This is your 2nd attempt\n";
            break;
        case 2:
            std::cout << "This is your 3rd attempt\n";
            break;
        default:
            std::cout << "This is your " << i + 1 <<"th attempt\n";
            break;
        }
        
        //Here you get to input a number everytime the loop goes
        int number;
        std::cout << "Enter a number: ";
        std::cin >> number;
        std::cin.ignore();




        //The following if conditions check whether the 10 attempts have been reached or if the user inserted the correct number (2)
        if (number == 2)
        {
            std::cout<<"You won!";
            break;
        }
        
        if (i == 9)
        {
            std::cout<<"You lose";
            break;
        }
    }

So your loop start at ten, goes down (i.e. 10, 9, 8 ...) and keeps going so long as the number is less than 20.
Is the problem obvious yet?

//Logicly the number of attempts goes from the lowest to the highest, ain't that right? //eg. 1st attempt, then 2nd attempt, etc etc.

Yes thats right, but the OP had the logic / language back to front, in more ways than one. Probably the idea was to count down like having a number of lives remaining

Read iMalc's reply again, and get rid of the array. Declare new variables as needed close to where you need them.

you still have the logic in your for loop wrong!