如果阻塞,总会执行一些逻辑错误？

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块并提供密码更改,即使键入了错误的当前密码我想知道是否存在逻辑错误？

我正在开发一个8051项目,我想建立一个秘密密码更改,即如果我忘记了密码,我输入密码为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");
}