• Advertisement
Sign in to follow this  

Goto: How should I work the code out?

This topic is 1934 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Recommended Posts

Here's a[size=6] [color=#006400][b][u]Java pseudo-code[/u][/b][/color] [/size]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 [url="http://docs.oracle.com/javase/tutorial/java/nutsandbolts/branch.html"]Java doc here[/url] that said "break" keyword works for for...loop, do...while..., and switch...case... only. Anything I can do?

Share this post


Link to post
Share on other sites
Advertisement
Why can't you do this:
[CODE]
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)
}
[/CODE]
This does the same thing as your code, but doesn't use a goto statement. Unless there is something I'm missing?

Share this post


Link to post
Share on other sites
[quote name='tom_mai78101' timestamp='1349339128' post='4986685']
Here's a[size=6] [color=#006400][b][u]Java pseudo-code[/u][/b][/color] [/size]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 [url="http://docs.oracle.com/javase/tutorial/java/nutsandbolts/branch.html"]Java doc here[/url] that said "break" keyword works for for...loop, do...while..., and switch...case... only. Anything I can do?
[/quote]
[list=1]
[*]You cannot use goto in Java.
[*]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.
[/list]

Share this post


Link to post
Share on other sites
[quote name='RulerOfNothing' timestamp='1349340025' post='4986689']
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?
[/quote]

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.

[quote name='thok' timestamp='1349340478' post='4986693']

[list=1]
[*]You cannot use goto in Java.
[*]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.
[/list]
[/quote]


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.


[quote name='Ashaman73' timestamp='1349343262' post='4986702']
In your pseudocode just exchange [i]goto [/i]through [i]break[/i], that's it.

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

You mean, "break <label>"? I googled it, and it turns out this is an actual Java code. [url="http://geekycoder.wordpress.com/2008/06/25/tipjava-using-block-label-as-goto/"]Haven't seen this before.[/url] Thanks! :D

Share this post


Link to post
Share on other sites
[quote name='tom_mai78101' timestamp='1349344825' post='4986706']
You mean, "break "? I googled it, and it turns out this is an actual Java code. Haven't seen this before. Thanks! :D
[/quote]
Yes, you can label a loop (write it before the loop starts), but in your case you just need break; (without label).

Share this post


Link to post
Share on other sites
[quote name='tom_mai78101' timestamp='1349344825' post='4986706']


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.
[/quote]
If you could use goto like in your first example, then the code in booleanFlag1 would be skipped.

Share this post


Link to post
Share on other sites
[quote name='RulerOfNothing' timestamp='1349348492' post='4986718']
If you could use goto like in your first example, then the code in booleanFlag1 would be skipped.
[/quote]
Yes, that would be true. I'm sorry for being mistakened. :(

Share this post


Link to post
Share on other sites
Sign in to follow this  

  • Advertisement