Data.table: Säubern

Erstellt am 24. Mai 2019  ·  3Kommentare  ·  Quelle: Rdatatable/data.table

Follow-up, um hier zu kommentieren:

https://github.com/Rdatatable/data.table/pull/3582#discussion_r287075480

Auch zu Contributing hinzugefügt

Es gibt eine anständige Menge an manueller Bereinigung, es sei denn, jemand hat eine bessere Regex, um die gültigen Verwendungen <- abzufangen:

grep -r "<-" R | wc -l
628

Ich frage mich, wie machbar es ist, die = , die Fehler verursachen, einfach pauschal zu ersetzen und dann zu patchen, weil <- erforderlich ist 🤔

Low beginner-task internals

Hilfreichster Kommentar

Wie @jangorecki glaube ich auch nicht, dass ich mir Mühe gegeben hätte. Aber jetzt hat Michael, es ist ganz nett, das aus dem Weg zu räumen; es ist höherwertig als ich dachte. Es sollte PRs einfacher/schneller machen, zumal wir hoffentlich neue Mitwirkende sehen.
Mit = brauchen wir uns keine Gedanken über die Gefahr von x<-3 zu machen, wenn x < -3 gemeint war. Und da die Zuweisung eine Funktion in R ist, die einen Wert hat (und -3 stillschweigend auf TRUE gezwungen wird), kann dieser Fehler unentdeckt bleiben. Mit = müssen wir keine Qualitätskontrollprüfungen implementieren und aufrechterhalten, um sicherzustellen, dass sich in der Nähe jedes Bedieners ausreichend Platz befindet. Ich frage mich, wie viele der Leute, denen gesagt wurde, dass sie <- verwenden sollen, und davon überzeugt sind, dass <- am besten ist, weil alle Experten der Meinung sind, dass es richtig sein muss, dies erkennen.

Alle 3 Kommentare

IMO ist das nicht wirklich so wichtig. Ich bevorzuge die Verwendung = , aber wenn ich <- in der Nähe sehe, richte ich mich am Stil in der Datei aus, in die ich schreibe. AFAIR verwenden unsere Funktionsdefinitionen ziemlich konsequent <- .
Es ist nicht wirklich schädlich (im Gegensatz zu dem, was weithin gepredigt wird), beides zu haben, vorausgesetzt, Leute, die data.table-Code lesen, kennen R. Der Schaden könnte nur für Neulinge in der R-Sprache sein, an die diese Predigten wahrscheinlich gerichtet sind . Können wir diesen schließen?

Mir selbst fällt nicht wirklich viel auf, aber der verlinkte Kommentar und dieser Commit: https://github.com/Rdatatable/data.table/pull/3580/commits/50243432f2c20310107989606d56ab52fab28ad4

veranlasste mich, dies im Geiste der Konsistenz einzureichen.

Low hinzugefügt, da die Wichtigkeit extrem gering ist.

Wie @jangorecki glaube ich auch nicht, dass ich mir Mühe gegeben hätte. Aber jetzt hat Michael, es ist ganz nett, das aus dem Weg zu räumen; es ist höherwertig als ich dachte. Es sollte PRs einfacher/schneller machen, zumal wir hoffentlich neue Mitwirkende sehen.
Mit = brauchen wir uns keine Gedanken über die Gefahr von x<-3 zu machen, wenn x < -3 gemeint war. Und da die Zuweisung eine Funktion in R ist, die einen Wert hat (und -3 stillschweigend auf TRUE gezwungen wird), kann dieser Fehler unentdeckt bleiben. Mit = müssen wir keine Qualitätskontrollprüfungen implementieren und aufrechterhalten, um sicherzustellen, dass sich in der Nähe jedes Bedieners ausreichend Platz befindet. Ich frage mich, wie viele der Leute, denen gesagt wurde, dass sie <- verwenden sollen, und davon überzeugt sind, dass <- am besten ist, weil alle Experten der Meinung sind, dass es richtig sein muss, dies erkennen.

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen