尝试使用 strtok 时调用构造函数

Question

我编写了一个名为 DSString 的自定义字符串类，其中包含一个私有成员 char* 数据。我正在尝试解析 csv 文件中的行，将文件行中找到的每个单词添加到 DSString 对象的 std::set 中。我调用 getline 将文件中的一行读入 char* 缓冲区，然后调用 strtok 来获取单词。但是，下面的 while 循环不起作用；我的调试器告诉我，在到达那行代码时，它会在 DSString.h 中使用参数 NULL 调用我的构造函数，然后程序结束。

int main (int argc, char ** argv) {
    
    ifstream input;
    input.open(argv1);

    int reviewLinesCtr = 0;
    char line[16000];

    set <DSString>positiveWordSet;

    while (input.getline(line,16000)) {
        if (reviewLinesCtr == 10) break;

            //Push review to positive set
            DSString word = strtok(line, " ");
            while (word != nullptr) {         //Program ends abruptly here
                positiveWordSet.insert(word);
                word = strtok(nullptr, " ");
            }
        }
        reviewLinesCtr++;
    }

    return 0;
}

DSString 只有私有成员 char *data 和 int size。 这是DSString中的相关代码：

    DSString::DSString(const char* param) {  //This is the constructor being  passed, param = NULL, which crashes the program.
        data = new char[strlen(param) + 1];
        this->size = strlen(param) + 1;
        strcpy(data, param);
    }

    DSString& DSString::operator= (const char* source)  {
        if (data != nullptr) {
            delete[] data;
        }
        data = new char[strlen(source + 1)];
        size = strlen(source) + 1;
        strcpy(data, source);
        return *this;
    }

    bool DSString::operator!= (const DSString& rhs) const {
        if (strcmp(this->data, rhs.data) != 0) {
            return true;
        }
        else {
            return false;
        }
    } 

这是大学项目的一部分，因此我们需要实现自定义 DSString 类。否则我不会使用它，而是使用 std::string。除了从文件中读取的缓冲区外，我们也不能在程序中的任何地方使用 char*。当我尝试开始执行此 while 循环时，为什么会调用此构造函数并将 NULL 作为参数？我该如何解决？

如果这是一个重复的问题，我深表歉意，我已经在互联网上搜索了一段时间，但没有找到任何有助于解释这里发生的事情的信息。 任何帮助将不胜感激。

Answer 1

DSString word = ....
while (word != nullptr) {         //Program ends abruptly here

在这里，您将DSString 对象与nullptr 进行比较。再来看看算子重载：

bool DSString::operator!= (const DSString& rhs) const {

它接受DSString 引用，而不是指针。但是，您的类可以从指针隐式转换，因此可以从 astd::nullptr_t 间接转换，因为它有一个转换构造函数：

SString::DSString(const char* param) {  //This is the constructor being  passed, param = NULL, which crashes the program.

由于构造函数调用 strlen(param) 和 strcpy(data, param);，将 null 传递给该构造函数具有未定义的行为。

为避免使用隐式转换意外调用此构造函数，您应将其声明为explicit。

将 null 传递给构造函数在这里也是一个问题：DSString word = strtok(line, " ");。

要修复逻辑，您应该使用带有strtok 的指针，并且仅在您检查它是否为非空后才创建您的字符串：

        char* word = strtok(line, " ");

---

我们也不能在程序中的任何地方使用 char*，除了从文件中读取的缓冲区。

坦率地说，使用strtok 但不使用char* 只是愚蠢的要求。

好像你已经违反了这个要求：

DSString 只有私有成员 char *data 和 ...

无论如何，如果您需要遵循这样的限制，您需要更改构造函数以在将 null 传递给它时正确运行。具体来说，在这种情况下，它可能不会将该指针传递给具有未定义行为的函数。

而且，如果将成员设置为 null，那么在将成员传递给上述这些函数之前，还必须检查该成员是否为 null。

Answer 2

eeorika 的回答中很可能概述了崩溃的原因。没有必要在这里重复。

为什么要调用构造函数：

DSString word = strtok(line, " ");

是Copy Initialization，所以调用的是构造函数，而不是赋值运算符。

旁注：

还有另一个致命错误。由于strtok returns nullptr在没有token的时候，DSString::DSString(const char* param)在没有token的时候会被调用

data = new char[strlen(param) + 1];

将在空指针上调用strlen。 Kabooom.

解决方案：在const char * 变量中捕获strtok 的返回值，并对其进行nullptr 测试。只有在有令牌的情况下才进行构建。

由于赋值要求是不能使用char * 变量，请确认是否可以使用const char *。如果不是，那是一个有趣（而且非常愚蠢）的泡菜。

赋值运算符泄漏内存。

DSString& DSString::operator= (const char* source)  {
    delete[] data; // must release the object's current string
                   // before allocating a new one
    data = new char[strlen(source + 1)]; 
    size = strlen(source) + 1;
    strcpy(data, source);
    return *this;
}