使用 strcmp 和嵌套循环检查数组的值

Question

我已经完成了该计划的最后部分，但我再次面临一些问题。

<小时/>

问题#1:当我在标记化的 while 循环中打印数组的分隔字符串部分时，这些值是正常的。但是，当我打印 puts(tokenArray[1]) 时，它会导致仅显示命令的一个字母，例如:输入:“qwe rty”| while 循环中的打印输出:qwe rty |在循环外使用 puts(tokenArray[1]) 打印输出:“e”(是的，只是字母)。

<小时/>

问题#2:我在调试时注意到了这一点。当我输入随机输入然后输入“history”后，tokenArray的第一个位置填充“history”，下一个位置填充“tory”。为此，我无法使用嵌套循环和 strcmp，如下所示，为了检查第一部分是否是“历史记录”，如果是，请检查第二部分。如果第二部分为空，则仅显示命令的历史记录，或者如果为“1”/“2”等(当用户输入“history 1”时，则执行历史记录中的第一个命令。

这是我迄今为止的进展:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int main (int argc, char *argv[])
{
    int i=0; int j=0; int k=0;
    char inputString[100];
    char *result=NULL;
    //char *result2=NULL;
    char delims[] = " ";
    char historyArray[100][100] = {0};
    char historyKey[] = "history";
    char *tokenArray[100] = {0} ;
    //char exitString[] = "exit";

    do
    {
        printf("hshell>");
        gets(inputString);
        strcpy (historyArray[k], inputString);
        k++;

        // Break the string into parts
        result = strtok(inputString, delims);


        while (result!=NULL)
        {
            //result2 = result;
            tokenArray[j] = result;
            //puts(result);
            result= strtok(NULL, delims);
            //puts(tokenArray[j]);
            j++;
        }
        //j = 0;
        puts(tokenArray[1]);
        if (strcmp(tokenArray[0],historyKey) == 0)
        {
            if (strcmp(tokenArray[1], " " ) == 0)
            {
                for (i=0; i<k; i++)
                {
                    printf("%d. %s \n",i+1,historyArray[i]);
                }
            }
        }
        else if (strcmp ("exit",inputString) != 0)
        {
            printf("\nCommand not found \n");
        }

    } while (strcmp ("exit", inputString) != 0);
    return 0;
}

Answer 1

有几件事立即显而易见。

1. 您永远不会将 j 的值从一个命令行重置到下一个命令行。
2. strtok() 不会仅仅因为您输入“command”之类的内容而将单空格字符串添加到您的输入标记中。它会剥夺定界，所以你的逻辑在这个想法上是有缺陷的。 “command”的输入标记将是单个标记，“command one”的输入标记将是“command”和“one”。您显然认为它会根据您使用 strcmp(tokenArray[1], "") 的操作为您提供一个单空格字符串。
3. 主要问题，但与您眼前的问题无关，当 k 变得足够大(准确地说是 100 个命令)时，最终您将溢出堆栈
4. 感谢@unwind，您需要扔掉使用 fgets() 获取的换行符，我在测试时就是这样做的。 Prolly 应该提到这一点。

修复第一个，重新考虑第二个，并考虑使用替代结构来保存第三个历史。最后，我用 fgets(inputString, 100, stdin); 测试了它，它确实有效，所以不知道你这边出了什么问题。