Проблема с возвратом строки из вложенного цикла

Question

Я искал другие примеры в StackOverflow, которые могли бы ответить на мой вопрос, но ни один из ответов в других вопросах не касался цикла вложенных циклов. Я делаю код переводчик Морзе, а сама программа работает нормально, если я это сделать:

public static void StringtoMorse(String str){ 
    char Alphabet [] = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z', ' '}; 
    String MorseCode [] = {".-", "-...", "-.-.", "-..", ".", "..-.", "--.", "....", "..", ".---", "-.-", ".-..", "--", "-.", "---", ".--.", "--.-", ".-.", "...", "-", "..-", "...-", ".--", "-..-", "-.--", "--..", "|"}; 


    for (int i = 0; i < str.length(); i ++){ 
     for (int j = 0; j < Alphabet.length; j ++){ 
      if (str.charAt(i) == Alphabet[j]){ 
       System.out.print(MorseCode[j] + " "); 
      } 
     } 
    } 


} 

Но я хочу, чтобы сделать метод так, что он возвращает строку (Азбука Морзе [J] + "«) , Так вот как я отредактировал свой метод для этого:

public static String StringtoMorse(String str){ 
    char Alphabet [] = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z', ' '}; 
    String MorseCode [] = {".-", "-...", "-.-.", "-..", ".", "..-.", "--.", "....", "..", ".---", "-.-", ".-..", "--", "-.", "---", ".--.", "--.-", ".-.", "...", "-", "..-", "...-", ".--", "-..-", "-.--", "--..", "|"}; 


    for (int i = 0; i < str.length(); i ++){ 
     for (int j = 0; j < Alphabet.length; j ++){ 
      if (str.charAt(i) == Alphabet[j]){ 
       return(MorseCode[j] + " "); 
      } 
     } 
    } 


} 

Но это приводит к ошибке компиляции. Ошибка говорит: «Этот метод должен возвращать результат типа String», но я думал, что (MorseCode [j] + «») является строковым типом. Я знаю, что MorseCode [j] должен быть строкой, потому что я определил MorseCode как массив String.

Если я использую первый метод (с помощью метода System.out.println()), он правильно возвращает результат.

Answer 1

Вашего метод должны иметь обратные инструкции после для петель тоже в том случае, для петель никогда не вводится или входная строка содержит символ, который не соответствует ни одному из Alphabet (в этом случае условие if никогда правда).

public static String StringtoMorse(String str){ 
    char Alphabet [] = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z', ' '}; 
    String MorseCode [] = {".-", "-...", "-.-.", "-..", ".", "..-.", "--.", "....", "..", ".---", "-.-", ".-..", "--", "-.", "---", ".--.", "--.-", ".-.", "...", "-", "..-", "...-", ".--", "-..-", "-.--", "--..", "|"}; 


    for (int i = 0; i < str.length(); i ++){ 
     for (int j = 0; j < Alphabet.length; j ++){ 
      if (str.charAt(i) == Alphabet[j]){ 
       return(MorseCode[j] + " "); 
      } 
     } 
    } 
    return null; 
} 

Однако, я не уверен, что ваш метод делает то, что вы хотите, чтобы это сделать, так как это было бы просто вернуть код Морзе первого символа в строке ввода. Вероятно, вы хотите перевести всю строку ввода в Morse.

Вот альтернативная реализация, которая будет конвертировать всю входную строку Морзе:

public static String StringtoMorse(String str){ 
    char Alphabet [] = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z', ' '}; 
    String MorseCode [] = {".-", "-...", "-.-.", "-..", ".", "..-.", "--.", "....", "..", ".---", "-.-", ".-..", "--", "-.", "---", ".--.", "--.-", ".-.", "...", "-", "..-", "...-", ".--", "-..-", "-.--", "--..", "|"}; 

    StringBuilder morse = new StringBuilder(); 
    for (int i = 0; i < str.length(); i ++){ 
     for (int j = 0; j < Alphabet.length; j ++){ 
      if (str.charAt(i) == Alphabet[j]){ 
       morse.append(MorseCode[j]); 
       morse.append(' '); 
      } 
     } 
    } 
    return morse.toString(); 
} 

Конечно, вы можете сделать его более эффективным и устранить внутренний цикл:

public static String StringtoMorse(String str){ 

    String MorseCode [] = {".-", "-...", "-.-.", "-..", ".", "..-.", "--.", "....", "..", ".---", "-.-", ".-..", "--", "-.", "---", ".--.", "--.-", ".-.", "...", "-", "..-", "...-", ".--", "-..-", "-.--", "--..", "|"}; 

    StringBuilder morse = new StringBuilder(); 
    for (int i = 0; i < str.length(); i ++){ 
     if (str.charAt(i) >= 'a' and str.charAt(i) <= 'z'){ 
      morse.append(MorseCode[str.charAt(i)-'a']); 
      morse.append(' '); 
     } else if (str.charAt(i) == ' ') { 
      morse.append("| "); 
     } 
    } 
    return morse.toString(); 
}