Can anyone see the problem with my code? problem understanding switch (newbish)

[keith]

//mostly taken from Java programming for dummies
// below I pasted the errors I get

import java.util.Scanner;
import java.util.Random;
import static java.lang.System.out;

class TheOldSwitcheroo {

	public static void main(String args[]);
		Scanner myScanner=new Scanner(System.in);
		Random myRandom=new Random();
		
		int randomNumber;
		
		out.print("Type you question...");
		myScanner.nextLine();
		
		randomNumber=myRandom.nextInt(4)+1;
		
		switch (randomNumber) {
		
		case 1:
			out.print("Yes, isnt it obvious");
			break;
		
		case 2:
			out.print("No way!");
			break;
		
		case 3:
			out.print("Anything is possible");
			break
		
		case 4:
			out.print("I doubt it");
			break;
		
		default:
			out.print("you think you have problems");
			out.print("the random number generator ");
			out.print("in this program is broken");
			break ;
		}
		out.println("Good Bye");

	}
}

--------------------------------------------------
 ----jGRASP exec: javac -g TheOldSwitcheroo.java

TheOldSwitcheroo.java:13: <identifier> expected
		out.print("Type you question...");
		         ^
TheOldSwitcheroo.java:13: illegal start of type
		out.print("Type you question...");
		          ^
TheOldSwitcheroo.java:14: <identifier> expected
		myScanner.nextLine();
		                  ^
TheOldSwitcheroo.java:16: <identifier> expected
		randomNumber=myRandom.nextInt(4)+1;
		            ^
TheOldSwitcheroo.java:18: illegal start of type
		switch (randomNumber) {
		^
TheOldSwitcheroo.java:18: <identifier> expected
		switch (randomNumber) {
		                    ^
TheOldSwitcheroo.java:20: orphaned case
		case 1:
		^
TheOldSwitcheroo.java:30: ';' expected
			break
			     ^
TheOldSwitcheroo.java:42: <identifier> expected
		out.println("Good Bye");
		           ^
TheOldSwitcheroo.java:42: illegal start of type
		out.println("Good Bye");
		            ^
TheOldSwitcheroo.java:45: class, interface, or enum expected
}
^
11 errors

 ----jGRASP wedge2: exit code for process is 1.
 ----jGRASP: operation complete.

[Tolls]

One of the problems is you haven't posted your code using CODE tags, so the formatting has gone, making it harder to read.

You haven't defined "out".

ETA: Gah!
Missed the static import!
:)

It's a bracket thing...check your curly braces, especially after main()

[jlmp]

First of all, i havent seen out.println... out is a variable class of System, so you have to put

System.out.println();

Try this and tell us...

or as tolis said...
you can define out as an instance and a method inside called print() but as you are new i suppose that all you need is System.out.print();

ETA: Gah!
Missed the static import! ME2!!!!!!!!!!!!!!!!! :d Tolis you are right (once again ;) ) !

[Jnoobs]

anytime you are using a main (or any other type) of method and placing things inside, you must use { and } to open and close the function. so do

public static void main(String args[]) {

instead of

public static void main(String args[]);

use

System.out.println("blahhhhhhh")

and place a ; at the end of break in Case 2 and you should be fine

[Fubarable]

anytime you are using a main (or any other type) of method and placing things inside, you must use { and } to open and close the function. so do

public static void main(String args[]) {

instead of

public static void main(String args[]);

Jnoobs: Sharp eyes -- nice catch!

To the original poster, I've added code tags to your post. To learn how to do this yourself in future posts, read the top link in my signature below. Also a key technique when creating Java programs is to compile often and to always fix compilation problems before adding new code.

Much luck!

[Tolls]

Excuse me, but which bit of "It's a bracket thing...check your curly braces, especially after main() " is confusing?

...;)

[Fubarable]

Excuse me, but which bit of "It's a bracket thing...check your curly braces, especially after main() " is confusing?

...;)

Oops.... <sheepish grin>

[JosAH]

Excuse me, but which bit of "It's a bracket thing...check your curly braces, especially after main() " is confusing?

...;)

Too tinny; you should've used more woody words like "garlic" or "bicycle" or "matrass".

kindest regards,

Jos ;-)

[Tolls]

It's a garlic thing...check your bicycles, especially after the mattress().

[JosAH]

It's a garlic thing...check your bicycles, especially after the mattress().

Quite nice, nice and woody; like "woman's undies" ... nice and woody too ...

kind regards,

Jos ;-)