2017-11-05 5 views
-1

私が改善できるものは何ですか?私はPythonの初心者です。私のコードをよりきれいに見せたり、もっとプロフェッショナルに見せるために何かできることはありますか?このサイトは、コードレビューのためではなく、あなたは新しいしているので、実際には明白であるものがあることに感謝し私のコードのすべてが見栄えが良いですか?

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

をhttps://codereview.stackexchange.comにアクセスしてください。 –

+2

いくつかの声明:コミュニティで受け入れられている標準(例えばPEP-8)に準拠し、プレーンテキストファイルにパスワードを格納しないでください) –

+0

意見をベースに質問を閉じる投票 –

答えて

0

個人的にユーザーのパスワードを取得するとき、私はGETPASS https://docs.python.org/3/library/getpass.htmlライブラリを使用すると、また、あなたがやってそれを単純化している可能性:

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

またはGETPASSとを:作業コードに関する質問は

import getpass 

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

注:

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 + '?') 

あなたが入力するためのユーザーをお願いしたいと思います 、文字列を入力関数のパラメーターとして指定することができます。またPassChoiceinput()が返す型であるため、文字列に変換する必要はありません。

+0

あなたは平文の入力フィールドに "パスワード"を与えないでください。 –

+0

はい、確かに彼はコードを改善する方法を尋ねていました。私はこれが教育目的のためであることを望みます。実際のパスワードをテキストファイルに格納するのはばかげているでしょう。 – scharette

+0

有益な情報ありがとうございました。私もそれは馬鹿だと思っていたが、私はそれをやっている他の方法を知らない。何か案は? –

関連する問題