为什么我的display（）方法不能打印出所有ArrayList元素？

Question

我有一个类Container，在该类中，用户应该可以输入任意数量的单词，直到他没有键入任何内容。 我有addWord（Word）方法，每次运行do / while循环时，每个输入都会添加到ArrayList单词中。 我每次循环运行时都会将用户输入值作为参数传递给addWord（）方法。

现在，一旦do / While循环停止运行，我想使用display（）方法显示数组的所有元素。 但是由于某些原因，当我尝试调用方法display（）时，它仅显示一个空数组[]。

有什么可以帮助的吗？

import java.util.ArrayList;                                             
import java.util.List;                                              
public class Container {                                                
    private List<String> words;                                             
    public Container() {                                                
    }                                               
    public List<String> getWords() {                                                
        return words;                                               
    }                                               
    public void setWords(List<String> words) {                                              
        this.words = words;                                             
    }                                               
    public void addWord(String word) {                                              
      words = new ArrayList<String>();                                              
      words.add(word);                                          
    }                                               
    public void display() {                                             
      System.out.println(words);                    
    }                                               
}

主要方法：

import java.util.Scanner;                                               
public class ContainerMain                                              
{                                               
  public static void main(String[] args)                                                
  {                                             
    Container one = new Container();                                                
    Scanner myScan = new Scanner(System.in);                                    
    String word = "s";                          
    do                                              
      {                                             
        word = myScan.nextLine();                                               
        one.addWord(word);                                              
      }                                             
    while (!word.equals(""));

    if (word.equals("")) {                                              
      one.display();                                                
    }                                               
    else {                                              
      System.out.println("No hope fam");                
    }                                               
  }                                             
}

Answer 1

查看您的addWord方法：

public void addWord(String word) {                                              
  words = new ArrayList<String>();                                              
  words.add(word);                                          
}

每次您调用它时，它都会创建一个新列表-因此，您永远都无法在其中包含多个单词。

第一行，初始化words ，应该在构造函数中（或作为字段初始化程序）。 然后，从addWord删除该行-最好同时使words字段为final，以避免将来发生此类错误，并删除setWords方法，除非您确实需要其他方法。

在Container ，这就是所有的错误（尽管目前尚不清楚，除了直接使用List<String>之外，它是否确实提供了其他任何值）。 如注释中所述，当前您在main do / while循环将在末尾添加一个空字符串。 另外，在循环之后检查word是否为空没有必要-必须这样做，否则您就不会退出循环！