Java Experts Please Come In?

Question

Please correct my method below

private boolean scanCapital (String s)
{
System.out.println(s);
boolean result = true;
String AZ = "AIUEO";
for (int i = 0; i<= AZ.length()-1 && result; i++)
{
if (AZ.indexOf(s.charAt(0)) != -1)
result = true;
}
return result;
}

Answer 1

public boolean scanCapital (String s)
{
System.out.println(s);
boolean result = true;
String AZ = "AIUEO";
for (int i = 0; i<= AZ.length()-1 && result; i++)
{
if (AZ.indexOf(s.charAt(0)) != -1)
result = true;
}
return result;
}

Answer 2

First of all, your method name doesn't really convey what the method is for or why it returns a boolean. Perhaps a better name would be "beginsWithUppercaseVowel(String s)".

Anyway, here is my revision:
private boolean scanCapital(String s) {
System.out.println(s);
boolean result = false;//presumed until determined
String AZ = "AIUEO";
for (int i = 0; i <= AZ.length() - 1 && !result; i++) {
if (AZ.indexOf(s.charAt(0)) != -1){
result = true;
}
}
return result;
}

Of course, this could all be replaced with the existing String method matches():

String s = ...;//your string here
String regex = "[AIUEO]";//regular expression
boolean uppercaseVowel = s.substring(0,1).matches(regex);

Note: I leave it to you to verify that the String contains at least one character. Also, if your browser truncates the text or code, just hover your mouse over the ellipses (...) and a popup will show the full text.

Answer 3

Inside the for, u have to put one more else part for ur if, because no where u are reinitialising result variable to false.
This is the mistake i found.

Answer 4

setText public void setText(String t)gadgets the textual content it really is equipped by technique of this textual content area of be the wanted textual content. Parameters: t - the hot textual content; if this parameter is null then the textual content is wanting to the empty string "" See also: getText()