在列表列表中创建一个新的空列表

Question

我试图用这段代码做的是它让用户在列表中的 2 个项目之间进行选择，并且所选项目被带到列表列表中的新列表中。 我只是想知道我的新列表创建的实现是否正确。 我在程序中有另一个不相关的错误，所以我无法测试实现是否有效。

public static void compareTwo(List<List<String>> list){
       int result;
        for (int j = 0;j<list.size();j++) {
            if ((list.get(j).size() / 2) % 2 == 0) {
                for (int i = 0; i < list.get(j).size(); i = +2) {
                    String x = list.get(j).get(i);// gets stuff inside the inner list
                    String y = list.get(j).get(i+1);
                    System.out.println("1 ("+x+") or 2 ("+y+")" );
                    result = Integer.valueOf(scan.nextLine());

                    list.add(j + 1, new ArrayList<String>() {
                    });// add another inner list to biglist

                    list.get(j+1).clear();//clears the new inner list

                    if (result==1){
                        String z = list.get(j).get(i);
                        list.get(j+1).add(z);
                    }
                    else{
                        String z = list.get(j).get(i+1);
                        list.get(j+1).add(z);
                    }


                }
    }

Answer 1

list.add(j + 1, new ArrayList<String>() {
});// add another inner list to biglist

list.get(j+1).clear();//clears the new inner list

这是创建ArrayList的匿名子类：除非您有充分的理由这样做，否则不要这样做。 此外，您不需要清除新创建的ArrayList ，它已经是空的。

因此，您可以将此代码简化为：

list.add(j + 1, new ArrayList<>());

Answer 2

我可以在您的代码中看到两个严重的问题：

1. 由于以下行试图访问索引i + 1处的值，因此for循环必须是for (int i = 0; i < list.get(j).size() - 1; i = +2)以避免IndexOutOfBoundsException . 在您的代码中，它是for (int i = 0; i < list.get(j).size(); i = +2) 。

String y = list.get(j).get(i+1);

2. 在以下行中，您没有处理用户输入非整数值时的异常。

result = Integer.valueOf(scan.nextLine());

它应该像

boolean valid;
do {
    valid = true;
    try {
        result = Integer.parseInt(scan.nextLine());
    } catch (NumberFormatException e) {
        System.out.println("This is not an integer. Please try again.");
        valid = false;
    }
} while (!valid);

除了这两个严重的问题之外，以下代码片段中的第二行没有任何意义，因为在新添加的内部列表中没有要清除的元素

list.add(j + 1, new ArrayList<String>() {});// add another inner list to biglist    
list.get(j+1).clear();//clears the new inner list

Answer 3

您可以使用此版本以实现简单性和可读性

List<String> inner = new ArrayList<>();
list.add(j + 1, inner);

，或者一些java的双括号初始化

list.add(j + 1, new ArrayList<>(){{ 
  add("hello"); 
  add("world");
}});