Data.table: Purge

Créé le 24 mai 2019  ·  3Commentaires  ·  Source: Rdatatable/data.table

Suite à commenter ici:

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

Également ajouté à la contribution

Il y a une quantité décente de nettoyage manuel à faire, à moins que quelqu'un n'ait une meilleure regex pour attraper les utilisations valides <- :

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

Je me demande s'il est faisable de remplacer simplement la couverture, puis de corriger les = qui causent des erreurs car <- est requis 🤔

Low beginner-task internals

Commentaire le plus utile

Comme @jangorecki , je ne pense pas non plus que j'aurais fait des efforts là-dedans. Mais maintenant que Michael l'a fait, c'est plutôt agréable de s'en débarrasser; c'est une valeur plus élevée que je ne le pensais. Cela devrait rendre les relations publiques plus faciles / plus rapides, d'autant plus que nous voyons de nouveaux contributeurs, espérons-le.
En utilisant = , nous n'avons pas à nous soucier du danger de x<-3 alors que x < -3 était visé. Et puisque l'affectation est une fonction dans R qui a une valeur (et que -3 est forcé à TRUE en silence), cette erreur peut passer inaperçue. En utilisant = , nous n'avons pas besoin de mettre en œuvre et de maintenir des contrôles de qualité pour garantir que les espaces sont autour de chaque opérateur. Je me demande combien de personnes à qui on a dit d'utiliser <- et qui sont convaincues <- est la meilleure parce que tous les experts sont d'accord, donc ça doit être juste, réalisez-le.

Tous les 3 commentaires

IMO ce n'est pas vraiment ce truc important. Je préfère utiliser = , mais quand je vois <- autour, je m'aligne sur le style du fichier dans lequel j'écris. AFAIR nos définitions de fonctions utilisent <- manière assez cohérente.
Il n'y a pas vraiment de mal (contrairement à ce qui est largement prêché) à avoir les deux, en supposant que les personnes qui lisent le code data.table connaissent R. Le mal pourrait être réservé aux nouveaux venus dans le langage R, à qui ces prêches sont probablement destinés. . Pouvons-nous fermer celui-ci ?

Je ne remarque pas vraiment grand-chose moi-même, mais le commentaire lié et ce commit : https://github.com/Rdatatable/data.table/pull/3580/commits/50243432f2c20310107989606d56ab52fab28ad4

m'a amené à classer cela dans un esprit de cohérence.

Ajout Low car il est extrêmement peu important.

Comme @jangorecki , je ne pense pas non plus que j'aurais fait des efforts là-dedans. Mais maintenant que Michael l'a fait, c'est plutôt agréable de s'en débarrasser; c'est une valeur plus élevée que je ne le pensais. Cela devrait rendre les relations publiques plus faciles / plus rapides, d'autant plus que nous voyons de nouveaux contributeurs, espérons-le.
En utilisant = , nous n'avons pas à nous soucier du danger de x<-3 alors que x < -3 était visé. Et puisque l'affectation est une fonction dans R qui a une valeur (et que -3 est forcé à TRUE en silence), cette erreur peut passer inaperçue. En utilisant = , nous n'avons pas besoin de mettre en œuvre et de maintenir des contrôles de qualité pour garantir que les espaces sont autour de chaque opérateur. Je me demande combien de personnes à qui on a dit d'utiliser <- et qui sont convaincues <- est la meilleure parce que tous les experts sont d'accord, donc ça doit être juste, réalisez-le.

Cette page vous a été utile?
0 / 5 - 0 notes