ここにコメントするためのフォローアップ:
https://github.com/Rdatatable/data.table/pull/3582#discussion_r287075480
寄稿にも追加
誰かが有効な<-
の使用法をキャッチするためのより良い正規表現を持っていない限り、かなりの量の手動クリーンアップを行う必要があります。
grep -r "<-" R | wc -l
628
<-
が必要なため、エラーの原因となる=
を包括的に置き換えてからパッチを適用することは、どれほど実現可能かと思います🤔
IMOこれはそれほど重要なことではありません。 私は=
を使用することを好みますが、 <-
が表示されたら、書き込み先のファイルのスタイルに合わせます。 AFAIRの関数定義は、非常に一貫して<-
を使用します。
data.tableコードを読む人がRを知っていると仮定すると、(広く説教されていることとは対照的に)実際には害はありません。害は、おそらくそれらの説教が対象としているR言語の初心者にのみある可能性があります。 。 これを閉じてもいいですか?
私はあまり気づいていませんが、リンクされたコメントとこのコミット: https ://github.com/Rdatatable/data.table/pull/3580/commits/50243432f2c20310107989606d56ab52fab28ad4
一貫性の精神でこれを提出するように私を導きました。
重要度が非常に低いため、 Low
追加しました。
@jangoreckiのように、私もこれに力を入れていなかったと思います。 しかし今、マイケルはそうしています。これを邪魔にならないようにするのはとてもいいことです。 思ったより高い値です。 これにより、PRがより簡単に、より速くなるはずです。特に、新しい貢献者がいることを願っています。
=
を使用すると、 x < -3
が意味されたときに$ x<-3
の危険性を心配する必要はありません。 また、割り当ては値を持つRの関数であるため(そして-3
はサイレントにTRUEに強制変換されます)、このエラーは検出されない可能性があります。 =
を使用すると、すべてのオペレーターの周囲にスペースがあることを確認するために品質管理チェックを実装および維持する必要はありません。 <-
を使用するように言われ、 <-
が最善であると確信している人の何人がいるのだろうか。
最も参考になるコメント
@jangoreckiのように、私もこれに力を入れていなかったと思います。 しかし今、マイケルはそうしています。これを邪魔にならないようにするのはとてもいいことです。 思ったより高い値です。 これにより、PRがより簡単に、より速くなるはずです。特に、新しい貢献者がいることを願っています。
=
を使用すると、x < -3
が意味されたときに$x<-3
の危険性を心配する必要はありません。 また、割り当ては値を持つRの関数であるため(そして-3
はサイレントにTRUEに強制変換されます)、このエラーは検出されない可能性があります。=
を使用すると、すべてのオペレーターの周囲にスペースがあることを確認するために品質管理チェックを実装および維持する必要はありません。<-
を使用するように言われ、<-
が最善であると確信している人の何人がいるのだろうか。