2017-11-05 3 views
-1

Alles, was ich verbessern könnte ?? Ich bin ein Python Anfänger. Gibt es etwas, was ich tun könnte, um meinen Code sauberer zu machen oder ihn nur professioneller aussehen zu lassen? DankSieht alles in meinem Code gut aus?

while True: 
    print('What password will you be storing?') 
    PassChoice = input() 
    print('What is your username or email for this account?') 
    UserOrEmail = input() 
    print("What is the password for " + str(PassChoice) + '?') 
    Password = input() 
    print('Your password for ' + str(PassChoice) + ' has been stored.') 
    WebsiteAndPassword = (str(PassChoice) + '| ' + str(UserOrEmail) +': ' + 
str(Password) + '\n') 
    break 

PasswordFile = open('importantpasswords.txt', 'a') 
PasswordFile.write(str(WebsiteAndPassword)) 
PasswordFile.close() 
+6

Für Fragen zu Arbeits Code Gehe zu https://codereview.stackexchange.com. –

+2

Einige Beobachtungen: Halten Sie sich an Community-anerkannten Standards (PEP-8 zum Beispiel) und speichern Sie keine Passwörter in einer einfachen Textdatei;) –

+0

Voting zu schließen Frage als Stellungnahme basierend –

Antwort

0

Persönlich würde ich die getpass https://docs.python.org/3/library/getpass.html Bibliothek verwenden, wenn ein Benutzer Passwort bekommen und auch Sie konnten es vereinfacht, indem Sie:

passChoice = input("What password will you be storing?") 

oder mit getpass:

import getpass 

getpass.getpass(prompt="What password will you be storing?") 
2

Beachten Sie, dass diese Seite nicht für Code-Review ist, aber da Sie neu, es ist etwas, das wirklich offensichtlich ist:

PassChoice = input('What password will you be storing?') 
    UserOrEmail = input('What is your username or email for this account?') 
    Password = input('What is the password for ' + PassChoice + '?') 

Wenn Sie den Benutzer zu einer Eingabe fragen wollen, Sie können Ihre Zeichenfolge als Parameter für die Eingabefunktion angeben. Außerdem müssen Sie PassChoice nicht in Zeichenfolge string, da es der Typ ist, der input() zurückgibt.

+0

Sie sollten vermeiden, "Passwörter" in Klartext-Eingabefeldern sowieso geben –

+0

Ja, das ist sicher, er fragte, wie man seinen Code verbessern kann. Ich hoffe, das ist für Bildungszwecke. Das Speichern echter Passwörter in Textdateien wäre albern. – scharette

+0

Vielen Dank für die nützlichen Informationen. Ich dachte auch, dass es dumm ist, aber ich weiß einfach nicht anders. Irgendwelche Ideen? –

Verwandte Themen