How can I get this to work? It's Java

[darth_bubbles]

class test{	public static String method (String arg)	{		switch(arg){			case "hello":	return "hello";				break;			case "goodbye":	return "goodbye";				break;			default: return "Error";				break;		}	}	public static void main(String[] args)	{		System.out.println(method(hello) + method(goodbye));	}}

This is copy/pasted from my code but with the variable names changed and stuff. It still gives me the same error "Cannot find symbol"

[MrSuperb]

switch/case only works with chars/ints

[fizzlesticks]

System.out.println(method(hello) + method(goodbye));

Needs quotes around the strings.

[darth_bubbles]

switch/case only works with chars/ints

Oh Really? okay.

[MrSuperb]

Oh Really? okay.

 

oh, I am sorry. With java 7 and on strings should also work

[darth_bubbles]

Needs quotes around the strings.

class test{	public static String method (String arg)	{		switch(arg){			case "hello":	return "hello";				break;			case "goodbye":	return "goodbye";				break;			default: return "Error";				break;		}	}	public static void main(String[] args)	{		System.out.println(method("hello") + method("goodbye"));	}}

I am gettig this now:

test.java:7: error: unreachable statement				break;				^test.java:9: error: unreachable statement				break;				^test.java:11: error: unreachable statement				break;				^test.java:13: error: missing return statement	}	^4 errors

[MrSuperb]

class test

{

public static String method (String arg)

{

switch(arg){

case "hello":

return "hello";

case "goodbye":

return "goodbye";

default:

return "Error";

}

}

public static void main(String[] args)

{

System.out.println(method("hello") + method("goodbye"));

}

}

[darth_bubbles]

class test{	public static String method (String arg)	{		switch(arg){			case "hello":					return "hello";			case "goodbye":					return "goodbye";			default: 				return "Error";		}	}	public static void main(String[] args)	{		System.out.println(method("hello") + method("goodbye"));	}}

So I don't need a break; because return exits it already?

[MrSuperb]

So I don't need a break; because return exits it already?

 

yes. return always exits the current method.

 

you could also write something like this:

then you need the "break;"s

class test{	public static String method (String arg)	{        String result;		switch(arg){			case "hello":					result = "hello";				break;			case "goodbye":					result = "goodbye";				break;			default:				result = "Error";		}        return result;	}	public static void main(String[] args)	{		System.out.println(method("hello") + method("goodbye"));	}}

[darth_bubbles]

 

yes. return always exits the current method.

 

you could also write something like this:

then you need the "break;"s

class test{	public static String method (String arg)	{        String result;		switch(arg){			case "hello":					result = "hello";				break;			case "goodbye":					result = "goodbye";				break;			default:				result = "Error";		}        return result;	}	public static void main(String[] args)	{		System.out.println(method("hello") + method("goodbye"));	}}

Is there a benefit to doing it this way?