Ist es besser, eine Funktion aufzurufen, die zu diesem Zeitpunkt keine Auswirkung hat, WENN dies die Code-Klarheit verbessert?

Ich habe drei Ansichten in meinem Programm (iOS-App). Es ist immer nur einer von ihnen gleichzeitig aktiv, daher habe ich die Sichtbarkeit für zwei von ihnen deaktiviert und die Sichtbarkeit geändert, während der Benutzer die Tasten drückt. Die Ansichten werden als sichtbar initialisiert, sodass ich die Sichtbarkeit im Code ausschalte, bevor die Hauptansicht angezeigt wird.

ich kann

[view1 setAlpha:0.0f];
[view2 setAlpha:0.0f];

für zwei der Ansichten, aber jetzt ist die dritte (die zu Beginn der App sichtbar sein sollte) nicht angesprochen. Ich habe eine

[view3 setAlpha:1.0f];

nach den ersten beiden, weil ich denke, es hält klar, dass es tatsächlich drei Ansichten gibt, nicht zwei, wie man vielleicht denkt, wenn man den Code sieht. Wie machen andere Programmierer das? Ist es reine Präferenz oder gibt es einige Konventionen?

Wenn der Anruf sehr schwer ist, ist es natürlich besser, ihn nicht anzurufen, wenn dies nicht erforderlich ist, aber ich habe mich über kleine Dinge wie mein Beispiel gewundert.

Sie haben eine Invariante:

Es ist immer nur eine Ansicht (von 3) aktiv (und sichtbar).

Dann schlage ich vor, dass Sie eine Funktion bereitstellen, um die Aktivität und Sichtbarkeit ALLER Ansichten auf einmal zu ändern:

[setActiveView viewID:2]

Diese Funktion wird:

• Überprüfen Sie, ob die Ansicht bereits aktiv ist, um unnötige Arbeit zu vermeiden
• Stellen Sie die Ansicht als aktiv und sichtbar ein
• Setze die anderen 2 Ansichten als inaktiv und unsichtbar

Es hat mehrere Vorteile über einen rohen Anruf an setVisibility:

• freundlich: unnötiges Aufrufen verursacht kein Leistungsproblem
• defensiv: sein einzelner Parameter ist viel schwieriger zu verpfuschen, wohingegen setVisibilityes schwieriger ist, sich daran zu erinnern, dass der Wertebereich ist 0.0f - 1.0fund dass nur einer eingestellt werden muss1.0f
• belastbar: der nächste kann nicht versehentlich eine der ansichten vergessen
• anpassbar: Zum Hinzufügen / Entfernen einer Ansicht muss nicht der gesamte Anwendungscode überprüft werden, um herauszufinden, wo sich die Schalter befinden. Eine einzelne Funktion (diese) muss aktualisiert werden

Idealerweise sollte keine andere Funktion in der Lage sein, diese Einstellung zu ändern, um die Invariante zu erzwingen ...

Alternative Idee: Wenn Ihr Ziel darin besteht, das Auftreten von Fehlern zu verhindern, weil die Leute vergessen, dass es drei Ansichten gibt, und mit nur zwei davon etwas zu tun haben, das sie wirklich mit allen machen sollten, dann machen Sie eine Funktion, die es unmöglich macht, zu vergessen:

setViewVisibilities(0.0f, 0.0f, 1.0f)

Jetzt haben Sie etwas weitaus Stärkeres - Kompilierungszeitgarantien, die Sie nicht vergessen haben . Wenn Sie einen Parameter vergessen, werden Sie vom Compiler angeschrien. Dies ist weitaus nützlicher als Kommentare oder unnötiger Code, da ein streng benanntes Protokoll erstellt wird, das die von Ihnen gewünschte Eigenschaft erzwingt.

Für den Fall, view3dass die Sichtbarkeit nicht geändert werden muss, können Sie ein Verhalten hinzufügen, bei dem die Übergabe eines bestimmten Werts wie -1.0oder niloder etwas in diese Richtung bedeutet, dass die Sichtbarkeit der Ansicht überhaupt nicht geändert wird. Dies umgeht das Problem des unnötigen Einstellens von Sichtbarkeiten.

Ich würde glauben, dass es am besten ist, einen Kommentar hinzuzufügen, der erklärt, dass der Anruf nicht benötigt wird (und warum).

^{(Vielleicht ist die Tatsache, dass ein Anruf nicht benötigt wird oder dass Sie einen Kommentar dazu benötigen, ein Codegeruch.)}

In diesem speziellen Fall hat @Mattieu M. die richtige Lösung.

Im allgemeineren Fall, in dem es keine ähnliche Transformation gibt, müssen Sie sich fragen: Gibt es eine Chance, dass ein zukünftiger Programmierer dies vermasselt?

Die Antwort lautet normalerweise ja. Das heißt, ja, Sie sollten den Anruf hinzufügen. Möglicherweise beginnt eine zukünftige Version des Frameworks mit allen Sichten auf OFF anstatt auf ON.