java-修复创建新对象的“add”方法中的bug

Question

public void addFlight(String planeID, String departure, String arrival) {

    Flight newFlight = null;
    for(Plane p : this.planes) {
        if(p.getID().equals(planeID)) {
            newFlight = new Flight(p, departure, arrival);
        }
    }
    flights.put(planeID, newFlight);
}

由于输入的planeID字符串确定了分配给新航班的飞机，因此我在planes数组列表中运行了一个循环，以检查名称是否匹配。我很确定这是什么使我不能做多个“航班”有相同的“飞机”，但我不知道我还能改变什么。我尝试添加这个附加的if语句，但它并没有像我想的那样起作用。

else if (this.flights.keySet().contains(p.getID())) {
    newFlight = new Flight(p, departure, arrival); //In case it is a repeated Plane
} 

Answer 1

为飞机创建HashMap并为航班创建ArrayList。

您的设计将得到更多的简化，您将得到预期的行为。

添加代码将成为

public void addFlight(String planeID, String departure, String arrival) {
    flights.add(planes.get(planeID), departure, arrival);
}

• 您正在使用平面ID作为散列映射的键。所以无论哪一个是唯一的w.r.t。键，应该放在hashmap中。在本例中为平面。
• 对于存储航班，有两种选择
  1. 聚合:hashmap 按照章鱼建议
  2. 关联:将 与plane列表为flight类的成员。

两种选择都很容易使用，但在第一选择中，飞行类中的飞机参考是多余的。所以我会选择第二个选择。

你应该选一个最适合你的。