Click here to Skip to main content
15,069,020 members
Please Sign up or sign in to vote.
0.00/5 (No votes)
See more:
Python
# Test Login 

userlist = {"user" : "password", "user2" : "password2"}

login = input('do you have an account (Y/N)?')
if (login == 'n'):
    New_User = input('create user: ')
    New_Pass = input('create password: ')
    userlist[New_User] = New_Pass

loop = 'true'
while (loop == 'true'):
    username = input("Please enter your username: ")
    for x in userlist:
        if (username == x):
            password = input("Please enter your password: ")
            if (password == userlist[x]):
                print ("Logged in successfully as " + username)
                loop = 'false'
            else:
                print ("Password incorrect!")
        else: 
            print ("User incorrect")


What I have tried:

Python
# Test Login 

userlist = {"user" : "password", "user2" : "password2"}

login = input('do you have an account (Y/N)?')
if (login == 'n'):
    New_User = input('create user: ')
    New_Pass = input('create password: ')
    userlist[New_User] = New_Pass

loop = 'true'
while (loop == 'true'):
    username = input("Please enter your username: ")
    for x in userlist:
        if (username == x):
            password = input("Please enter your password: ")
            if (password == userlist[x]):
                print ("Logged in successfully as " + username)
                loop = 'false'
            else:
                print ("Password incorrect!")
    else: 
        print ("User incorrect")
Posted
Updated 26-Jan-18 20:18pm
v2

1 solution

For debuf purpose, change your code to:
Python
userlist = {"user" : "password", "user2" : "password2", "user3" : "password3", "user4" : "password4", "user5" : "password5"}

and:
Python
print ("User incorrect "+x)

and run the code.

You will see that the message is printed every time the input is not the username you check. The problem is your logic, you know the user is incorrect only once you have checked against every usernames in your list and that there was no match.
What you have to do:
Python
loop in userlist
  check for a match with input
if there was no match
  user is inxorrect


There is a tool that allow you to see what your code is doing, its name is debugger. It is also a great learning tool because it show you reality and you can see which expectation match reality.
When you don't understand what your code is doing or why it does what it does, the answer is debugger.
Use the debugger to see what your code is doing. Just set a breakpoint and see your code performing, the debugger allow you to execute lines 1 by 1 and to inspect variables as it execute.

Debugger - Wikipedia, the free encyclopedia[^]
27.3. pdb — The Python Debugger — Python 3.6.1 documentation[^]
Debugging in Python | Python Conquers The Universe[^]
pdb – Interactive Debugger - Python Module of the Week[^]
The debugger is here to show you what your code is doing and your task is to compare with what it should do.
There is no magic in the debugger, it don't find bugs, it just help you to. When the code don't do what is expected, you are close to a bug.
   

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

  Print Answers RSS
Top Experts
Last 24hrsThis month



CodeProject, 20 Bay Street, 11th Floor Toronto, Ontario, Canada M5J 2N8 +1 (416) 849-8900