C++ strtok 和二维数组。程序编译但崩溃

Question

#include <iostream>
#using namespace std; 
#include <cstring>
#include <fstream>

int main(int argc, char *argv[] )
{
ifstream file;
// check for valid user input
if (argc != 2) 
{
    cout << "Please enter the file name." << endl;
    return 0;
}
else
{
    file.open (argv[1]);
    // check if file is valid
    if (file == 0)
    {
        cout << "File is not valid." << endl;
        return 0;
    }
}

char words[100][16]; // 2d array for unique words
char line[100]; // c-string to hold a line
char *token = NULL;
int h=0; // variable for counting array index 
int i, j; // counter variables

while (file.getline (line, 100, '\n'))
{
    bool unique = true; // boolian to test for unique words
    cout << line << endl; // echo print the line
    token = strtok(line, " ."); // 1st token
    if (token == NULL) // check if 1st token exists
    {
        break;
    }
    // loop to check if token is unique 
    for (i = 0; i < 100; i++)
    {
        if (strcmp(token, words[i]) == 0)
        {
            unique = false;
            break;
        }
    }
    if (unique == true)
    {
        strcpy(words[h], token);
        h++;
    }
    unique = false; 
    // another loop to continue strtok and check for unique words
    while (token != NULL)
    {
        token = strtok(NULL, " .");
        for (i = 0; i < 100; i++)
        {
            if (strcmp(token, words[i]) == 0) 
            {
                unique = false;
                break;
            }
        }
        if (unique == true)
        {
            strcpy(words[h], token);
            h++;
        }
    }
}



return 0;
}

到目前为止，这是我的代码，我的所有字符都应该适合数组，并且循环看起来在逻辑上是合理的。我不明白为什么我的程序可以编译但不能正确运行。我猜测它可能与二维数组以及我选择用于 strcmp 和 strcpy 的语法有关。但是我尝试用 words[h][0] 代替 words[h] ，但这也不起作用。我有点不知所措，请帮忙！

Answer 1

首先你必须对数组进行零初始化

char words[100][16] = {};

这个循环

for (i = 0; i < 100; i++)
{
    if (strcmp(token, words[i]) == 0)
    {
        unique = false;
        break;
    }
}

应该改为

for (i = 0; i < h; i++)
{
    if (strcmp(token, words[i]) == 0)
    {
        unique = false;
        break;
    }
}

这段代码sn-p

while (token != NULL)
{
    token = strtok(NULL, " .");
    // ...

必须替换

while ( ( token = strtok(NULL, " .") ) != NULL)
{
    // ...

我认为这是程序崩溃的主要原因。

您也不会检查文件中的唯一词是否比数组的大小更多，以及令牌的大小是否大于数组的第二个大小。