-1

Пишу свой список, не могу переопределить список. Вот мой метод

public String toString(){
   Konj temp = head; return ("[ ");
    while (temp.next != null) {
       return(" "+temp.values);
        temp = temp.next;
    }
    return(" ]");
}

Почему возникает ошибка unreachable statement?

2
  • 1
    С JAVA не сильно знаком, но разве так и должно быть - после Konj temp = head; сразу же return ("[ "); Ведь остальная часть не выполнится. Или я не прав?
    – BOPOH
    7 фев 2013 в 16:52
  • @BOPOH, ты прав, от языка не зависит.
    – dlarchikov
    7 фев 2013 в 18:12

1 ответ 1

6

Потому что в первой же строке вы возвращаете значение "[ " из этого метода и программа возвращается к вызывающему этот код методу.

Вы сначала всё конкатенируйте и только потом делайте return:

{
   String result = "";
   Konj temp = head; 
   result += "[ ";
   while (temp.next != null) 
   {
      result += " " + temp.values;
      temp = temp.next;
   }
   result += " ]";
   return result;
}

UPDATED: как подсказывает @VladD, можно использовать класс StringBuilder, дабы увеличить производительность этого метода. Делаем это так:

{
   StringBuilder result = new StringBuilder("");
   Konj temp = head; 
   result.append("[");
   while (temp.next != null) 
   {
      result.append(" ").append(temp.values);
      temp = temp.next;
   }
   result.append(" ").append("]");
   return result;
}
2
  • ... а ещё лучше использовать StringBuilder.
    – VladD
    7 фев 2013 в 18:03
  • 1
    А ещё для красоты после открывающей скобки не нужен пробел, он и так будет добавлен в цикле.
    – VladD
    7 фев 2013 в 18:04

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.