Почему Eclipse говорит мне, что это мертвый код?

Question

Я пытаюсь создать программу отзыва, которая отправляет текстовые сообщения 200 + людям, а затем выполняет поиск по электронной почте, в которой также отправляются ответы.

Этот метод предназначен для поиска списка ответов, который создается с использованием другого метода, но он работает неправильно. Он будет работать, только если самое первое сообщение в списке массивов совпадает с самым первым номером в списке контактов.

Это некоторые другие проблемы, но мой главный вопрос здесь в том, почему он говорит, что код, специфически внутри моего цикла for, является мертвым кодом?

public static boolean searchForPhone(String phone){ 
     CharSequence phoneN = phone; 
     for(int i=0;i<myMessages.size();i++){ 
       if(myMessages.get(i).contains(phone)){ 
        return true; 
       } 
       else{ 
        return false; 
       } 
     } 
     return false; 
     } 

Answer 1

В остальное части вам нужно continue искать. Иначе, если ваш кулачный элемент не соответствует, он вернет false и не будет проверять другой элемент.

public static boolean searchForPhone(String phone) { 
    CharSequence phoneN = phone; 
     for (int i = 0; i < myMessages.size(); i++) { 
      if (myMessages.get(i).contains(phone)) { 
      return true; 
      } else { 
      //return false this conditional return cause 
      // the complain it as dead code. Since for loop will become not 
      //loop 
      continue; // will search for other elements. 
      } 
     } 
     return false; 
} 

Теперь вы можете упростить этот код, чтобы следующий, потому что else часть не является действительно необходимым.

public static boolean searchForPhone(String phone) { 
    CharSequence phoneN = phone; 
    for (int i = 0; i < myMessages.size(); i++) { 
     if (myMessages.get(i).contains(phone)) { 
      return true; 
     } 
    } 
    return false; 
} 

Answer 2

Ваша петля всегда возвращается из функции в конце первой итерации. Это делает i++ мертвым кодом, так как он никогда не выполняется.

В любом случае, удалите пункт else, чтобы исправить код.

Answer 3

Будет ли это помогать?

public static boolean searchForPhone(String phone){ 
    CharSequence phoneN = phone; 
    for(int i=0;i<myMessages.size();i++){ 
      if(myMessages.get(i).contains(phone)){ 
       return true; 
      } 
    } 
    return false; 
    } 

Посмотрите, что вы перебираете n-элементный список. Когда вы получите первый элемент в списке, который вы получили, если/else. Таким образом, вы будете иметь дело с двумя вещами, ведьма ведьма возвращается. Таким образом, ваша программа выйдет из первого возвращенного элемента.

Чтобы сделать это проще, ваш код равен:

CharSequence phoneN = phone; 
if (myMessages.size() ==0){ 
return false; 
} 

return myMessages.get(0).contains(phone); 

Answer 4

Это ваш код, правильно отформатирован:

public static boolean searchForPhone(String phone) { 
    for (int i = 0; i < myMessages.size(); i++) { 
    if (myMessages.get(i).contains(phone)) { 
     return true; 
    } else { 
     return false; 
    } 
    } 
    return false; 
} 

Конструкт помечено как Dead кода является корзины в i++ for-loop header. Это действительно мертвый код, потому что тело for цикла безоговорочно возвращает метод. Поэтому «шаговая» часть заголовка для заголовка недоступна. Мертвый.

Тот же факт делает ваш код неправильным, BTW. Извлечение оговорки else будет большим улучшением.

Answer 5

Try из Window> Preferences> Java> Compiler> Error/Предупреждения

Изменить Мертвый код (например 'если (ложь)') и Ненужные 'еще' заявление к ошибке.