Data.table: Удалять

Созданный на 24 мая 2019  ·  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 Комментарий

Имхо, это не так уж и важно. Я предпочитаю использовать = , но когда я вижу <- вокруг, я выравниваю стиль в файле, в который пишу. 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 рейтинги