Boolesche Rückgabe von set.add () in if conditional?

Der Operator add der Klasse set gibt einen Booleschen Wert zurück, der true ist, wenn das Element (das hinzugefügt werden soll) noch nicht vorhanden ist, andernfalls false. Schreibt

if (set.add(entry)) {
    //do some more stuff
}

als guter Stil in Bezug auf das Schreiben von sauberem Code? Ich frage mich, da Sie zwei Dinge gleichzeitig tun. 1) Hinzufügen des Elements und 2) Überprüfen, ob das Element vorhanden ist.

Ja ist es.

Wenn eine Operation einen Booleschen Wert zurückgibt, können Sie ihn normalerweise verwenden, um Entscheidungen zu treffen, z if Konstrukts. Die einzige andere Möglichkeit, dieses Potenzial zu realisieren, besteht darin, den Rückgabewert in einer Variablen zu speichern und ihn dann sofort in einer wiederzuverwenden if, was einfach albern und dem Schreiben keineswegs vorzuziehen ist if(operation()) { ... }. Also mach einfach weiter, wir werden dich nicht beurteilen.

Ich würde sagen, es ist nicht so sauber wie möglich, weil es den Betreuer zwingt, entweder bereits zu wissen oder nachzuschlagen, was der Rückgabewert bedeutet. Bedeutet dies, dass der Wert bereits vorhanden war, noch nicht vorhanden war und erfolgreich eingefügt wurde? Wenn Sie es nicht oft benutzen, werden Sie es nicht wissen, und selbst wenn Sie es tun, ist es eine viel größere mentale Belastung.

Ich würde folgendes bevorzugen:

boolean added = set.add(entry);

if (added) {
    //do some more stuff
}

Ja, ein bisschen ausführlicher, aber der Compiler sollte so ziemlich den gleichen Bytecode generieren, und selbst Leute, die seit Jahren keine Java-Sets mehr verwendet haben, können der Logik folgen, ohne etwas nachzuschlagen.

Wenn wahr Erfolg bedeutet, dann ist es guter, klarer Code.

Es gibt eine weit verbreitete Konvention, dass eine Funktion oder Methode bei Erfolg true (oder etwas, das als true ausgewertet wird) zurückgibt. Solange Ihr Code dem folgt, denke ich, dass es in Ordnung ist, die Methode in die Bedingung zu setzen.

Code wie dieser ist aus meiner Sicht unnötig unübersichtlich:

boolean frobulate_succeeded = thing.frobulate();

if (frobulate_succeeded) {
    ...
}

Es fühlt sich an, als würdest du dich wiederholen.

Die Frage ist jedoch zweideutig in Bezug auf die Bedeutung des Rückgabewerts. Sie sagen "ein Boolescher Wert, der angibt, ob das hinzugefügte Element bereits vorhanden war". Dies könnte bedeuten, dass true bedeutet, dass das Element vorhanden war (und add nicht aufgetreten ist). In diesem Fall würde ich das Rückgabeverhalten der Methode im Idealfall konventioneller gestalten. Wenn dies nicht möglich ist, würde ich eine zusätzliche Zwischenvariable hinzufügen, mit der Sie das Rückgabeergebnis in Ihrem Code eindeutig kennzeichnen können (wie von anderen vorgeschlagen).

Ich würde sagen, es ist sehr C-like. Die meiste Zeit würde ich es vorziehen, eine beschreibend benannte Variable für ein Mutationsergebnis zu haben, und keine Mutation, die in einer auftrittif Bedingung auftritt.

Ein Compiler entfernt diese Variable, wenn sie sofort wiederverwendet wird. Ein Mensch wird es leichter haben, die Quelle zu lesen. Für mich ist es wichtiger.

Sollte jemand die Bedingung durch Hinzufügen einer and/ or-Klausel zur if-Bedingung erweitern müssen, kann dies dazu führen, dass er .add()in bestimmten Fällen aufgrund einer Kurzschlussbewertung nicht anruft . Wenn nicht ausdrücklich ein Kurzschluss erwartet wird, kann dies zu einem Fehler führen.

Ihr Code scheint die Trennung von Befehlsabfragen zu unterbrechen . Dies wird im Clean Code-Buch und im Funktionsstruktur- Video erläutert . Aus der Perspektive von Clean Code denke ich, dass dies nicht als guter Stil angesehen wird.

„Sauberer Code ist einfach und direkt. Sauberer Code liest sich wie gut geschriebene Prosa. Sauberer Code verdeckt niemals die Absicht des Designers, sondern steckt voller klarer Abstraktionen und direkter Kontrolllinien.

- Grady Booch Autor von objektorientierter Analyse und Design mit Anwendungen “

Für mich ist die Absicht Ihres Codes unklar. Wird das if both ausgeführt, wenn der Eintrag erfolgreich hinzugefügt wurde oder auch wenn er bereits vorhanden ist? Was kehrt add()zurück? der Gegenstand? Der Fehlercode?