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;
}

到目前为止，这是我的代码，我的所有字符都应该适合数组，并且循环看起来在逻辑上是合理的。我不明白为什么我的程序编译但运行不正常。我猜它可能与2维数组和我选择用于strcmp和strcpy的语法有关。但我尝试用单词[h] [0]代替单词[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;
    }
}

此代码段

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

必须替换

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

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

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

C ++ strtok和2d数组。程序编译但崩溃

1 个答案: