Data.table: Membersihkan

Dibuat pada 24 Mei 2019  ·  3Komentar  ·  Sumber: Rdatatable/data.table

Tindak lanjut untuk berkomentar di sini:

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

Juga ditambahkan ke Berkontribusi

Ada jumlah pembersihan manual yang layak untuk dilakukan, kecuali seseorang memiliki regex yang lebih baik untuk menangkap penggunaan <- yang valid:

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

Saya bertanya-tanya seberapa layak untuk hanya mengganti selimut & kemudian menambal di = yang menyebabkan kesalahan karena <- diperlukan

Low beginner-task internals

Komentar yang paling membantu

Seperti @jangorecki , saya rasa saya juga tidak akan berusaha untuk ini. Tapi sekarang Michael punya, cukup menyenangkan untuk menyingkirkan ini; itu nilai yang lebih tinggi dari yang saya kira. Seharusnya membuat PR lebih mudah/lebih cepat, terutama karena kami berharap dapat melihat kontributor baru.
Dengan menggunakan = , kita tidak perlu khawatir tentang bahaya x<-3 ketika x < -3 dimaksudkan. Dan karena penugasan adalah fungsi dalam R yang memiliki nilai (dan -3 dipaksa ke TRUE secara diam-diam), kesalahan ini dapat tidak terdeteksi. Dengan menggunakan = kita tidak perlu menerapkan dan memelihara pemeriksaan kontrol kualitas untuk memastikan ruang ada di sekitar setiap operator. Saya bertanya-tanya berapa banyak orang yang telah diberitahu untuk menggunakan <- , dan yakin <- adalah yang terbaik karena semua ahli setuju jadi itu pasti benar, sadari ini.

Semua 3 komentar

IMO ini bukan hal yang terlalu penting. Saya lebih suka menggunakan = , tetapi ketika saya melihat <- sekitar, maka saya menyelaraskan gaya dalam file yang saya tulis. AFAIR definisi fungsi kami menggunakan <- cukup konsisten.
Sebenarnya tidak ada salahnya (berlawanan dengan apa yang sedang diberitakan secara luas) untuk memiliki keduanya, dengan asumsi orang yang membaca kode data.table tahu R. Kerugiannya mungkin hanya untuk pendatang baru ke bahasa R, kepada siapa khotbah tersebut mungkin ditargetkan. . Bisakah kita menutup yang ini?

Saya sendiri tidak terlalu memperhatikan, tetapi komentar tertaut dan komit ini: https://github.com/Rdatatable/data.table/pull/3580/commits/50243432f2c20310107989606d56ab52fab28ad4

menuntun saya untuk mengajukan ini dalam semangat konsistensi.

Menambahkan Low karena sangat rendah kepentingannya.

Seperti @jangorecki , saya rasa saya juga tidak akan berusaha untuk ini. Tapi sekarang Michael punya, cukup menyenangkan untuk menyingkirkan ini; itu nilai yang lebih tinggi dari yang saya kira. Seharusnya membuat PR lebih mudah/lebih cepat, terutama karena kami berharap dapat melihat kontributor baru.
Dengan menggunakan = , kita tidak perlu khawatir tentang bahaya x<-3 ketika x < -3 dimaksudkan. Dan karena penugasan adalah fungsi dalam R yang memiliki nilai (dan -3 dipaksa ke TRUE secara diam-diam), kesalahan ini dapat tidak terdeteksi. Dengan menggunakan = kita tidak perlu menerapkan dan memelihara pemeriksaan kontrol kualitas untuk memastikan ruang ada di sekitar setiap operator. Saya bertanya-tanya berapa banyak orang yang telah diberitahu untuk menggunakan <- , dan yakin <- adalah yang terbaik karena semua ahli setuju jadi itu pasti benar, sadari ini.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat