Goto: How should I work the code out?

Started by
6 comments, last by tom_mai78101 11 years, 6 months ago
Here's a

[color=#006400]Java pseudo-code I'm working on shown below:

[source lang="java"]public void someFunction(//Some parameters) {
//some boolean flags here.
switch (p.orientation) {
case VERTICAL:
if (booleanFlag1){
if (booleanFlag2){
//Some other codes.
goto Label;
}
//Some codes
}
else if (booleanFlag3){
if (booleanFlag4){
//Some other codes.
}
//Some codes
}
reflectResponse();
break;
default:
return;
}
Label:
//Some more codes... (required)
}[/source]

I'm very tempted to use "goto" in Java, but as always, goto isn't allowed because of said issue. I will digress from that.

I only know from Java doc here that said "break" keyword works for for...loop, do...while..., and switch...case... only. Anything I can do?

Advertisement
Why can't you do this:

public void someFunction(//Some parameters) {
//some boolean flags here.
switch (p.orientation) {
case VERTICAL:
if (booleanFlag1){
if (booleanFlag2){
//Some other codes.
}
else if
{
//Some codes
}
}
else if (booleanFlag3){
if (booleanFlag4){
//Some other codes.
}
//Some codes
}
reflectResponse();
break;
default:
return;
}
//Some more codes... (required)
}

This does the same thing as your code, but doesn't use a goto statement. Unless there is something I'm missing?

Here's a

[color=#006400]Java pseudo-code I'm working on shown below:

[source lang="java"]public void someFunction(//Some parameters) {
//some boolean flags here.
switch (p.orientation) {
case VERTICAL:
if (booleanFlag1){
if (booleanFlag2){
//Some other codes.
goto Label;
}
//Some codes
}
else if (booleanFlag3){
if (booleanFlag4){
//Some other codes.
}
//Some codes
}
reflectResponse();
break;
default:
return;
}
Label:
//Some more codes... (required)
}[/source]

I'm very tempted to use "goto" in Java, but as always, goto isn't allowed because of said issue. I will digress from that.

I only know from Java doc here that said "break" keyword works for for...loop, do...while..., and switch...case... only. Anything I can do?



  1. You cannot use goto in Java.
  2. If you want to execute some code _only_ in the VERTICAL case when booleanFlag1 and booleanFlag2 are both true, don't do it this way. This is what you're trying to do, right? Instead, put this code in a separate function or method and call it inside the second `if` clause.
In your pseudocode just exchange goto through break, that's it.

Longer answer: break will break from the current for/while/switch statement, regardless if it is embedded itself in an other non-loop/switch statement, like an if-statement.

Why can't you do this:
<snip>
This does the same thing as your code, but doesn't use a goto statement. Unless there is something I'm missing?


If both "booleanFlag1" and "booleanFlag2" are true, I need to execute code in "booleanFlag2" and then execute code in "booleanFlag1". So, the extra "else if" would not be the correct way.




  1. You cannot use goto in Java.
  2. If you want to execute some code _only_ in the VERTICAL case when booleanFlag1 and booleanFlag2 are both true, don't do it this way. This is what you're trying to do, right? Instead, put this code in a separate function or method and call it inside the second `if` clause.




1. I understand that. Hence I was asking if there are any alternatives.
2. It's not just VERTICAL case, there are other more cases. And no, it's not. I wanted to execute codes when booleanFlag1 and booleanFlag2 is true, but skip the code in booleanFlag1 and the rest of the code before the "Label". If booleanFlag2 is not true, continue to execute code in booleanFlag1, and call on reflectResponse() before continuing to execute more code.



In your pseudocode just exchange goto through break, that's it.

Longer answer: break will break from the current for/while/switch statement, regardless if it is embedded itself in an other non-loop/switch statement, like an if-statement.


You mean, "break <label>"? I googled it, and it turns out this is an actual Java code. Haven't seen this before. Thanks! :D

You mean, "break "? I googled it, and it turns out this is an actual Java code. Haven't seen this before. Thanks! :D

Yes, you can label a loop (write it before the loop starts), but in your case you just need break; (without label).



View PostRulerOfNothing, on 04 October 2012 - 06:40 PM, said:
Why can't you do this:

This does the same thing as your code, but doesn't use a goto statement. Unless there is something I'm missing?

If both "booleanFlag1" and "booleanFlag2" are true, I need to execute code in "booleanFlag2" and then execute code in "booleanFlag1". So, the extra "else if" would not be the correct way.

If you could use goto like in your first example, then the code in booleanFlag1 would be skipped.

If you could use goto like in your first example, then the code in booleanFlag1 would be skipped.

Yes, that would be true. I'm sorry for being mistakened. :(

This topic is closed to new replies.

Advertisement