动态数组删除用户指定元素时出现垃圾值问题求助

咱们来拆解你遇到的垃圾值问题，主要有两个核心原因，还有几个细节需要调整：

1. 新数组赋值时索引错误（最直接的垃圾值来源） #

在remove函数里，你用原数组的索引i直接给新数组change赋值，这会导致原数组中被设为0的位置对应的新数组位置完全没被赋值，这些位置的内存是未初始化的，自然就是垃圾值。

举个例子：原数组是[1, 2, 0, 3]（假设2是要删除的元素，被设为0），你循环到i=0时赋值change[0]=1，i=1时跳过，i=2时跳过，i=3时赋值change[3]=3，那change[1]和change[2]就都是垃圾值！

修复方法： #

给新数组单独用一个计数器来追踪赋值的位置，代码修改如下：

int* remove(int* a,int search){
    final = 0; // 先重置final，避免全局变量残留旧值
    for (int i = 0; i < size; i++){
        if (a[i] == search)
            a[i] = 0;
        else final++;
    }
    int* change = new int[final]; // 不需要额外+1，final就是有效元素的数量
    int j = 0; // 新数组的专属索引计数器
    for (int i = 0; i < size; i++){
        if (a[i] != 0){ // 别用>0，原数组如果有负数就会误删
            change[j++] = a[i];
        }
    }
    delete[] a;
    a = nullptr;
    return change;
}

2. read函数中while(!a.eof())导致的多计数问题 #

用while(!a.eof())读取文件会让size多算1：当读到最后一个元素后，a>>temp会失败，但size++还是会执行一次，导致你创建的数组比实际元素多一个位置，这个位置的元素是未初始化的垃圾值，会被带到后续操作中。

修复方法： #

改成直接在循环里判断读取是否成功，同时用seekg重置文件指针，不用反复关闭打开：

int* read(ifstream& a){
    int temp;
    size = 0; // 重置全局size，避免重复调用出错
    a.open("data(1).txt");
    while (a >> temp){ // 只有读取成功时才计数
        size++;
    }
    a.clear(); // 清除文件的eof状态标记
    a.seekg(0); // 把文件指针移回开头
    int* arr = new int[size];
    for (int i = 0; i < size; i++)
        a >> arr[i];
    a.close();
    return arr;
}

3. 其他细节优化 #

• 全局变量风险：size和final作为全局变量容易残留旧值，至少要在read和remove开头重置它们；如果可以的话，建议改成函数返回值或参数传递，更安全。
• 内存泄漏修复：main函数最后要记得释放ptr指向的内存，避免内存泄漏。

修复后的完整代码 #

#include<iostream>
#include<fstream>
using namespace std;
int size = 0;
int final = 0;

int* read(ifstream& a){
    int temp;
    size = 0;
    a.open("data(1).txt");
    while (a >> temp){
        size++;
    }
    a.clear();
    a.seekg(0);
    int* arr = new int[size];
    for (int i = 0; i < size; i++)
        a >> arr[i];
    a.close();
    return arr;
}

int* remove(int* a,int search){
    final = 0;
    for (int i = 0; i < size; i++){
        if (a[i] == search)
            a[i] = 0;
        else final++;
    }
    int* change = new int[final];
    int j = 0;
    for (int i = 0; i < size; i++){
        if (a[i] != 0){
            change[j++] = a[i];
        }
    }
    delete[] a;
    a = nullptr;
    return change;
}

int main(){
    int* ptr = nullptr;
    int num;
    cout << "please enter the number to remove: ";
    cin >> num;
    ifstream in;
    ptr=read(in);
    ptr=remove(ptr, num);
    for (int i = 0; i < final; i++)
        cout << ptr[i] << " ";
    cout<<endl;
    // 释放最后分配的内存
    delete[] ptr;
    ptr = nullptr;
    system("pause");
    return 0;
}

这样修改后，垃圾值的问题就彻底解决了，同时也优化了文件读取逻辑和内存管理。

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