Set Iterator toString - JAVA

Question

Я выполняю задание, и в своем классе у меня есть Set, чтобы читать несколько авторов для моего класса Book. Я пытаюсь понять, как правильно реализовать toString. Буду ли я сделать ToString, как обычно, а затем реализовать итератор, как эти:

public String toString(){ 
     Iterator itr = set.iterator(); 
     while(itr.hasNext()){ 
      Object element = itr.next(); 
     } 

     Return "Book, ISBN:" + isbn +", Author:" + element 
} 

ли, что делать какой-либо смысл?

Answer 1

Прежде всего, вы должны использовать точно такой же метод подписи ToString, который:

public String toString() 

Во-вторых, первая линия в вашей реализации является оператор возврата, который делает остальную часть ваших линий недостижим.

В-третьих, поставив оператор возврата в цикл всегда спорно.

Хотя я не совсем уверен, как вы хотите реализовать свой toString, но сначала вам нужно изменить их.

Answer 2

я бы подумал, что-то больше, как это:

@Override 
public String toString() { 
    StringBuilder sb = new StringBuilder(); 
    sb.append("Book, isbn =" + isbn + ", Author: "); 
    for (String s : authorSet) { 
     sb.append(s).append(" "); 
    } 
    return sb.toString(); 
} 

Answer 3

Вы должны соответствовать подписи точно. Используйте аннотацию @Override, чтобы убедиться, что вы это сделали. Второй взгляд на StringBuilder, чтобы создать новую строку небольшими шагами. Однако для этой проблемы вы, вероятно, можете обойтись только с +.

@Override 
public String toString(){ 
     String retString = "Book, isbn =" + isbn + ", Author:"; 
     Iterator itr = set.iterator(); 
     while(itr.hasNext()){ 
      Object element = itr.next(); 
      retString = retString + element.toString(); 
     } 
     return retString; 
} 

Есть немного способов улучшить это, я позволю вам работать над ними.