C++常量表达式报错及类析构函数正确使用问题求助

1. "this cannot be used in a constant expression"报错修复 #

报错的根本原因是标准C++不支持栈数组使用运行时可变的长度，你写的int res[mSize];中mSize是类的成员变量，只有运行时才能确定具体值，不满足栈数组要求编译期常量的规则。
另外你可以注意到，res数组在整个FindMissing函数里完全没有被使用，直接删除这一行代码就可以解决这个编译报错。
如果你后续确实需要使用这个可变长度的数组，有两种替代方案：

• 动态分配堆内存：int* res = new int[mSize];，使用结束后记得调用delete[] res;释放内存
• 使用标准库容器：引入#include <vector>后声明std::vector<int> res(mSize);，自动管理内存无需手动释放

2. 析构函数正确性检查 #

你当前的~MyContainer() {}空析构是符合你现有代码逻辑的：你的构造函数直接绑定了外部传入的栈数组指针，没有在类内申请堆内存，所以不需要在析构里做内存释放操作。
如果后续你改造容器让它自己管理内部数组内存（比如在构造函数里用new分配数组空间），那需要修改析构函数为：

~MyContainer() {
    delete[] mHead;
}

同时还要补充实现拷贝构造函数、拷贝赋值运算符，避免浅拷贝导致的double free问题。

3. 其他潜在严重问题修复 #

你的代码还有多个会导致运行时崩溃、结果异常的隐藏问题：

• Delete方法未处理找不到目标元素的情况：index变量没有初始化，要是循环里没找到匹配的元素，后续访问*(mHead + index)会触发非法内存访问。可以修改为：

  void MyContainer::Delete(int a)
  {
      int index = -1; // 先初始化index为无效值
      for (int i = 0; i < mSize; i++)
      {
          if (*(mHead + i) == a)
          {
              index = i;
              break;
          }
      }
      if (index == -1) return; // 没找到直接返回
      for (int i = index; i < mSize - 1; i++) // 注意边界是mSize-1，避免访问数组外的内存
      {
          *(mHead + i) = *(mHead + i + 1);
      }
      mSize--;
  }
  

• Add方法存在数组越界：你传入的外部栈数组长度只有5，多次调用Add会往数组后面的栈空间写数据，触发栈溢出，属于未定义行为。如果需要支持动态添加元素，建议改成容器内部用vector存储数据，或者自己实现动态扩容逻辑。
• FindMissing方法未处理无缺失的情况：如果1到mSize的整数都存在，temp变量没有初始化，返回值会是随机垃圾值，可以给temp初始化为mSize + 1作为默认返回值。

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