数组移除指定Flight对象方法返回false问题排查

阿华AIGC实验室

2026-5-15

问题分析与解决方案

首先，你的方法返回false最可能的原因是**Flight类没有正确重写equals()方法**，我们一步步拆解问题：

1. 核心问题：equals方法的默认行为

Java中，默认的Object.equals()方法是比较对象的引用地址，而不是对象的内容。也就是说，只有当你传入的Flight f和数组中的某个元素是同一个对象实例（引用完全相同）时，_flightsSchedule[i].equals(f)才会返回true。

如果你传入的是一个内容相同但引用不同的Flight实例（比如通过new创建的、属性和数组里的航班一致的对象），默认的equals会直接返回false，导致方法找不到匹配项，最终返回false。

解决方法：重写Flight类的equals()和hashCode()

你需要在Flight类中重写这两个方法，根据航班的唯一标识（比如航班号、出发地+目的地+时间组合等）来判断两个对象是否相等。示例代码如下：

import java.util.Objects;

public class Flight {
    private String flightNumber;
    // 其他属性，比如出发时间、目的地等...

    @Override
    public boolean equals(Object o) {
        // 先判断引用是否相同
        if (this == o) return true;
        // 判断是否为空或类型不匹配
        if (o == null || getClass() != o.getClass()) return false;
        Flight flight = (Flight) o;
        // 基于航班号（或其他唯一标识）判断相等
        return Objects.equals(flightNumber, flight.flightNumber);
    }

    @Override
    public int hashCode() {
        // 与equals保持一致，基于航班号生成哈希值
        return Objects.hash(flightNumber);
    }

    // 其他业务方法...
}

注意：按照Java规范，重写equals()时必须同时重写hashCode()，否则会导致HashMap等集合类的行为异常。

2. 当前removeFlight方法的潜在漏洞

即使equals方法没问题，你当前的移除逻辑也不够严谨：你只把当前位置的元素替换成i+1位置的元素，但后面的元素没有整体向前移位。比如数组是[F1, F2, F3]，移除F1后数组会变成[F2, F2, F3]，虽然_noOfFlights--后不会访问到第三个元素，但数组内留下了重复元素，可能引发后续问题。

修正后的removeFlight方法

public boolean removeFlight(Flight f) {
    for (int i = 0; i < _noOfFlights; i++) {
        if (_flightsSchedule[i].equals(f)) {
            // 将当前位置之后的所有元素向前移动一位
            System.arraycopy(_flightsSchedule, i + 1, _flightsSchedule, i, _noOfFlights - i - 1);
            // 把最后一个有效位置置为null，避免内存泄漏（可选但推荐）
            _flightsSchedule[_noOfFlights - 1] = null;
            _noOfFlights--;
            return true;
        }
    }
    return false;
}

这里用System.arraycopy()高效完成元素移位，比手动循环更简洁高效。