打印ArrayList的所有项目

Question

我知道这是一个非常基本的问题，但是我已经搜索过并且没有找到适合我问题的合适答案。 我有一个从扫描器填充的对象的ArrayList。 当我尝试打印出列表中的所有元素时，我只会使最后一个元素重复与列表中的对象一样多的次数。

首先是Object类：

public class Colors {
    String name;

    @Override
    public String toString(){
        return name.toString();
    }
}

现在的主要课程：

import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

 public class Vec { 
    public static void main(String[] args) {
    Colors col= new Colors();
    List <Colors> l = new ArrayList<>();
    boolean flag = true;

    while(flag == true){
       System.out.println("Type in name of color");
       Scanner kb = new Scanner (System.in);
       col.name = kb.next();
       l.add(col);

       for (int j= 0; j<l.size();j++) {
           Object g = (Object)l.get(j);
           System.out.println("Name of color: "+g);
       }
     }
   }
} 

我到底在做什么错？

Answer 1

您正在使用new关键字在Java中声明对象。 您仅声明一个Colors实例，然后一遍又一遍地更改其名称。 然后一遍又一遍地将相同的Colors添加到列表中。 您需要做的是更改声明Colors ，这可以在while语句中完成。

import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

public class Vec
{
    public static void main(String[] args) {
        List <Colors> l = new ArrayList<>();
        boolean flag = true;

        Scanner kb = new Scanner (System.in);

        while (flag) {
            System.out.println("Type in name of color");

            Colors col = new Colors ();
            col.name = kb.next();
            l.add(col);

            for (int j = 0; j<l.size();j++) {
                Object g = (Object)l.get(j);
                System.out.println("Name of color: "+g);
            }
        }
    }
}

此处的区别在于，现在在您的while语句的本地范围中重新声明了col 。

---

小注释，这些注释不合主题，旨在进行代码审查，而不是回答问题。

---

while (flag == true) ，无需与true比较。 您也不需要一直创建新的Scanner 。

---

我建议对您的变量使用更好的名称。 l ， col ， kb ， flag基本上可以是任何东西。

一些建议 （仅仅是这些建议）：

• l是您的列表，其中包含许多Colors ，现在Colors也具有误导性，因为它只是一种颜色，我建议将类名更改为Color ，并将列表l更改为colors 。
• 上校，很好。
• kb，请改用scanner 。
• 标志，什么？ 取而代之的是running ， playing ， on playing 。

---

另一个补充是，由于您知道l存储位置，因此可以for每一个使用a。

            for (Colors c : l) {
                System.out.println("Name of color: " + c);
            }