Question

我有一个名为“Vertex.hpp”的类，如下所示：

 #include <iostream>
 #include "Edge.hpp"
 #include <vector>
   using namespace std;

/** A class, instances of which are nodes in an HCTree.
 */
class Vertex {
public:
Vertex(char * str){
 *name=*str;
 }

vector<Vertex*> adjecency_list;
vector<Edge*> edge_weights;
char *name;

 };

#endif

当我按如下方式实例化Vector类型的对象时：

 Vertex *first_read;
 Vertex *second_read;

  in.getline(input,256);

  str=strtok(input," ");
  first_read->name=str;


  str=strtok(NULL, " ");
  second_read->name=str;

当实例化多个类型为Vector的对象时，会发生分段错误。如果实例化多于一个对象，为什么会发生这种情况？如何允许实例化多个对象？

Answer 1

*name=*str;

在您首先指向某个指针之前，不能取消引用指针。

你可能意味着：

Vertex(char * str) {
    name=strdup(str);
}

但你真的应该使用std::string。

Answer 2

我认为你复制字符串的方式是错误的。

*name=*str;

name ans str都是char *类型。您正在取消引用这些指针。这意味着您可以查看它们指向的内存位置并将其解释为char。

当你第一次调用某个位置str指向的位置时，它的第一个字符被复制到随机地址（因为你从未初始化name）。

第二次你没那么幸运。调用NULL的strtok返回NULL strtok at cplusplus

现在你尝试使用空指针指向的记忆，这很糟糕。

您需要为name分配内存并使用正确的复制功能。

name = new char[SomeMaxLenght];
strcpy(name, str);

Answer 3

这是一种非常简单的C方式，在现代C ++中非常不推荐。请记住，C ++应该被视为不同的语言，而不是C的严格超集。

首先，您应该通过查看that list来获得一本好书，因为您似乎缺少许多基础知识。

至于你的问题，主要的问题是name未初始化，所以你遇到了所谓的未定义的行为（即任何事情都可能发生;在你的情况下它崩溃了第二个实例）。我可以通过动态分配内存来深入研究如何修复它，但为什么要这么麻烦？只需使用std::string：

class Vertex {
    std::string name; // string instead of char *
public:
    Vertex(const std::string &str) { // pass by const reference
        name = str; // I should really use an initializer list there, but oh well
    }

    // the rest of the class is the same
};

看看这有多简单？现在你不必乱用指针，这些指针使用起来很痛苦。简而言之：使用标准库。并获得一本好书。真。

对多于1个对象的初始化进行分段错误

3 个答案: