Apakah saya tetap bisa membuat perubahan gaya pengkodean pada proyek sumber terbuka yang tidak mengikuti praktik terbaik?


38

Baru-baru ini, saya menemukan sejumlah proyek open source Ruby (atau sebagian besar adalah Ruby) di GitHub yang ketika diperiksa dengan alat penganalisa kode seperti Rubocop , membuat banyak pelanggaran .

Sekarang, sebagian besar pelanggaran ini termasuk menggunakan tanda kutip ganda alih-alih tanda kutip tunggal (bila tidak interpolasi), tidak mengikuti aturan 2 spasi per level, melebihi aturan panjang garis 80 karakter, atau menggunakan {dan }untuk blok multi-garis.

[The] Ruby style guide merekomendasikan praktik terbaik sehingga programmer Ruby dunia nyata dapat menulis kode yang dapat dikelola oleh programmer Ruby dunia nyata lainnya. ~ Sumber: Panduan Gaya Ruby

Meskipun kecil dan mudah diperbaiki, apakah pantas untuk mengubah gaya pengkodean proyek sumber terbuka dengan memperbaiki pelanggaran dan membuat Permintaan Tarik? Saya mengakui bahwa beberapa proyek, seperti Rails, tidak menerima perubahan kosmetik dan beberapa terlalu besar untuk "diperbaiki" sekaligus (Rails misalnya menghasilkan lebih dari 80.000 pelanggaran ketika Rubocop dijalankan - terlepas dari itu, mereka memiliki serangkaian kecil kode sendiri konvensi yang harus diikuti ketika berkontribusi). Lagi pula, Panduan Gaya Ruby ada karena suatu alasan bersama dengan alat-alat seperti Rubocop.

Orang - orang menghargai konsistensi sehingga membuat perubahan semacam ini adalah melakukan hal yang baik untuk komunitas Ruby secara umum, bukan?

[Penulis Panduan Gaya Ruby] tidak datang dengan semua aturan entah dari mana - mereka sebagian besar didasarkan pada karir saya yang luas sebagai insinyur perangkat lunak profesional, umpan balik dan saran dari anggota komunitas Ruby dan berbagai sumber daya pemrograman Ruby yang sangat dihargai, seperti "Pemrograman Ruby 1.9" dan "Bahasa Pemrograman Ruby". ~ Sumber: Panduan Gaya Ruby

Bukankah tidak mengikuti konvensi gaya pengkodean komunitas dan praktik terbaik pada dasarnya mendorong praktik buruk ?


3
Pertanyaan serupa: Haruskah saya refactor kelas F dari iklim kode? , tetapi dengan lebih fokus pada arsitektur.
amon


5
Banyak dari masalah gaya ini, terdengar cukup kecil tbh.
JL235


4
@MathewFoscarini tidak, yang mereka tanyakan adalah, "jika saya membeli senjata radar, dapatkah saya memutuskan apa hukum mengemudi setempat?"
Jon Hanna

Jawaban:


65

Tanyakan pada pengelola.

Gaya pengkodean adalah diskusi yang cukup subyektif, dan aturan seperti panjang garis maksimum 80 karakter cukup subyektif - sementara kesepakatan umum seharusnya bahwa garis pendek lebih baik untuk dibaca, 80 mungkin terlalu ketat untuk beberapa orang dengan ukuran layar saat ini dan IDE.

Aturan lain dapat diabaikan dengan sengaja juga. Misalnya, pengembang mungkin menganggap penggunaan global tanda kutip ganda lebih baik untuknya dan bersedia menerima "risiko" interpolasi yang tidak disengaja dan peningkatan waktu parsing yang sangat kecil.

Banyak pengelola juga tidak menyukai perubahan gaya pengkodean besar karena membosankan untuk ditinjau dan ada kemungkinan bahwa hal itu dapat menyebabkan kesalahan. Sebagai contoh, sebuah string dapat dialihkan ke kutipan tunggal, meskipun itu berisi interpolasi yang disengaja dan seharusnya menggunakan kutipan ganda. Maintainers lebih suka melakukan pembersihan gaya saat mengerjakan kode aktual sehingga mereka dapat memverifikasi perubahan gaya tidak memperkenalkan bug baru.


34
Banyak pengelola juga tidak menyukai perubahan besar yang tidak sepenuhnya diperlukan karena mereka cenderung menciptakan konflik penggabungan yang juga tidak perlu.
Jan Hudec

4
Anda akan kehilangan fungsi anotasi (yang membuat garis kode) dalam kontrol sumber, jika ada bug, sulit untuk menyalahkan di mana ia diperkenalkan dan melacak jika ada lebih banyak kesalahan semacam itu.
rekan

4
Selain itu - jika konvensi khusus proyek konsisten, Anda dapat membuat konfigurasi khusus proyek untuk alat pengecekan konvensi dan menawarkan konfigurasi konvensi sebagai permintaan tarik. Jadi pengelola dapat memeriksa proyek mereka menggunakan konfigurasi mereka. (Saya tidak tahu apakah ini mungkin dengan alat yang Anda gunakan.)
Peter Kofler

+1 pada konflik gabungan. Saya baru saja menerima pengerjaan ulang gaya pengkodean dan saya berharap tidak, karena itu menyebabkan konflik gabungan dengan PR orang lain. Sangat mudah untuk diselesaikan tetapi saya lebih suka melakukannya sepotong demi sepotong
Daenyth

Ini adalah IDE yang membatasi jika tidak memungkinkan menampilkan dua file terpisah berdampingan, bukan kesalahan dari kode pendek.
unperson325680

48

Anda tampaknya termotivasi sebagian besar dengan menghormati otoritas alat rubocop dan Ruby Style Guide, yang mungkin tidak dibagi oleh pengelola. Mereka sudah memiliki gaya mereka sendiri, dan sudah terbiasa dengan hal itu, sehingga setiap perubahan akan memengaruhi setiap orang yang mengerjakan proyek, dan itu banyak pekerjaan, terutama jika proyek itu besar.

Pertimbangkan motivasi para pengelola. Mereka (mungkin) ingin orang baru bergabung dengan mereka dengan mengirimkan perbaikan bug, laporan bug, kode kerja, dan dokumentasi yang ditulis dengan baik. Jika Anda muncul dan berkata "Anda memiliki pelanggaran di rubocop" mereka tidak akan berpikir "Oh bagus, seseorang yang baru membantu berbagi beban", mereka akan berpikir "Siapa orang ini? Mengapa dia memberitahu kami melakukan apa?".

Proyek open source cenderung meritocracies: Anda mendapatkan rasa hormat berdasarkan kualitas pekerjaan Anda. Jika Anda muncul dan melakukan hal-hal besar dan kemudian memunculkan kekhawatiran gaya Anda, mereka lebih cenderung mendengarkan, meskipun mereka mungkin masih mengatakan tidak. Ada sumber terbuka yang mengatakan "Bicara itu murah, tunjukkan kodenya".


1
Jawaban Terbaik. Anda harus menghormati upaya mereka yang datang sebelumnya saat mengajukan permintaan tarik. Mengubah gaya proyek di bawah kaki semua pengembang yang ada, terutama setiap orang yang memiliki 'peringkat' lebih tinggi dari Anda dalam meritokrasi, menyulitkan mereka untuk mengingat aturan baru yang Anda perkenalkan. Ini akan melukai kemampuan mereka untuk mempertahankan proyek seperti sebelumnya. Selain itu, jika Anda sudah aktif dalam pengembangan proyek, Anda mungkin akan tahu pemikiran pengelola tentang perubahan gaya dan titik terbaik dalam siklus hidup proyek untuk memperkenalkannya.
Chris Keele

1
Persis. Misalnya proyek Linux, meskipun memiliki panduan gaya yang sangat ketat untuk potongan kode baru, tidak akan memungkinkan Anda untuk hanya mengirimkan permintaan tarik besar untuk memperbaiki masalah gaya, karena itu buggers dengan penggabungan. Bahkan meminta Anda untuk mempertahankan gaya lokal, bahkan jika itu berarti itu bertentangan dengan panduan gaya proyek.
Miles Rout

25

Pragmatisme atas Dogma, selalu. Panduan gaya pengkodean adalah bentuk kejahatan yang sangat berbahaya yang menarik perhatian dari masalah arsitektur menuju omong kosong sembrono seperti pengutipan tunggal / ganda. Tanyakan kepada diri sendiri: Apakah itu benar-benar membuat perbedaan?

Mereka mungkin baik sampai titik tertentu, tetapi begitu Anda memperlakukan mereka dengan semangat yang hampir religius, Anda sudah melangkah terlalu jauh. Itu adalah pedoman, saran, opini, BUKAN fakta.

Haruskah mereka diabaikan begitu saja? Tidak, ada gunanya menggunakan alat untuk mendapatkan gambaran umum tentang apa yang perlu dilihat, tetapi tidak lebih.

Sungguh mengherankan seberapa sering tipe junior membingungkan opini.


11
Perlu ditambahkan bahwa memformat ulang perubahan cenderung menciptakan konflik penggabungan yang merupakan alasan yang sangat bagus bagi sebagian besar pengelola untuk membenci memformat ulang hanya demi itu.
Jan Hudec

13

Anda dapat menarik perubahan ini hanya jika ada masalah terbuka untuk memperbaiki pemformatan. Kalau tidak, mulailah cabang Anda sendiri, dan jika penulis melihat bahwa lebih banyak orang menggunakan cabang Anda hanya karena lebih mudah dibaca. Mereka akan bergabung di cabang sendiri, tetapi bersiaplah untuk mempertahankan cabang Anda dengan menggabungkan pembaruan dan terus memperbaiki pemformatan.

Jika proyek itu tidak cukup penting bagi Anda untuk tetap mempertahankan cabang Anda sendiri, maka itu tidak layak dibersihkan di tempat pertama.

Permintaan tarik dapat diambil secara pribadi oleh penulis. Itu bukan mekanisme untuk menawarkan kritik, dan memformat ulang semua kode bisa dianggap sebagai kritik.

Jika Anda tidak ingin mempertahankan cabang Anda sendiri, tetapi Anda ingin berkontribusi pada proyek. Buka masalah baru dan jelaskan mengapa format saat ini menyebabkan Anda mengalami masalah, kemudian tawarkan untuk menyelesaikan masalah bagi penulis. Jika penulis setuju, maka mereka akan memberikan masalah kepada Anda dan Anda sekarang memiliki izin untuk mengajukan permintaan.

Anda telah menyentuh subjek yang saya setuju adalah masalah merajalela di GitHub . Selain memformat, ada sejumlah proyek yang salah menggunakan anotasi dan yang menyebabkan kekacauan dengan banyak IDE. Saya dapat memikirkan tiga proyek yang sangat populer yang menggunakan flag yang tidak benar yang menyebarkan pesan peringatan di IDE saya. Saya telah mengirim permintaan tarikan untuk memperbaikinya, tetapi penulis tidak menggunakan IDE yang sama, sehingga permintaan tarikan diabaikan.

Bercabang, menggabungkan dan memperbaiki tampaknya menjadi satu-satunya solusi.


Menurut pendapat Anda, apakah Anda akan menganggap manfaat bagi kontributor masa depan sebagai "alasan" yang sah untuk pelanggaran penetapan Permintaan Tarik? Sebagai contoh, sehingga kontributor masa depan tidak perlu khawatir melihat seluruh basis kode untuk memahami gaya pengkodeannya.
raf

@RafalChmiel Ketika seorang penulis menerima permintaan tarikan, orang yang mengirim tarikan itu akan ditambahkan ke repo dan terdaftar secara publik sebagai kontributor proyek dan mereka tetap terdaftar sebagai kontributor. Saat meninjau permintaan tarik, penulis akan mengingat ini saat memutuskan untuk menerimanya. Ingatlah itu saat mengirim permintaan tarik. Lakukan hal-hal yang layak menjadi kontributor.
Reactgular

Apa yang saya maksudkan adalah akan menguntungkan kontributor masa depan dengan memperbaiki pelanggaran menjadi alasan yang sah untuk menggabungkan PR dengan perbaikan seperti itu? Mengapa pengelola harus menggabungkan PR seperti itu? Mungkin kata-kata dari pertanyaanku agak membingungkan.
raf

2
@RafalChmiel semua yang Anda nyatakan sebagai alasan hanyalah pendapat Anda. Jika itu kode ofensif yang terlihat seperti banteng, maka tulis ketakutan Anda dalam suatu masalah. Jika penulis bertahan melawan tarikan seperti itu, maka bersihkan air mata Anda dengan tisu.
Reactgular

10

Diambil dari situs Rubocop sendiri (penekanan milik saya):

Satu hal yang selalu mengganggu saya sebagai pengembang Ruby - pengembang Python memiliki referensi gaya pemrograman yang hebat (PEP-8) dan kami tidak pernah mendapatkan panduan resmi , mendokumentasikan gaya pengkodean Ruby dan praktik terbaik.

Tolong mengerti:

Tidak ada Panduan Gaya Ruby Resmi

Saya tidak mengatakan bahwa panduan gaya buruk. Tetapi tidak hanya tidak ada panduan resmi, tetapi memiliki panduan gaya adalah pilihan yang dibuat pada tingkat pribadi, proyek, tim, perusahaan. Panduan gaya adalah, untuk menggunakan istilah psikologi, "norma sosial dalam kelompok".

Apa artinya itu bagimu? Nah, jika Anda tidak dianggap sebagai bagian dari grup, itu berarti apa pun yang Anda - atau situs web lain katakan - kemungkinan besar tidak akan berpengaruh pada grup. Jadi, jika Anda bukan kontributor aktif untuk proyek tertentu, maka kemungkinan besar saran Anda akan diabaikan, atau paling tidak akan menjadi pengingat pertimbangan sebelumnya tentang memiliki panduan gaya. Paling buruk itu akan dianggap sebagai penghinaan, atau sebagai penyelundup atau bikeshedder menempel hidung mereka di tempat yang bukan miliknya.

Tidak bisakah Anda menyarankan Panduan Gaya?

Sebenarnya, itulah yang ingin Anda lakukan: Anda percaya pada nilai panduan gaya, Anda sangat menghargai konsistensi, dan Anda ingin menginjili untuk dedikasi pada pedoman gaya terpadu.

Tidak apa-apa, selama Anda benar-benar jelas tentang apa yang Anda maksudkan dan apa yang ingin Anda capai. Jika Anda percaya pada Panduan Gaya tertentu dan percaya itu adalah The One True Style Guide, atau setidaknya lebih baik daripada apa pun yang dilakukan oleh orang-orang kafir yang melanggar hukum ini, maka itu juga tidak masalah.

Tetapi apa yang orang tidak hargai adalah diberi tahu bahwa perilaku mereka tidak sesuai dengan aturan tidak resmi, tidak mengikat, dan sebagian besar arbitrer yang berasal dari sumber yang mereka tidak anggap sebagai otoritas yang sah. Ketika itulah yang ingin Anda lakukan, atau jika hanya itu yang dianggap Anda lakukan, Anda akan mendapatkan lebih sedikit "perawatan karpet merah", dan lebih banyak "penduduk asli yang marah dengan tombak dan panci besar mendidih" pengobatan.


3

Untuk memberikan pendapat alternatif di sini, dalam banyak kasus, perubahan seperti itu akan diterima. Proyek open source cenderung memiliki banyak penulis. Seringkali tidak ada "gaya pengkodean"; gaya adalah apa pun yang digunakan orang yang menulis kode tersebut untuk menggunakannya. Jika satu file ditulis oleh orang yang berbeda dari yang lain, gayanya mungkin juga berbeda. Bahkan dalam proyek-proyek di mana ada gaya konsensus, kecuali mereka memeriksa gaya ini secara teratur, sering tidak digunakan.

Contoh umum dari hal ini dalam pengalaman saya adalah ketika seseorang berkontribusi beberapa kode melalui permintaan tarik yang berkualitas relatif rendah. Namun, kode dapat berfungsi. Orang yang berbeda memiliki pandangan berbeda tentang hal ini. Beberapa orang akan menolak untuk menggabungkan permintaan tarik kecuali gayanya bagus. Beberapa orang tidak peduli, asalkan kodenya berfungsi. Beberapa orang lebih suka memiliki gaya yang baik, tetapi mereka tidak ingin menakut-nakuti kontributor dengan sekelompok "memperbaiki ruang putih di sini" komentar (Saya pribadi selalu merasa sedikit bersalah membuat komentar ini, meskipun saya tahu mereka lebih baik baik dari basis kode, karena itu merasa seperti itu mungkin menakut-nakuti kontributor pergi).

Jadi jangan langsung berasumsi bahwa gaya yang Anda lihat adalah gaya yang diinginkan proyek. Bahkan, itu mungkin dapat digeneralisasi untuk berkontribusi pada open source secara umum: jangan berasumsi bahwa kode yang dimiliki proyek open source adalah kode yang diinginkan .

Anda harus mengetahui beberapa hal, meskipun:

  • Beberapa orang religius tentang gaya. Jika menjadi jelas bahwa mereka tidak mau mengalah, maka jangan repot-repot.

  • Ini adalah masalah besar bikeshed . Semua orang dan saudara mereka memiliki pendapat tentang hal-hal ini. Menggabungkan permintaan tarikan seperti itu bisa jadi sulit karena hal ini.

  • Mungkin juga sulit untuk mendapatkan permintaan tarikan seperti itu karena akan mendapatkan konflik penggabungan dengan sangat cepat; pada dasarnya setiap kali ada bagian dari basis kode yang Anda ubah perubahan, bahkan jika itu dalam cara sepele.

Saya akan tetap dengan pendekatan "minta dulu". Jika mereka terbuka untuk itu, dan Anda bersedia untuk tetap dengan permintaan tarik sampai selesai, maka lakukanlah.


2

Saya dulu suka memiliki panduan gaya yang baik, tetapi mengingat keadaan di Ruby, "Saya sudah pindah".

Pada dasarnya saya hidup dengan apa yang saya kerjakan dan mengikuti konvensi umum yang saya pelajari dari sejumlah pekerjaan.

Untuk Ruby, yang merupakan bahasa pilihan saya, saya juga (di kepala saya) membagi gaya menjadi preferensi dan praktik terbaik yang diterima secara universal. Hal-hal yang diterima secara universal saya dapat mengirimkan perubahan sebagai bagian dari permintaan perubahan untuk masalah atau permintaan cabang fitur.

Contoh masing-masing gaya (menurut saya):

Diterima secara universal untuk Ruby:

  • spasi bukan tab
  • dua indentasi ruang
  • method_names_use_underscores
  • Konstanta mulai dengan Caps.

Diterima secara umum:

  • Jangan gunakan tanda kurung jika tidak perlu
  • Gunakan { }untuk satu blok garis dan do enduntuk blok multi-garis.
  • CONSTANTS_ARE_IN_ALL_CAPS
  • Gunakan pernyataan predikatif ( cond ? true : false) lebih if thenjika ekspresi cocok pada 1 baris.

Preferensi pribadi:

  • Panjang garis 120 karakter
  • whenpernyataan kasus diberi indentasi 2 dari pernyataan kasus.
  • Gunakan tanda kutip tunggal kecuali jika dobel diperlukan.

Tidak ada perjanjian:

  • Pernyataan Kasus
  • Panjang garis

Praktik terbaik:

  • metode kecil, <= 5 baris jika memungkinkan.
  • kelas kecil, <= 100 baris jika memungkinkan.
  • Hindari konstanta
  • Kode memiliki tes

Akhirnya, seperti yang orang lain perinci, Anda harus bertanya terlebih dahulu. Pada akhirnya, kunci gaya adalah komunikasi antara pengembang dan kepekaan terhadap orang lain. Sebagai contoh jika saya ingin membuat perubahan gaya pada proyek open source, saya akan sering melakukan permintaan tarikan untuk satu atau dua fitur aktual atau perbaikan bug terlebih dahulu. Setelah pengelola mengetahui saya dan melihat bahwa saya telah berkontribusi, maka saya dapat menyarankan perubahan gaya. Saya hanya menyarankan mereka. Misalnya "Saat melakukan fitur lain pada proyek x, saya perhatikan bahwa Anda memiliki 4 spasi indentasi dalam beberapa file dan saya bertanya-tanya apakah saya dapat mengubahnya menjadi 2?"


1

Meskipun kecil dan mudah untuk diperbaiki, apakah Anda pikir tidak apa-apa untuk mengubah gaya pengkodean proyek sumber terbuka dengan memperbaiki pelanggaran dan membuat Permintaan Tarik?

Singkatnya, tidak!

Tentu saja, saya berasumsi di sini bahwa ini hanya masalah gaya dan bukan bug yang sebenarnya - menarik permintaan untuk yang terakhir akan selalu masuk akal (IMO.) Saya juga berasumsi bahwa Anda adalah orang asing di proyek dan tidak berhubungan dengan orang-orang yang sudah memeliharanya.

Namun, di antara bahasa apa pun selalu ada beberapa orang yang memiliki preferensi mereka yang berbeda dari panduan gaya, baik atau buruk, dan mencoba untuk menerapkan perubahan gaya pada orang yang tidak Anda kenal , dan proyek yang bukan bagian Anda dari jika tidak ada lagi yang bisa datang di sebagai sedikit kasar. Lagi pula - apa yang Anda capai (pada kenyataannya) jika permintaan itu diterima? Kemungkinan besar, jika anggota dari satu proyek ingin mengubah gaya menjadi sesuatu yang berbeda, mereka akan sudah melakukannya - dan semua yang akan Anda lakukan dengan permintaan itu adalah memaksakan gaya pada mereka yang tidak selalu bekerja lebih baik untuk anggota yang ada.

Ini sedikit berubah jika Anda bersedia berkontribusi pada proyek dengan cara lain selain melakukan "perbaikan" gaya. Saya akan mengatakan jika Anda ingin membuka dialog dengan pengelola tentang bagaimana Anda ingin bekerja pada proyek tetapi menemukan gaya non-standar sulit, maka itu tidak masalah. Tapi saya benar-benar tidak akan hanya berkeliling secara membabi buta membuat permintaan tarikan untuk banyak proyek yang hanya berisi perubahan gaya!


1

Perubahan APAPUN bertanggung jawab untuk memperkenalkan bug, bahkan mengubah format tipografi kode.
Oleh karena itu tidak ada perubahan yang harus dilakukan pada kode kecuali ada kasus bisnis yang valid, dan "tetapi kode tidak terlihat bagus" atau "kode tidak mengikuti standar pengkodean" bukan kasus bisnis yang valid. Risikonya terlalu besar.
Sekarang, jika Anda membuat perubahan besar pada file sumber, menarik seluruh file sesuai dengan standar mungkin dapat diterima, tetapi kemungkinan besar Anda akan membuat perubahan kecil dalam hal ini hampir secara universal lebih disukai untuk menyimpan perubahan Anda sendiri di sejalan dengan kode yang ada meskipun kode yang ada tidak mengikuti standar pengkodean.
Heck, kode bisa jadi hasil dari pembuat kode dan kadang-kadang dibuat ulang. Generator kode terkenal karena menghasilkan kode jelek ...


1

Saya memiliki beberapa proyek .NET yang cukup berhasil, dan saya memiliki beberapa PR dari orang-orang yang tampaknya telah melalui kode dengan ReSharper dan StyleCop dan "memperbaiki" banyak hal. Saya tidak menerima PR tersebut, karena beberapa alasan:

  • Walaupun banyak dari perubahan itu adalah untuk yang lebih baik, beberapa dari mereka berdampak negatif pada kode dalam beberapa cara, biasanya kinerja, dan memetik bagian yang baik terlalu banyak kerja keras.
  • Bahkan jika semua perubahan itu jinak atau bahkan bermanfaat, setiap perubahan dalam setiap file di setiap komit perlu ditinjau, dan sekali lagi, itu hanya memakan waktu terlalu lama.

Yang mengatakan, jika ada yang ingin menambahkan beberapa kesalahan memeriksa atau komentar doc, saya akan menerima PR itu dalam sekejap.


-1

Anda perlu mencari tahu apakah pengelola menganggap pelanggaran gaya pengkodean ini sebagai cacat, atau jika proyek memiliki standar berbeda untuk gaya pengkodean.

Jika memiliki standar yang berbeda, Anda dapat menawarkan untuk membantu mendokumentasikan atau memformalkannya. Maka mungkin ternyata ada beberapa pelanggaran gaya itu, dan Anda bisa memperbaikinya.


ini tampaknya tidak menawarkan sesuatu yang berharga atas jawaban lain yang telah diposting beberapa jam sebelum yang ini
agas
Dengan menggunakan situs kami, Anda mengakui telah membaca dan memahami Kebijakan Cookie dan Kebijakan Privasi kami.
Licensed under cc by-sa 3.0 with attribution required.