`if key in dict` vs.` try / Except` - welche Redewendung ist besser lesbar?

Ich habe eine Frage zu Redewendungen und Lesbarkeit, und es scheint einen Konflikt zwischen Python-Philosophien für diesen speziellen Fall zu geben:

Ich möchte Wörterbuch A aus Wörterbuch B erstellen. Wenn in B kein bestimmter Schlüssel vorhanden ist, tun Sie nichts und fahren Sie fort.

Welcher Weg ist besser?

try:
    A["blah"] = B["blah"]
except KeyError:
    pass

oder

if "blah" in B:
    A["blah"] = B["blah"]

"Tu und bitte um Vergebung" vs. "Einfachheit und Aussagekraft".

Welches ist besser und warum?

Ausnahmen sind keine Bedingungen.

Die bedingte Version ist klarer. Das ist natürlich: Dies ist eine unkomplizierte Flusskontrolle, für die Bedingungen ausgelegt sind, keine Ausnahmen.

Die Ausnahmeversion wird hauptsächlich als Optimierung verwendet, wenn diese Suchvorgänge in einer Schleife ausgeführt werden: Bei einigen Algorithmen können Tests aus inneren Schleifen entfernt werden. Diesen Vorteil hat es hier nicht. Es hat den kleinen Vorteil, dass es nicht "blah"zweimal gesagt werden muss, aber wenn Sie viele davon tun, sollten Sie wahrscheinlich move_keytrotzdem eine Hilfsfunktion haben.

Im Allgemeinen würde ich dringend empfehlen, standardmäßig bei der bedingten Version zu bleiben, es sei denn, Sie haben einen bestimmten Grund, dies nicht zu tun. Bedingungen sind der naheliegende Weg, um dies zu tun. Dies ist normalerweise eine starke Empfehlung, eine Lösung einer anderen vorzuziehen.

Es gibt auch einen dritten Weg, der sowohl Ausnahmen als auch die doppelte Suche vermeidet. Dies kann wichtig sein, wenn die Suche teuer ist:

value = B.get("blah", None)
if value is not None: 
    A["blah"] = value

Für den Fall , erwarten Sie das Wörterbuch enthalten NoneWerte, können Sie einige exotischere Konstanten wie verwenden NotImplemented, Ellipsisoder einen neuen machen:

MyConst = object()
def update_key(A, B, key):
    value = B.get(key, MyConst)
    if value is not MyConst: 
        A[key] = value

Auf jeden Fall update()ist die Verwendung für mich die am besten lesbare Option:

a.update((k, b[k]) for k in ("foo", "bar", "blah") if k in b)

Soweit ich weiß, möchten Sie Diktat A mit Schlüssel-Wert-Paaren aus Diktat B aktualisieren

update ist eine bessere Wahl.

A.update(B)

Beispiel:

>>> A = {'a':1, 'b': 2, 'c':3}
>>> B = {'d': 2, 'b':5, 'c': 4}
>>> A.update(B)
>>> A
{'a': 1, 'c': 4, 'b': 5, 'd': 2}
>>> 

Direktes Zitat aus dem Python-Performance-Wiki:

Mit Ausnahme des ersten Males schlägt der Test der if-Anweisung jedes Mal fehl, wenn ein Wort angezeigt wird. Wenn Sie eine große Anzahl von Wörtern zählen, werden viele wahrscheinlich mehrmals vorkommen. In einer Situation, in der die Initialisierung eines Werts nur einmal erfolgt und die Erhöhung dieses Werts um ein Vielfaches erfolgt, ist es billiger, eine try-Anweisung zu verwenden.

Es scheint also, dass beide Optionen je nach Situation realisierbar sind. Weitere Informationen finden Sie unter folgendem Link: Try-Except-Performance

Ich denke, die allgemeine Regel hier ist A["blah"]normalerweise, wenn ja, versuchen Sie es - außer ist gut, wenn nicht, dann verwenden Sieif "blah" in b:

Ich denke, "versuchen" ist in der Zeit billig, aber "außer" ist teurer.

Ich denke, das zweite Beispiel ist das, wofür Sie sich entscheiden sollten, es sei denn, dieser Code macht Sinn:

try:
    A["foo"] = B["foo"]
    A["bar"] = B["bar"]
    A["baz"] = B["baz"]
except KeyError:
    pass

Beachten Sie, dass der Code abgebrochen wird, sobald ein Schlüssel nicht vorhanden ist B. Wenn dieser Code sinnvoll ist, sollten Sie die Ausnahmemethode verwenden, andernfalls die Testmethode. Meiner Meinung nach ist es viel einfacher zu lesen als die Ausnahmemethode, da es kürzer ist und die Absicht klar zum Ausdruck bringt.

Natürlich sind die Leute, die Ihnen sagen, dass Sie verwenden updatesollen, korrekt. Wenn Sie eine Version von Python verwenden, die das Verständnis von Wörterbüchern unterstützt, würde ich diesen Code sehr bevorzugen:

updateset = {'foo', 'bar', 'baz'}
A.update({k: B[k] for k in updateset if k in B})

In anderen Sprachen gilt die Regel, Ausnahmen für außergewöhnliche Bedingungen zu reservieren, dh Fehler, die bei regelmäßiger Verwendung nicht auftreten. Ich weiß nicht, wie diese Regel für Python gilt, da StopIteration nach dieser Regel nicht existieren sollte.

Persönlich neige ich zur zweiten Methode (aber mit has_key):

if B.has_key("blah"):
  A["blah"] = B["blah"]

Auf diese Weise besteht jede Zuweisungsoperation nur aus zwei Zeilen (anstelle von 4 mit try / exception), und alle Ausnahmen, die ausgelöst werden, sind echte Fehler oder Dinge, die Sie verpasst haben (anstatt nur zu versuchen, auf Schlüssel zuzugreifen, die nicht vorhanden sind). .

Wie sich herausstellt (siehe die Kommentare zu Ihrer Frage), has_keyist es veraltet - also denke ich, es ist besser geschrieben als

if "blah" in B:
  A["blah"] = B["blah"]

Ab Python 3.8dem Start und der Einführung von Zuweisungsausdrücken (PEP 572) ( :=Operator) können wir den Bedingungswert dictB.get('hello', None)in einer Variablen valueerfassen, um zu überprüfen, ob dies nicht der Fall ist None(da dict.get('hello', None)entweder der zugehörige Wert oder zurückgegeben wird None), und ihn dann im Hauptteil von zu verwenden die Bedingung:

# dictB = {'hello': 5, 'world': 42}
# dictA = {}
if value := dictB.get('hello', None):
  dictA["hello"] = value
# dictA is now {'hello': 5}

Obwohl die Betonung der akzeptierten Antwort auf das Prinzip "Schau, bevor du springst" für die meisten Sprachen gilt, ist möglicherweise mehr Python der erste Ansatz, der auf den Python-Prinzipien basiert. Ganz zu schweigen davon, dass es sich bei Python um einen legitimen Codierungsstil handelt. Wichtig ist, dass Sie den try-Block außer im richtigen Kontext verwenden und die Best Practices befolgen. Z.B. zu viele Dinge in einem try-Block tun, eine sehr breite Ausnahme abfangen oder schlimmer noch - die bloße Ausnahmeklausel usw.

Es ist einfacher, um Vergebung zu bitten als um Erlaubnis. (EAFP)

Siehe die Python-Dokumentreferenz hier .

Auch dieser Blog von Brett, einem der Hauptentwickler, geht kurz darauf ein.

Schaue andere SO Diskussion hier :