2016-06-25 5 views
-1

ちょっと私はコーディングが新しく、この比較的簡単なコードを作成して、自分の基本的な言語知識としました。しかし、私はパスワードが長すぎる場合、それをシャットダウンすることを含むいくつかの望ましくないコードと関係していました。誰もがこれを改善することで私を助けることができます。事前にロックコード(誰かが改善する方法を教えてくれますか)

おかげで、 ジョシュ

import time 


passwordx = float(input('create your password')) 
def password(): 
    passwordx 
    if passwordx>9999: 
     print('password is wrong\n innitiating shutdown') 
     x=5 
     time.sleep(1) 
     print(x) 
     x-=1 
     time.sleep(1) 
     print(x) 
     x-=1 
     time.sleep(1) 
     print(x) 
     x-=1 
     time.sleep(1) 
     print(x) 
     x-=1 
     time.sleep(1) 
     print(x) 
     x-=1 
     time.sleep(1) 
     print(x) 
     x-=1 
     quit() 



    else: 
     print('code saved') 
     x=1 
     while x <1000: 
      print('###############') 
      x+=1 

def login(): 
    passwordguess = float(input('please enter password')) 

    if passwordguess == passwordx: 
     print('access granted') 

    else: 
     print('wrong') 
     print('here\'s a clue', (passwordx-passwordguess)**2) 
     login() 

password() 
login() 
+0

これはあなたが改善することができると考えていること**ワーキングコード**の場合は、[codereview.se]を参照してください。そうでない場合は、[mcve]で問題を明確にしてください。 – jonrsharpe

+0

ごめんなさい申し訳ありませんがこれを取り下げます –

答えて

-1

いくつかの一般的なもの:

  1. は絶対に必要に変数の有効性を制限します。
  2. 異なる変数の依存関係を互いに限定するようにしてください。
  3. 可能な限り枝を避けてください。

Architectures Optimization Reference Manual

PythonSpeed

+0

私はあなたに迷惑をかけて申し訳ありませんが、私はあなたが使用している用語を理解していない私はわずか14歳であり、新しいものすべてとほとんど動作する方法を知っています。申し訳ありません –

+0

私の好みのプログラミング言語は実際にはC#ですが、Pythonで同じ最適化を適用することができます。 – Wiffzack

+0

私は多くの言語を学んでいますが、ちょうど始めたばかりです。そして、私はPythonが始めるのが良い場所だと聞きました。最初は私はちょうど私の知識と理解で上記のようなものを行うことができましたが、今はYouTubeのビデオをコピーしていたが、明らかにそれは完全ではありません。 –

関連する問題