根据类的值将一个数组分成两个数组。 爪哇

Question

所以我有三个类， Person ， GroupOfPeople和一个Main类。

班级人：

public class Person {
    
    private String name;
    private int age;
    private char gender;
    private double weight;

    public char getGender() {
        return gender;
    }
    
    public void setGender(char gender) {
        this.gender = gender;
    }
    
    ... other getters and setters respectively
}

如您所见，Person 类有一个性别实例变量。 在GroupOfPeople我创建了一组人员，如下所示：

private Person[] personGroup = new Person[5];

这将存储所有人，无论其性别如何。 我想要的是一种将男人与数组中的所有其他人分开的方法。

因此，我创建了findMen()方法：

public void findMen() {
        int counter = 0;
        Person[] allMen = new Person[5];
        
        while (counter < personGroup.length) {
            if (personGroup[counter].getGender() == 'M') {
                allMen[counter] = personGroup[counter];
            } else {
                break;
            }
            counter++;
        }
        
        System.out.println("All the men are:");
        for (int i = 0; i < allMen.length; i++) {
            if (allMen[i] != null) {
                System.out.println(allMen[i].getName());
            }
        }
    }

这不起作用，它只向allMen数组添加了 1 个人，而我在主方法中定义了更多人：

GroupOfPeople gop = new GroupOfPeople();
        
        Person p1 = new Person();
        p1.setName("Person 1");
        p1.setAge(40);
        p1.setGender('M');
        
        Person p2 = new Person();
        p2.setName("Person 2");
        p2.setAge(30);
        p2.setGender('F');
        
        Person p3 = new Person();
        p3.setName("Person 3");
        p3.setAge(20);
        p3.setGender('M');

        gop.findMen();

输出是：

All the men are:
Person 1

虽然预期的输出是：

All the men are:
Person 1
Person 3

我希望你能给我一些关于我做错了什么的见解。

Answer 1

您的 break 语句放置不正确。 第一次你没有得到一个男人时它会中断。 当您遍历完整个 personGroup 时，中断。

Answer 2

没关系，解决了...

Person[] allMen = new Person[5];

for (int i = 0; i < personGroup.length; i++) {
            if (personGroup[i] != null) {
                if (personGroup[i].getGender() == 'M') {
                    for (int j = 0; j < allMen.length; j++) {
                        if (allMen[j] == null) {
                            allMen[j] = personGroup[i];
                            break;
                        }
                    }
                }
            }
        }

Answer 3

问题就在这里。 摆脱休息；

 while (counter < personGroup.length) {
        if (personGroup[counter].getGender() == 'M') {
              allMen[counter] = personGroup[counter];
        } else {
                break;  // get rid of this, you stop if no M is found. 
        }
            counter++;
  }

像这样试试

while (counter < personGroup.length) {
        if (personGroup[counter].getGender() == 'M') {
              allMen[counter] = personGroup[counter];
        } 
        counter++;
}

Answer 4

一旦你找到一个女性，休息就会导致它结束循环。 删除时出现 NullPointerException 的原因是因为使用了数组

ArrayList<Person> allMen = new ArrayList<Person>();
  for(Person p : personGroup){
    if(p.getGender() == 'M') allMen.add(p);
}

最后，迭代和输出 man 只需做同样的事情：

for(Person p : allMen){
  System.out.println(p.getName());
}

Answer 5

您的 else 正文包含一个break 。 这意味着当找到一个人而不是一个人时，循环结束。 您可能想在此处使用continue （或完全删除else ）。

不过我有两个建议：

列表

使用List而不是数组。 List接口的一个广泛使用的实现是ArrayList 。 优点是列表是动态大小的。 这意味着您可以在不知道容量的情况下将无限数量的项目添加到列表中。

// Convert the array to a list, if necessary. However, I suggest you replace all arrays
// with lists
List<Person> allPersons = Arrays.asList(personGroup);

List<Person> men = new ArrayList<>();
for (Person p : allPersons) {
    if (p.getGender() == 'M') {
        men.add(p);
    }
}

流

Java 8 附带了 Streams API。 这可以使您的代码更加简洁。

List<Person> men = allPersons.stream() // Stream over all persons
    .filter(p -> p.getGender() == 'M') // Select only men
    .collect(Collectors.toList());     // Put them into a List