Boolean expression to put a name in proper form

[zhider]

I am assigned to make a boolean method that takes a name as a parameter and determines wether or not it is in common form by seeing if there is a comma (standard form is last, first. non standard is first lasy.) . After it determines this, it is supposed to put it into common form if it is not already, by calling on the method that sees wether or not it has a comma. here is my code
package name;
import java.util.Scanner;
/**
*
* @author Zack
*/
public class Name {

/**
* @param args the command line arguments
*/
public static void main(String[] args) {
Scanner reader = new Scanner (System.in);
System.out.println("What is the name you want to test? ");
String name = reader.nextLine();
hasComma(name);
}
public static boolean hasComma(String name){
boolean oName;
int x= name.indexOf(',');
if (x == -1) {
oName= false;
}else{
oName= (true);
}
return (oName);
}
public static void convertName() {
}
}

any help would be appreciated, thanks in advance.

[Junky]

What is your specific question. I see you have posted your requirements and I see you have posted your code. At a quick glance the code seems to do what you want. So what is your issue?

BTW your hasComma method can be reduced to a single line of code.

[zhider]

Well right now, after I type in the name, it does not return anything, it just says build stopped and I have no Idea what to do for the convertname method.

[Junky]

it does not return anything

Incorrect! The code does return a boolean value, it simply does nothing with it and the program ends.

[zhider]

How do I make it print the result?

[Junky]

How do you make a program print anything? Use a print statement! You can either wrap the print statement around the call to the hasComma method. Or you can assign the return value to a variable and then print the variable. If you do not know/understand either of those options then I suggest reading your text book or searching the web as there will be numerous examples of both.

[zhider]

Thank you so much for the help! you saved me. I only have one more thing to do on this code. I have to make it loop in the main method, asking for a name, until a blank is entered, what change would I have to make in main to do this? Thanks again for all of your help.

public class Name {

/**
* @param args the command line arguments
*/
public static void main(String[] args) {
Scanner reader = new Scanner (System.in);
System.out.println("What is the name you want to test? ");
String name = reader.nextLine();
convertName(name);
}
public static boolean hasComma(String name){
boolean oName;
int x= name.indexOf(',');
if (x == -1) {
oName= false;
}else{
oName= (true);
}
return (oName);
}
public static boolean convertName(String name) {
boolean convert = false;
hasComma(name);
if (hasComma(name)== true) {
convert= true;
}else{
String s = name;
String[] arr = s.split(" ");
System.out.println (arr[1]+ ","+ arr[0]);
}
return (convert);
}
}

[penguinCoder]

BTW your hasComma method can be reduced to a single line of code.

I can see how it could be cut down, but to a single line of code?.. Would you mind illustrating this, via a post or pm? I would like to see how, for my own personal knowledge...

5 lines is the shortest I see it, excluding the signature.

[Tolls]

return name.indexOf(',') > -1;

For the OP, please use [code] tags [/code] when posting code.
Many of us won't bother reading unformatted code.

[Junky]

Oh the irony!

Tolls supplied the code despite his signature. But yes, that is exactly how I would have coded it.

[gowthamgutha]

You can simply use.

if(name.contains(",")) return true;
else return false;

[Moderator edit: link removed]

[Tolls]

Oh the irony!

Tolls supplied the code despite his signature. But yes, that is exactly how I would have coded it.

I know...but I figured I'd let myself off, just this once.
:)
Anyway, the OP is clearly putting in some effort and seems to know what they're doing.

As for gowthamgutha's code, how is that an improvement on my single line?