打印屏幕上的一个重复循环

Question

我有一个重复结构在我的Java类，想要呈现的数据如下：

Peter Black 
John Red 

我不知道，如果结构是正确的，因为如果我离开显示颜色只，数据被覆盖

public class Test { 

    public static void main(String[] args) { 
     List<Person> persons = new ArrayList<>(); 

     String[] names= {"Peter", "John"}; 
     String[] colors= {"Black", "Red"}; 

     for (String name: names) { 
      Person d = new Person(); 

      d.setName(name); 

      for (String color: colors) { 
       d.setColor(color); 
      } 
      persons.add(d); 
     } 

     for (Person a : persons) { 
      System.out.println(a.getName() + "-" + a.getColor()); 
     } 
    } 
} 

控制台：

Peter-Red 
John-Red 

Answer 1

不是使用嵌套for循环，而不是您正在尝试完成的循环，同时循环两个数组。

if (names.length != colors.length) { 
    // error! not a 1:1 relationship 
    return; 
} 
for (int i = 0; i < names.length && i < colors.length; i++) { 
    String name = names[i], color = colors[i]; 
    Person d = new Person(); 
    d.setName(name); 
    d.setColor(color); 
    persons.add(d); 
} 

我只是做i < names.length，但是这将打破，如果两个数组的大小不同，所以i < names.length && i < colors.length将确保i不会超过或者两个数组的长度。

编辑： 我认为这里真正的问题是你如何存储你的信息。为什么使用两个字符串数组，而不是包含Person对象的集合？

Answer 2

注意，你的颜色循环是INS IDE你的名字循环。因此，对于每个名称，您将其设置为第一种颜色的颜色，然后是第二种颜色，然后您将转到下一个名称。

你最大的问题可能是使用先进的for循环，它不会给你一个索引。如果你开始：

for(int i=0;i<names.length;i++) 

的解决方案将可能是明显的（和简化代码）

（我愿意付出更多，但它看起来像是你正在努力学习用，所以我会尽力为了避免剧透）

重新您的评论 - 如果你想简单地创建只有全部填写对象，请使用

for(int i=0;i<Math.min(names.length, colors.length);i++) 

如果你想创建的部分对象，请使用Math.max和处理数组长度小于i的情况。这是所有的业务逻辑代码...定义你想要做什么，并做到这一点。

如何处理坏数据（原问题的不部分）：

如果你想处理，其中阵列是不平等的，如果你使用上面的“Math.max”的解决方案，然后加入的情况下，在循环内部可能看起来像这样：

if(names.length < i) 
    d.setName(names[i]) 
if(colors.length < i) 
    d.setColor(colors[i]) 

这种方式没有例外，您可以安全地设置值。

但是，由于这会使对象无效，因此最好不要让用户使用不同数量的颜色和名称。如果他输入3个名字，然后扔掉空白的颜色，并保持重新提示，直到你有3种颜色 - 然后停止提示。尽快发现这样的问题总是最安全的。

另一个好的解决方法，先从：

if(names.length != colors.length) 
    throw new IllegalArgumentException("createUsers method must have the same number of names as colors but was called with "+names.length+" names and "+colors.length+" colors") 

确保您的参数是正确的，并抛出一个异常，如果他们不简化你的代码了很多，通常是一个很好的做法。

Answer 3

停止在名称循环内的颜色数组上循环。传统的基于索引的循环应该帮助你从每个阵列得到相同的名称和颜色：

public class Test { 

    /** 
    * @param args the command line arguments 
    */ 
    public static void main(String[] args) { 
     // TODO code application logic here 
     List<Person> persons = new ArrayList<>(); 

     String[] names= {"Peter", "John"}; 
     String[] colors= {"Black", "Red"}; 

     for (int i=0; i<names.length; i++) { 
      Person d = new Person(); 
      d.setName(names[i]); 
      d.setColor(colors[i]); 
      persons.add(d); 
     } 

     for (Person a : persons) { 
      System.out.println(a.getName() + "-" + a.getColor()); 
     } 
    } 
} 

Answer 4

可以使用枚举像

enum Entity { 
     Peter("Black"), 
     John("Red"); 

     private final String color; 

     Entity(String color) { 
      this.color = color; 
     }