c 文件输入函数无法按预期工作

Question

我有这个功能应该检查用户名是否被列入黑名单，如果没有，它应该通过检查该用户名是否存在于第二个文件中来“登录”用户。如果是，那么文件中的那一行将被记住，如果记住的行上的密码正确，函数应该检查第三个文件，所以基本上用户名的行就是他的密码的行，如果这两个匹配有了输入并且用户没有被列入黑名单，那么它应该返回一个肯定的答案。

不幸的是，无论我输入什么，我的函数似乎都会返回一个肯定的答案。任何人都可以指出我的错误是什么，以便它按预期工作吗？

编辑了无意的错误，我仍然有问题，如果文件不在 blacklist.txt 中，我无法停止阅读

代码:

#include <stdlib.h>
#include <stdio.h>
#include <errno.h>
#include <string.h>
#include <sys/types.h>
#include <netinet/in.h>
#include <sys/wait.h>
#include <sys/socket.h>
#include <signal.h>
#include <ctype.h>          
#include <arpa/inet.h>
#include <netdb.h>   

int login(char username[20],char pw[20])
{
    FILE *fp = fopen("blacklist.txt","r");          
    int ok = 0, pw_line=0,oku=0,savepwline=0,count=0,okp=0;

    while (ok==0)                                            
    {
        if (fp != NULL)
        {
            char line[20];
            while (fgets(line,sizeof(line), fp) != NULL)
            {                           
                size_t i=strlen(line)-1;
                if(line[i]=='\n')
                line[i]='\0';
                if (strcmp(username,line) == 0 ) ok=1;
            }
        }
    }
    fclose(fp);
    if (ok==1)
    {
        printf("user blacklisted\n");
    }
    else
    {
        FILE *fp2 = fopen("loginuser.txt","r"); 
        while (oku==0)                                            
        {
        if (fp2 != NULL)
        {
            char line3[20];
            while (fgets(line3,sizeof(line3), fp2) != NULL)
            {
                pw_line++;                            
                size_t i=strlen(line3)-1;
                if(line3[i]=='\n')
                line[i]='\0';
                if (strcmp(username,line3) == 0 ) { oku=1; savepwline=pw_line; }
            }
        }
        }
        FILE *fp3 = fopen("loginpw.txt","r"); 
        if (oku==1)
        if (fp3 != NULL)
        {
            char line2[20];
            while (fgets(line2,sizeof(line2), fp3) != NULL)
            {
                if ( count == savepwline )           
                {
                    size_t i=strlen(line2)-1;
                    if(line2[i]=='\n')
                        line2[i]='\0';
                    if (strcmp(pw,line2) == 0 ) okp=1;
                }
                else count++;
            }
        }
    fclose(fp2);
    fclose(fp3);
    }
    if(oku&&okp) return 1;
    else return 0;
}

int main()
{
 char a[20],b[20];
 scanf("%s",a);
 scanf("%s",b);
 if(login(a,b)) printf("yes");
else printf("no");
}

示例文件(“loginuser.txt”/“loginpw.txt”):

hihi
aloss
foif
distsd

黑名单.txt:

carl
gigc
ffgfd
gdfgdd
rreti

Answer 1

我看不出代码有什么问题。

• 如果用户名不在 blacklist.txt 中，它永远不会退出 while 循环。而是读到文件末尾。
• 您使用“=”代替“==”进行比较，这将进行赋值而不是比较。
• 在打开文件后立即检查文件指针的有效性，这样你就不需要在循环中每次都验证它。这是一个性能问题。
• 编写一个文件读取函数。这将使调试更容易。
• 如果您不会使用调试器，请添加一些 printf 语句，这将有助于您理解代码流。

请参阅下面更正后的代码。我没有添加密码比较检查。你可以自己做。

// Returns 1 if found. Returns 0 on not found/error
int isEntryFound(char *fileName, char *inputStr)
{
    FILE *fp = fopen(fileName,"r");
    if (fp == NULL)
    {
        // Couldnt verify. Mark as error
        return 0;
    }
    int ok = 0;

    char line[20];
    while (fgets(line,sizeof(line), fp) != NULL)
    {
        size_t i=strlen(line)-1;
        if(line[i]=='\n')
            line[i]='\0';
        if (strcmp(inputStr,line) == 0 ) ok=1;
    }
    fclose(fp);
    if(ok == 1)
        return 1;
    return 0;

}

int login(char *username,char pw[20])
{
    int blackListed = isEntryFound("blacklist.txt", username);

    if(blackListed)
    {
        printf("Blacklisted\n");
        return 0;
    }

    int userFound = isEntryFound("loginuser.txt", username);

    if(!userFound)
    {
        printf("User not Found\n");
        return 0;
    }

    // TODO: Read and compare the passwords the same way
    return 0;
}