C# Replace a goto statement with an if-else

Question

I'm trying to replace this block of code with any other loop (I thought of while and do, but for some reason I didn't get the logic down entirely.

repeat:
   ... 
   if (condition)
   {
     goto repeat
   }
   else
   {
     ...
   }

Can someone help me out with the logic here? I saw a few posts about replacing goto statements, but they only relied on a single if with no else's.

The thing that is messing my thinking up is the fact that there isn't anything inside the if statement, just the goto. If I were to try to translate it to a while statement, it leaves me with this:

while (condition)
{
   // don't know what goes here since there is nothing but goto in the if statement
}
// else stuff

Thanks

The comment in the while condition is wrong. This is should be what you have between "repeat:" and "if (condition)" — the_lotus, Sep 30 '13 at 18:55

score 3 · Accepted Answer · edited Sep 30 '13 at 18:49

3

It's simple do-while loop:

do
{
    // code between "repeat:" and the if here
} while (condition);

// else code here

edited Sep 30 '13 at 18:49

Scott Chamberlain

124,994
33
282
431

answered Sep 30 '13 at 18:43

MarcinJuraszek

124,003
15
196
263

I think this is the best way to go, because the condition check is at the end. This is only because within the if statement (near goto) is no code. +1 – Jeroen van Langen Sep 30 '13 at 18:48
1

I edited your answer to make it a little more obvious what the `// ...` represents. – Scott Chamberlain Sep 30 '13 at 18:50
I was able to translate it thanks to Jeroen's explanation and editing. It makes sense now. Thanks :) – AzureFrost Sep 30 '13 at 18:58

C# Replace a goto statement with an if-else

1 Answers1