Data.table: 清除

创建于 2019-05-24  ·  3评论  ·  资料来源: Rdatatable/data.table

后续评论在这里:

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

还添加到贡献

除非有人有更好的正则表达式来捕获有效的<-用法,否则需要进行大量的手动清理:

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

我想知道仅仅一揽子替换然后修补导致错误的=是多么可行,因为<- 🤔

Low beginner-task internals

最有用的评论

@jangorecki一样,我认为我也不会为此付出努力。 但现在迈克尔有了,能把它排除在外真是太好了。 它的价值比我想象的要高。 它应该使 PR 更容易/更快,特别是当我们希望看到新的贡献者时。
使用= ,当x < -3的意思时,我们不需要担心x<-3的危险。 而且由于赋值是 R 中的一个函数,它有一个值(并且-3被强制为 TRUE),所以这个错误可能不会被发现。 使用= ,我们不需要实施和维护质量控制检查以确保每个操作员周围都有空间。 我想知道有多少人被告知使用<-并且确信<-是最好的,因为所有专家都同意,所以它一定是正确的,意识到这一点。

所有3条评论

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),所以这个错误可能不会被发现。 使用= ,我们不需要实施和维护质量控制检查以确保每个操作员周围都有空间。 我想知道有多少人被告知使用<-并且确信<-是最好的,因为所有专家都同意,所以它一定是正确的,意识到这一点。

此页面是否有帮助?
0 / 5 - 0 等级