public static String translate(String[] sourceLanguage, String[] destinationLanguage, String toBeTranslated){
int i;
//assume the word to be translated is not translatable,
boolean isTranslatable = false;
//check every word in the sourceLanguage array for that word.
for (i = 0; i < (sourceLanguage.length); i++){
//If it is there conclude that it can be translated and hence return it's equivalent
if (toBeTranslated.equals(sourceLanguage[i])){
isTranslatable = true;
return (destinationLanguage[i]);
}
}
//if it still is not translatable, return "?????" to show confusion.
if (isTranslatable == false){
return ("?????");
}
return "This is dead code anyway";//the javac insisted on this..
}public static String translate(String[] sourceLanguage, String[] destinationLanguage, String toBeTranslated)
{
for (int i = 0; i < (sourceLanguage.length); i++)
{
if (toBeTranslated.equals(sourceLanguage[i]))
return (destinationLanguage[i]);
}
return ("?????");
}return destinationLanguage[i]; and return "?????";
isTranslatable has been set to false, even though it will always be false once that loop exits. This introduces your actual problem: The Javac is smart enouch to notice that you neglected to handle the else condition of your if statement, but not smart enough to notice that the if statement is itself completely redundant.public static String translate(String[] sourceLanguage, String[] destinationLanguage, String toBeTranslated) {
int i = lookup(toBeTranslated, sourceLanguage);
if (i < 0) { // ...then the string is not in sourceLanguage.
return "?????";
} else {
return destinationLanguage[i];
}
}
public static int lookup(String whatever, String[] inAnArray) {
for (int i = 0; i < inAnArray.length; i++) {
if (w.equals(whatever)) {
return i;
} else {
// do nothing, let the loop continue
}
}
// Wasn't found
return -1;
}