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.
Wenn Ihr Code * funktioniert *, fragen Sie nach Optimierung/Beratung auf http://codereview.stackexchange.com/ – Idos
Verwendung von Variablen wie 'user4',' user5' usw. ist fast immer ein Hinweis, dass stattdessen eine Liste namens "user" definiert werden soll. –
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). –