Ich bereite gerade Code mit pylint vor, um ihn für die Validierung vor dem Commit verwenden zu können. Ich habe eine Menge "unbenutztes Argument" Warnung, wenn sie tatsächlich verwendet werden. Hier ist ein Beispiel, das ein falsch positives Ergebnis auslöst.
%Vor%Ich möchte diese Warnung nicht global unterdrücken, weil ich die Zeiten sehen möchte, in denen ein Argument wirklich ungenutzt ist. Gibt es eine andere Option, die manuell einen Deaktivierungskommentar bei jedem Vorkommen hinzufügt? Ist es ein bekanntes Problem mit Pyint?
Dies ist ein vernünftiges Verhalten von pylint; Wenn das übergebene Objekt unveränderlich ist, dann ist die gegebene Aussage im Wesentlichen ein No-Op. Nur wenn es veränderlich ist, scheint es falsch zu sein.
Wenn Sie die Warnung nicht global deaktivieren möchten, müssen Sie sie leider pro Instanz deaktivieren.
Sie können es für jeden Bereich deaktivieren, indem Sie Folgendes hinzufügen:
%Vor%Siehe Ссылка
pylint ist im Allgemeinen ein guter Indikator für schlechten Stil. Selbst wenn es ein "falsch positives" gibt, liegt es wahrscheinlich daran, Dinge gegen Konvention zu machen. Ich bin kein Experte, aber ich würde sagen, dass eine Funktion, die nur eine Nebenwirkung hat, nicht optimal ist. Einige Leute (Robert Martin in Clean Code zum Beispiel) gehen so weit zu sagen, dass alle Nebenwirkungen Lügen sind.
Ich würde empfehlen (wieder, ich bin kein Experte):
%Vor%Argumente sollten nur Eingabe und Ausgabe sollte über Rückgabewert sein. Ausgangsargumente sind, soweit ich weiß, schlechte Methoden.