2016-07-26 9 views
0

Ich fing gerade an, Python zu lernen und fragte mich, ob es eine bessere Möglichkeit gab, das Folgende zu programmieren.Python, Bessere Art der Codierung. Loop-Array verwenden?

user1 = "username" 
userkey1 = "userkey" 
user2 = "username" 
userkey2 = "userkey" 
user3 = "username" 
userkey3 = "userkey" 
user4 = "username" 
userkey4 = "userkey" 
user5 = "username" 
userkey5 = "userkey" 
user6 = "username" 
userkey6 = "userkey" 
user7 = "username" 
userkey7 = "userkey" 

level = ["variable2", "variable2"] 

connect = ipaddress 
for article in connect.link("var1", "var2"): 
    if article["variable"] == '' and article["creator"] in level: 
     try: 
      action = dr.create(
        **{"person": user1,} 
      ) 

      output = output.sign([userkey1]) 

      output = tx.JsonObj(tx) 

      # Broadcast to network 
      broadcast 

    if article["variable"] == '' and article["creator"] in level: 
     try: 
      action = dr.create(
        **{"person": user2,} 
      ) 

      output = output.sign([userkey2]) 

      output = tx.JsonObj(tx) 

      # Broadcast to network 
      broadcast 

    if article["variable"] == '' and article["creator"] in level: 
     try: 
      action = dr.create(
        **{"person": user3,} 
      ) 

      output = output.sign([userkey3]) 

      output = tx.JsonObj(tx) 

      # Broadcast to network 
      broadcast 

Ich bin noch nicht so vertraut Whit While-Schleifen, Schleifen oder Arrays, aber es scheint mir der Code über etwas effizienter sein kann?

Der Code soll Eingabe greifen und nur immer Zyklus, wenn es findet, was seine Suche nach seinen eine Aktion für jeden Benutzer und ihren jeweiligen Schlüssel erstellen soll.

Jede Hilfe wäre willkommen.

+3

Wenn Ihr Code * funktioniert *, fragen Sie nach Optimierung/Beratung auf http://codereview.stackexchange.com/ – Idos

+2

Verwendung von Variablen wie 'user4',' user5' usw. ist fast immer ein Hinweis, dass stattdessen eine Liste namens "user" definiert werden soll. –

+0

Speichern Sie die Benutzer in einer [Liste] (https://docs.python.org/2/tutorial/introduction.html#lists) oder [dict] (https://docs.python.org/2/tutorial/datastructures) .html # Wörterbücher). –

Antwort

0

Wenn Ihr Code funktioniert, wie Sie es wollen, gut. Aber die Definition der langen Variablen ist zu viel. Nachschlagen (Google) Python-Liste und Wörterbuch. Schauen Sie sich diesen Beitrag an: How to implement associative array in Python?