0

我已经编写了这个 minishell，但我不确定我是否正确控制了错误。我知道 fgets 可以返回 feof 和 ferror ( http://www.manpagez.com/man/3/fgets/ ) 但我不知道如何使用它们。

我检查了 fgets 是否返回一个空指针（这表明缓冲区的内容是不确定的），但我想知道如何使用 feof 和 ferror。

    #include <stdio.h>
    #include <stdlib.h> 
    #include <string.h> 
    #include <stdbool.h>    
    #define LINE_LEN  50
    #define MAX_PARTS  50 
    int main ()
    {
    char* token;
    char str[LINE_LEN];
    char* arr[MAX_PARTS];
    int i,j;
    bool go_on = true;

    while (go_on == true){
        printf("Write a line:('quit' to end) \n $:");
        fgets(str, LINE_LEN, stdin);

        if (str==NULL) {
            goto errorfgets;
        } else {
            size_t l=strlen(str);
            if(l && str[l-1]=='\n')
                str[l-1]=0;

            i=0;
            /* split string into words*/
            token = strtok(str, " \t\r\n");
            while( token != NULL ) 
            {
                arr[i] = token;
                i++;
                token = strtok(NULL," \t\r\n");
            }

            fflush(stdin);

            /* check if the first word is quit*/
            if (strcmp(arr[0],"quit")==0)
            {
                printf("Goodbye\n");
                go_on = false;
            } else {

                for (j=0; j < i; j++){
                printf("'%s'\n", arr[j]);       
                }   
            }
        }
    }

    return 0;
    errorfgets:
        printf("fgets didn't work correctly");
        return -1;
}
4

3 回答 3

3 
fgets(str, LINE_LEN, stdin);

if (str==NULL) {
    goto errorfgets;
}

这不是你检查返回值的方式fgets。更重要的是，在您的代码str中永远不会被NULL定义。你想要这样的东西：

if (!fgets(....)) }
    /* error handling. */
}
于 2014-02-12T09:44:46.190 回答
2

你可以像这样使用 feof。

#open a file
fd = fopen (testFile,"r+b");

#read some data from file 
fread (&buff, 1, 1, fd);
..
..
..
#To check if you are at the end of file
if (feof (fd))
{
    printf("This is end of file");
}else{
    printf("File doesn't end. Do continue...");
}
于 2014-02-12T10:36:11.053 回答
2

首先，你的测试：

fgets(str, LINE_LEN, stdin);

[...]

if (str==NULL) {
    goto errorfgets;
}

是错的。参数是按值传递的str，不能被修改fgets()。相反，您应该检查fgets()（返回NULLEOF 或错误）返回的值。

关于您的具体问题：fgets()不“返回”feof或ferror. 两者feof()和ferror()实际上都是函数（参见手册页）。您可以按如下方式使用它：

if (!fgets(str, LINE_LEN, stdin)) {
    /* fgets returns NULL on EOF and error; let's see what happened */
    if (ferror(stdin)) {
        /* handle error */
    } else {
        /* handle EOF */
    }
}
于 2014-02-14T08:52:00.087 回答