c - 某些逻辑错误总是执行 if block ？

Question

已关闭。这个问题是 not reproducible or was caused by typos 。目前不接受答案。

---

这个问题是由拼写错误或无法再重现的问题引起的。虽然类似的问题可能是 on-topic在这里，这个问题的解决方式不太可能帮助 future 的读者。

已关闭 6 年前。

Improve this question

#include<stdio.h>
#include<string.h>
char Sys_Pass[]="3699";
char verify_Pass(char *P,char *Q)
{
    char ch;
    ch=strcmp(P,Q);
    printf("String Cmp %s,%s is %c %d\n",P,Q,ch,ch);
    if(ch==0)
        return 1;
    else
        return 0;
}
void main()
{
    char Pass[10],New[10];
    char fp=0,sp=0;
    printf("Enter Password : ");
    scanf("%s",Pass);
    if(fp=verify_Pass(Pass,Sys_Pass))
    {
Change:     printf("Enter a New Password : ");
        scanf("%s",Pass);
        printf("Re-Type New Password : ");
        scanf("%s",New);
        if(sp=verify_Pass(Pass,New))
        {
            strcpy(Sys_Pass,New);
            printf("Password Successfully changed\n");
            printf("New Password : %s\n",Sys_Pass);

        }
        else
            printf("Passwords Mismatch\n");
    }
    else
    {
        if(strcmp(Pass,"111999")==0);
            goto Change;
        printf("Wrong Password !!!\n");
    }
    printf("Fp : %c %d\nSp : %c %d\n",fp,fp,sp,sp);
}

此代码始终执行 if block 并提供密码更改，即使输入了错误的当前密码我想知道是否存在逻辑错误(如果有)？

我正在开发一个 8051 项目，我想建立一个 secret 密码更改，即如果我忘记了密码并且我输入密码为 111999，它应该引导我进入密码更改菜单，但这里它总是指示更改密码菜单。这实际上是嵌入的 C 代码，但我尝试使用 C 代码来纠正它，但在两种情况下都会产生相同的输出。

Answer 1

你有一个杂散的分号:

         // here ---------------v
    if(strcmp(Pass,"111999")==0);
        goto Change;

顺便说一句，这不是 goto 的好用。更好的做法是在 if 条件中检查当前密码或 111999。

printf("Enter Password : ");
scanf("%s",Pass);
if ((strcmp(Pass,"111999")==0) || (strcmp(Pass,Sys_Pass)==0))
{
    printf("Enter a New Password : ");
    ...
}
else
{
    printf("Wrong Password !!!\n");
}