Question

我不能为我的生活弄清楚为什么这不起作用。我不得不对文件中的单词列表进行频率检查，当读取它们时，我试图检查字符串数组中元素的当前单词，并确保它们在我之前不相等添加它。这是代码：

fin.open(finFile, fstream::in);

if(fin.is_open()) {
    int wordArrSize;
    while(!fin.eof()) {
        char buffer[49]; //Max number chars of any given word in the file
        wordArrSize = words.length();

        fin >> buffer;

        if(wordArrSize == 0) words.push_back(buffer);

        for(int i = 0; i < wordArrSize; i++) { //Check the read-in word against the array
            if(strcmp(words.at(i), buffer) != 0) { //If not equal, add to array
                words.push_back(buffer);
                break;
            }
        }



        totNumWords++; //Keeps track of the total number of words in the file
    }
    fin.close();

这是一个学校项目。我们不允许使用任何容器类，所以我构建了一个结构来处理扩展char **数组，推回和弹出元素等。

Answer 1

我认为你的代码words.push_back(buffer);应该在for循环之外。放一个标志来检查你是否在for循环中的数组中找到缓冲区并根据flag将它添加到for循环外的数组

Answer 2

for(int i = 0; i < wordArrSize; i++) { //this part is just fine
    if(strcmp(words.at(i), buffer) != 0) { //here lies the problem
         words.push_back(buffer);
         break;
    }
}

每次当前单词与数组中的if单词不匹配时，您都会输入i语句。因此，大多数情况下，当您进入循环时，它将是第一次迭代。这意味着在循环开始时（在字符串列表中与缓冲区不匹配的第一个单词），您将缓冲区添加到字符串列表中并打破循环。

你应该做的是完成检查整个words数组，然后才将缓冲区添加到数组中。所以你应该有这样的东西：

bool bufferIsInTheArray = false;//assume that the buffered word is not in the array.
for(int i = 0; i < wordArrSize; i++) { 
    if(strcmp(words.at(i), buffer) == 0) {
         //if we found a MATCH, we set the flag to true
         //and break the cycle (because since we found a match already
         //there is no point to continue checking)
         bufferIsInTheArray = true;
         break;
    }
//if the flag is false here, that means we did not find a match in the array, and 
//should add the buffer to it.
if( bufferIsInTheArray == false )
    words.push_back(buffer);
}

检查从文件读入的单词对阵列中的字符串

2 个答案: