在我的模板类示例中，即使我没有定义添加方法，它也会添加“分段错误（核心转储）”错误

Posted 2023-02-22

【中文标题】在我的模板类示例中，即使我没有定义添加方法，它也会添加“分段错误（核心转储）”错误

【英文标题】：In my template class example, Even if I dont define add method, it addstake "segmentation fault (core dumped)" error

【发布时间】：2017-05-16 14:21:17

【问题描述】：

！！注意：我在回答后多次编辑问题。但是下面的问题是第一个问题，第一个答案是一个有用的答案。请不要被某些cmets混淆。它们是在我多次更改问题后写的。

---

我有一个 Shop 模板类和 Cookie 类。 Shop 是一个保存名为 cookieShop 的 cookie 的列表。 Shop cotr 可以接受一个 cookie 作为参数，可以通过 Shop 模板类的 Add 方法添加更多。

我正在创建两个 cookie。一是通过 shop cotr 添加，二是通过 add 方法。即使我不在 add 方法中编写代码，第二个 cookie 也会添加到购物清单中。我试图理解它为什么会这样做，但无法理解。

这是我的代码：

//Shop.h

#ifndef SHOP_T
#define SHOP_T
#include <string>
using namespace std;

template<class type>
class Shop;

template<typename type>
ostream& operator<<(ostream& out, const Shop<type>& S)
    for(int i = 0; i < S.size; i++)
        out << i + 1 << ".\t" << S.list[i] << endl;


template<class type>
class Shop
    type *list;
    string name;
    int size;
public:
    Shop()  list = 0; size = 0; 
    Shop(type t);
    ~Shop() delete[] list; 
    void Add(type A);
    friend ostream& operator<< <>(ostream& out, const Shop<type>& S);
;

template<class type>
Shop<type>::Shop(type t) : size(0)
    list = new type;
    list = &t;
    size++;


template<class type>
void Shop<type>::Add(type A)
//  type *temp = new type[size+1];
//  for(int i = 0; i < size; i++)
//      temp[i] = list[i];
//  delete[] list;
//  temp[size] = A;
//  list = temp;
    size++;


#endif

//Cookie.h

#ifndef COOKIE
#define COOKIE
#include <string>
using namespace std;

class Cookie
    string name;
    int piece;
    float price;
public:
    Cookie();
    Cookie(string n, int pi, float pr);
    friend ostream& operator<<(ostream& out, const Cookie& C);
;

#endif

//Cookie.cpp

#include "Cookie.h"
#include <iostream>
#include <string>
using namespace std;

Cookie::Cookie() 
Cookie::Cookie(string n, int pi, float pr)
      name = n;
      piece = pi;
      price = pr;


ostream& operator<<(ostream& o, const Cookie& C)
    o << C.name << "\t" << C.piece << "\t" << C.price;

//main.cpp

#include <iostream>
#include <string>
#include "Shop.h"
#include "Cookie.h"

using namespace std;
int main()
    Cookie cookie1("Chocolate Cookie", 50, 180);
    Cookie cookie2("Cake Mix Cookie", 60, 200);

    Shop<Cookie> cookieShop(cookie1);
    cookieShop.Add(cookie2);

    cout << cookieShop <<endl;


    return 0;

如您所见，Add 方法中的代码已被注释。它如何添加第二个 cookie？

当编译 (gcc main.cpp Cookie.cpp) 并运行它 (./a.out) 时，它会给出以下几行：

巧克力曲奇 50 180 蛋糕混合饼干 60 200 分段错误（核心转储） 为什么我会收到分段错误错误？

注意：我是 *** 的新手。如果我有错误的行为。请告诉:)

【问题讨论】：

您忘记在Add 中注释掉size++;。

@NathanOliver。因为我想打印列表[1]。我认为问题不在于。是吗？

谁教你原始数组是一个有用的容器是大错特错。在这里使用std::vector 你会好多了

这是一个家庭作业。我必须这样做。截止日期已经结束。不幸的是，我按上述方式发送了作业。不过现在想学。

我为您学习的愿望鼓掌，但是 1）请不要以完全改变它们的方式编辑您的问题，以及 2）当您编写代码时，最好从小而简单的东西开始可以完美运行，当您在此处发布代码时，最好将其缩减为产生错误的最小、最简单的代码。

【参考方案1】：

这里：

template<class type>
Shop<type>::Shop(type t) : size(0)
  list = new type;
  list = &t;
  size++;

变量t 是这个函数的局部变量。您将list 设为指向该变量的指针。当函数终止时，变量超出范围，Shop 被留下一个悬空指针，你会得到未定义的行为。

【讨论】：

哦。我在这里做了非常糟糕的代码。我会尝试纠正它。谢谢。

还有list = new type、list = &t与delete [] list不匹配造成的UB

@Caleth 为什么我更改了您提到的代码后仍然出现分段错误错误？修改后的代码我也错了吗？

@Murat：现在你调用了你的Add 函数，但它仍然大部分被注释掉了；它不会添加 Cookie，只会增加 size。然后，当您尝试打印第二个 Cookie 时，您读取了数组末尾（这不是真正的数组，Cookie 不是Cookie[1]），并再次获得未定义的行为。

@Beta：我已经通过在 main.cpp 中注释部分：cookieShop.Add(cookie2); 来尝试代码。抱歉，我忘记在这里更改了。但我得到与上面相同的结果。