如何修复Java固定数组自定义删除方法无法正确删除元素的问题

原代码核心问题 #

• 元素匹配逻辑缺陷：直接使用==比较对象引用，未处理null值场景会触发空指针；若数组存在多个匹配元素，只会记录最后一个匹配下标
• 有效长度取值错误：将数组总容量list.length作为当前有效元素数量，固定数组场景下数组容量和实际存入元素数通常是分离维护的，直接取总长度会遍历到无意义的null元素
• 移位逻辑完全错误：原代码通过查找null值触发移位的逻辑不符合需求，嵌套循环会导致元素被错误覆盖，没有按照预期从删除下标开始向前移动后续元素

修复后代码 #

public boolean delete(E data) {
    int index = -1;
    // size为类中全局维护的当前数组有效元素数量，不要用list.length取值
    int effectiveSize = size;

    // 查找第一个匹配元素的下标，兼容null值比较
    for (int i = 0; i < effectiveSize; i++) {
        if (data == null) {
            if (list[i] == null) {
                index = i;
                break; // 如需删除所有匹配元素而非第一个，去掉该break后调整下方移位逻辑即可
            }
        } else if (data.equals(list[i])) {
            index = i;
            break; // 如需删除所有匹配元素而非第一个，去掉该break后调整下方移位逻辑即可
        }
    }

    if (index > -1) {
        // 从删除下标后一位开始，所有元素向前移动一位完成覆盖
        for (int i = index; i < effectiveSize - 1; i++) {
            list[i] = list[i + 1];
        }
        // 最后一位有效元素置空避免内存泄漏
        list[effectiveSize - 1] = null;
        // 更新全局维护的有效元素数量
        size = effectiveSize - 1;
        return true;
    }
    return false;
}

补充说明 #

如果你的需求是删除所有匹配元素而非第一个，删除查找循环中的break，同时调整移位逻辑为遍历过程中遇到匹配元素直接触发后续元素前移即可。

内容的提问来源于stack exchange，提问作者user15082428