[已解决]来个大佬吧，我的逻辑是不是有问题，怎么改啊？

爱吃菠萝包

while True:
    commend=input('请输入指令:')

    if commend=='1':
        name=input('请输入用户名：')
        if name in register:
            print('该用户名已存在')
        else:
            number=input('请输入密码：')
            result=hashlib.md5(number.encode('utf-8'))
            hashed_password=result.hexdigest()
            register[name]=hashed_password
            print('恭喜，注册成功~')

    elif commend=='2':
        while True:
            names=input('请输入用户名：')
            if names in register:
                numbers=input('请输入密码：')
                result=hashlib.md5(numbers.encode('utf-8'))
                hashed_password=result.hexdigest()
                while True:
                    if register[names]==hashed_password:
                        print('恭喜，登录成功~')
                        break                                    #来个大佬吧！这里我无法实现成功后退到最外层循环[[/b]/color]
                    else:
                        print('密码错误！')
                        numbers=input('请重新输入密码：')
                        result=hashlib.md5(numbers.encode('utf-8'))
                        hashed_password=result.hexdigest()
                        continue
            else:
                print('该用户名不存在。')
                register=input('请重新输入用户名：')
                continue

    elif commend=='3':
        break
            

最佳答案

月排行榜 / 总排行榜

player-none

前天 23:18

爱吃菠萝包 发表于 2025-5-7 22:20
我懂可以这样改，可是如果要加上我的这一串呢：numbers=input('请重新输入密码：')
我要实现的是输入密 ...

求最佳

其实你原来的代码应该可以实现break到最外层的
我稍微修改了下，现在逻辑稍微好点了

另有几点修改建议

• md5/sha1加密已被证实不安全（容易重复之类的问题）
• command而不是commend
• 非列表的变量名建议使用单数形式
• 合并print和它后面的input
• 不要加不加也可以的else
  

1. while True:
   
2.     commend=input('请输入指令:')
   
3.     if commend=='1':
   
4.         name=input('请输入用户名：')
   
5.         if name in register:
   
6.             print('该用户名已存在')
   
7.         else:
   
8.             number=input('请输入密码：')
   
9.             result=hashlib.md5(number.encode('utf-8'))
   
10.             hashed_password=result.hexdigest()
    
11.             register[name]=hashed_password
    
12.             print('恭喜，注册成功~')
    
13.     elif commend=='2':
    
14.         while True:
    
15.             names=input('请输入用户名：')
    
16.             if names in register:
    
17.                 numbers=input('请输入密码：')
    
18.                 result=hashlib.md5(numbers.encode('utf-8'))
    
19.                 hashed_password=result.hexdigest()
    
20.                 while True:
    
21.                     if register[names]==hashed_password:
    
22.                         print('恭喜，登录成功~')
    
23.                         break
    
24.                     print('密码错误！')
    
25.                     numbers=input('请重新输入密码：')
    
26.                     result=hashlib.md5(numbers.encode('utf-8'))
    
27.                     hashed_password=result.hexdigest()
    
28.                 break
    
29.             print('该用户名不存在。')
    
30.             names=input('请重新输入用户名：')
    
31.     elif commend=='3':
    
32.         break

复制代码

跳转到最佳答案楼层

爱吃菠萝包

FishC

问题分析：
1. register 应该是个字典保存用户名和密码，但当前被覆盖为字符串
2. 登录成功后缺少退出多层循环的机制
3. 用户名不存在时的错误处理有逻辑问题（错误地使用了register=input()）
4. 密码比对逻辑需要改进

修正后的代码：

1. 
   
2. import hashlib
   
3. 
   
4. register = {}  # 初始化字典保存用户数据
   
5. 
   
6. while True:
   
7.     cmd = input('请输入指令(1注册 2登录 3退出):')
   
8.    
   
9.     if cmd == '1':
   
10.         name = input('请输入用户名：')
    
11.         if name in register:
    
12.             print('该用户名已存在')
    
13.         else:
    
14.             pwd = input('请输入密码：')
    
15.             hashed_pwd = hashlib.md5(pwd.encode()).hexdigest()
    
16.             register[name] = hashed_pwd  # 正确保存键值对
    
17.             print('注册成功~')
    
18.    
    
19.     elif cmd == '2':
    
20.         login_success = False  # 登录状态标记
    
21.         
    
22.         while not login_success:
    
23.             name = input('用户名：')
    
24.             if name not in register:
    
25.                 print('用户不存在')
    
26.                 continue  # 继续用户名循环
    
27.                
    
28.             # 密码重试最多3次
    
29.             for _ in range(3):
    
30.                 pwd = input('密码：')
    
31.                 hashed_pwd = hashlib.md5(pwd.encode()).hexdigest()
    
32.                
    
33.                 if hashed_pwd == register[name]:
    
34.                     print('登录成功！')
    
35.                     login_success = True
    
36.                     break  # 退出密码循环
    
37.                 else:
    
38.                     print('密码错误')
    
39.             
    
40.             if login_success:
    
41.                 break  # 退出用户名循环
    
42.         
    
43.         # 登录成功后自动返回主循环
    
44.    
    
45.     elif cmd == '3':
    
46.         break
    

复制代码

主要改进点：

• 使用register = {}作为用户数据库（字典结构）
• 增加登录状态标记login_success控制循环层级
• 限制密码最多尝试3次（安全改进）
• 修复用户不存在时的错误处理逻辑
• 注册时正确保存用户名和密码的对应关系
  

这样修改后：
1. 注册成功时会保存用户名和对应的哈希密码
2. 登录成功后会自动退回到主指令循环
3. 密码错误超过3次会返回用户名输入界面
4. 输入3可以正常退出程序

以上回复来自 FishC AI 小助理，如未能正确解答您的问题，请继续追问。

jackz007

1. import hashlib
   
2. 
   
3. register = dict()
   
4. while True:
   
5.     command = '1' if not register else input('请输入指令:')
   
6.     if command == '1' :
   
7.         name = input('【注册】-> 请输入用户名：')
   
8.         if name in register . keys() :
   
9.             print('该用户名已存在')
   
10.         else:
    
11.             number = input('【登录】-> 请输入密码：')
    
12.             result = hashlib . md5(number.encode('utf-8'))
    
13.             hashed_password = result . hexdigest()
    
14.             register[name] = hashed_password
    
15.             print('恭喜，注册成功~\n')
    
16.     elif command == '2':
    
17.         f = True                                                        # 注意此处
    
18.         while f :                                                       # 注意此处
    
19.             names = input('【登录】-> 请输入用户名：')
    
20.             if names in register . keys() :
    
21.                 numbers=input('【登录】-> 请输入密码：')
    
22.                 result = hashlib . md5(numbers . encode('utf-8'))
    
23.                 hashed_password = result . hexdigest()
    
24.                 if register[names] == hashed_password:
    
25.                     print('恭喜，登录成功~\n')
    
26.                     break
    
27.                 else:
    
28.                     print('【错误】：* 密码错误 *\n')
    
29.                     f = False                                           # 注意此处
    
30.             else:
    
31.                 print('【错误】：* 用户不存在 *\n')
    
32. 
    
33.     elif command == '3' :
    
34.         break

复制代码

爱吃菠萝包

jackz007 发表于 2025-5-7 22:16

我懂可以这样改，可是如果要加上我的这一串呢：numbers=input('请重新输入密码：')
我要实现的是输入密码错误后的指令发生了改变

jackz007

爱吃菠萝包 发表于 2025-5-7 22:20
我懂可以这样改，可是如果要加上我的这一串呢：numbers=input('请重新输入密码：')
我要实现的是输入密 ...

        没看懂，

我要实现的是输入密码错误后的指令发生了改变

什么叫指令发生了改变，你举例说明一下呢。

player-none

爱吃菠萝包 发表于 2025-5-7 22:20
我懂可以这样改，可是如果要加上我的这一串呢：numbers=input('请重新输入密码：')
我要实现的是输入密 ...

求最佳

其实你原来的代码应该可以实现break到最外层的
我稍微修改了下，现在逻辑稍微好点了

另有几点修改建议

• md5/sha1加密已被证实不安全（容易重复之类的问题）
• command而不是commend
• 非列表的变量名建议使用单数形式
• 合并print和它后面的input
• 不要加不加也可以的else
  

1. while True:
   
2.     commend=input('请输入指令:')
   
3.     if commend=='1':
   
4.         name=input('请输入用户名：')
   
5.         if name in register:
   
6.             print('该用户名已存在')
   
7.         else:
   
8.             number=input('请输入密码：')
   
9.             result=hashlib.md5(number.encode('utf-8'))
   
10.             hashed_password=result.hexdigest()
    
11.             register[name]=hashed_password
    
12.             print('恭喜，注册成功~')
    
13.     elif commend=='2':
    
14.         while True:
    
15.             names=input('请输入用户名：')
    
16.             if names in register:
    
17.                 numbers=input('请输入密码：')
    
18.                 result=hashlib.md5(numbers.encode('utf-8'))
    
19.                 hashed_password=result.hexdigest()
    
20.                 while True:
    
21.                     if register[names]==hashed_password:
    
22.                         print('恭喜，登录成功~')
    
23.                         break
    
24.                     print('密码错误！')
    
25.                     numbers=input('请重新输入密码：')
    
26.                     result=hashlib.md5(numbers.encode('utf-8'))
    
27.                     hashed_password=result.hexdigest()
    
28.                 break
    
29.             print('该用户名不存在。')
    
30.             names=input('请重新输入用户名：')
    
31.     elif commend=='3':
    
32.         break

复制代码

爱吃菠萝包

jackz007 发表于 2025-5-7 22:25
没看懂，什么叫指令发生了改变，你举例说明一下呢。

哥，我跟你说一下哈，你的密码错误，就直接跳到请输入指令了(这运用到实际中也不适用嘛)，我要实现密码错误，然后给一个语句说：请重新输入密码，就是names=input('请重新输入密码’)，然后对密码进行验证

爱吃菠萝包

player-none 发表于 2025-5-7 23:18
求最佳

其实你原来的代码应该可以实现break到最外层的

大佬，牛啊，但是我想问一下原理，能细讲一下吗？这个把else去掉我没看懂这部分

jackz007

爱吃菠萝包 发表于 2025-5-8 21:39
哥，我跟你说一下哈，你的密码错误，就直接跳到请输入指令了(这运用到实际中也不适用嘛)，我要实现密码错 ...

        现在再试试呢？

1. import hashlib
   
2. 
   
3. register = dict()
   
4. while True:
   
5.     command = '1' if not register else input('请输入指令:')
   
6.     if command == '1' :
   
7.         name = input('【注册】-> 请输入用户名：')
   
8.         if name in register . keys() :
   
9.             print('【错误】* 用户已存在 *')
   
10.             print('请重新输入！\n')
    
11.         else:
    
12.             number = input('【注册】-> 请输入密码：')
    
13.             result = hashlib . md5(number.encode('utf-8'))
    
14.             hashed_password = result . hexdigest()
    
15.             register[name] = hashed_password
    
16.             print('恭喜，注册成功~\n')
    
17.     elif command == '2':
    
18.         f = True
    
19.         while f :
    
20.             names = input('【登录】-> 请输入用户名：')
    
21.             if names in register . keys() :
    
22.                 while f :
    
23.                     numbers=input('【登录】-> [%s] 请输入密码：' % names)
    
24.                     result = hashlib . md5(numbers . encode('utf-8'))
    
25.                     hashed_password = result . hexdigest()
    
26.                     if register[names] == hashed_password:
    
27.                         print('恭喜，登录成功~\n')
    
28.                         f = False
    
29.                     else:
    
30.                         print('【错误】：* 密码错误 *')
    
31.                         print('请重新输入！\n')
    
32.             else:
    
33.                 print('【错误】：* 用户不存在 *\n')
    
34.     elif command == '3' :
    
35.         break

复制代码

player-none

爱吃菠萝包 发表于 2025-5-8 22:16
大佬，牛啊，但是我想问一下原理，能细讲一下吗？这个把else去掉我没看懂这部分

你看:

1. while (blablabla):
   
2.     if (blablabla):
   
3.         print(1)
   
4.         break
   
5.     else: # 这个else就没有必要
   
6.         print(2)
   
7.         continue # 这也没必要
   

复制代码

while/for 代码块中最后的那个 continue 没意义，因为本来就要进入下一次循环了，continue 跳过了 0 行代码

关于多余的 else 你可以看看这个例子：

1. if (blablabla):
   
2.     print(1)
   
3. print(2)

复制代码

很明显，要让程序只输出 1/只输出 2 得加else，不然两个都输出。为什么两个都输出呢？print(2) 没有被跳过，它被执行了。

可是为什么加了break就不用else了呢？如果说“要用else”，得像上面一样，“没有被跳过”。可是break后的代码是被跳过了的。

（突然发现这玩意很难表述出来……代码写多了，bug出多了就习惯了）

爱吃菠萝包

player-none 发表于 2025-5-8 22:43
你看:

我悟了，有了break,就不会执行到print(2),所以else就多余了，感谢

player-none

jackz007 发表于 2025-5-8 22:18
现在再试试呢？

没必要这样搞吧，本来就是登陆成功break，外面那个 if names in register 块下面刚好也应该break了，可以直接退到最外层