Почему я получаю сообщение об ошибке «Этот метод должен возвращать результат типа ...»?

Question

Может ли кто-нибудь понять, почему я получаю сообщение об ошибке «Этот метод должен возвращать результат типа Card», когда я четко возвращаю эту переменную «card», которая имеет тип Card?

public Card playCard(int id){ 
    int i = 0; 
    for (Card element : hand){ 
     if (i <= hand.size()) 
     {   
      if (element.getID() == id) 
      { 
       Card card = hand.get(i); 
       hand.remove(i); 
       return card; 
      } 
      else 
      { 
       i++; 
      } 

     } 
     else 
     { 
      throw new NullPointerException("Card does not exist in  hand"); 
     } 
    } 
} 

Answer 1

Ваш метод не возвращает ничего, кроме одного возможного сценария. Он должен что-то вернуть (или выбросить исключение) в все возможных сценариев.

Я думаю вы имели в виду, чтобы сделать это:

public Card playCard(int id){ 

    for (Card element : hand) { 
     if (element.getID() == id) { 
      return element; 
     } 
    } 
    throw new SomeAppropriateException("Card does not exist in  hand"); 
} 

... но я предполагаю, что немного (я не знаю, что hand есть, но он много выглядел как List). Этот код всегда либо выполняет оператор return, либо генерирует исключение, и нет способа добраться до конца метода без какой-либо из этих вещей.

Обратите внимание, что бросание NullPointerException для условия, не вызванное null указателя является плохой идеей ^(тм). (. Это также лучше, чтобы быть последовательной в котором вы кладете { и })

Answer 2

Как намекают Tarlen, ваш код должен был бы быть изменен так:

public Card playCard(int id){ 
    int i = 0; 
    for (Card element : hand){ 
     if (i <= hand.size()) 
     {   
      if (element.getID() == id) 
      { 
       Card card = hand.get(i); 
       hand.remove(i); 
       return card; 
      } 
      else 
      { 
       i++; 
      } 

     } 
     else 
     { 
      throw new NullPointerException("Card does not exist in  hand"); 
     } 
    } 
    return null; 
} 

Я считаю, что будет учитывайте все возможные маршруты, которые ваша программа должна будет выполнять. Вы всегда должны следить за возвратом чего-то, КОГДА-ЛИБО метод может выйти. Если он может выйти, не нажав оператор return, вы увидите эту ошибку.

Answer 3

Ваш метод подписи:

public Card playCard(int id){ 

означает, что вы должны вернуть Card объект. У вашего кода только один оператор return, но есть много путей через код. Вы должны вернуть объект Card для каждого пути

Answer 4

Это потому, что если hand пуст, то значение не возвращается.

Добавить return или throw после вашего цикла for.

Answer 5

Для всего возможного пути выполнения в коде необходимо иметь оператор return по умолчанию (или исключение/ошибку) для всего метода или хотя бы один оператор return (или исключение/ошибка). Как сейчас, у вас нет ни одного из них.