Java: do while wont while

    Anfallare.armees = 10;
    Forsvarare.armees = 10;
 do{
    Anfallare.slaTarningar();
    Forsvarare.slaTarningar();

    Anfallare.visaTarningar();
    Forsvarare.visaTarningar();

    MotVarandra(Anfallare, Forsvarare);

    status(Anfallare, Forsva开发者_StackOverflow社区rare);

    }while(Anfallare.armees == 0 || Forsvarare.armees == 0);

is what I have, I tried also a

 System.out.println(Anfallare.armees + " :: " + Forsvarare.armees);

after this do while and it is 9 and 10, neither of them are 0.

Why does it run this once?

Because, as you point out, after the first run, Anfallare.armees is 9, which does not equal 0, and Forsvarare.armees is 10, which is not equal to 0.

Perhaps you meant

while(Anfallare.armees != 0 && Forsvarare.armees != 0)

while(Anfallare.armees == 0 || Forsvarare.armees == 0)

Means continue only if Anfallare.armees == 0 || Forsvarare.armees == 0. In your example, neither is 0 so it will not loop.

while(Anfallare.armees != 0 && Forsvarare.armees != 0)

Means, stop looping if either are zero (Anfallare.armees != 0 && Forsvarare.armees != 0)

You may actually be wanting:

while (Anfallare.armees != 0 || Forsvarare.armees != 0)

means, stop looping if both are 0

You need to do

while(Anfallare.armees != 0 || Forsvarare.armees != 0);

To make it loop until both of those variables are 0

Because you start as do{}, which forces to execute at least once the code, and then you have while x==0, which is not true. Since it is not true, the loop ends.

As you see in the other answers, you may need to check for x != 0 (not equal to), or even better x>0 (in the event x could take a negative number)

Take some time to read the Oracle's Java tutorials I linked in the other answer to you. It explains the loop logic very well.

If I understand you correctly, your do-while loop body is executed once, but you think it shouldn't have run at all - is this correct?

The reason is, in do-while the loop condition is checked only after the loop body is executed. I.e. it will always run at least once.

It's hard to tell without the complete code, but it may be that you just need to invert the condition:

}while(!(Anfallare.armees == 0 || Forsvarare.armees == 0));

You condition says ... as long as one of them is 0. What you'll want is this:

while(Anfallare.armees > 0 || Forsvarare.armees > 0);

the reason is simple

while loops when the expression is true.

so what you want is

while(Anfallare.armees != 0 || Forsvarare.armees != 0);

but even better I would suggest

while(Anfallare.armees > 0 || Forsvarare.armees > 0);

this would allow you to be sure that your armee count never gets negative ^^

Jason