Pertanyaan yang diberi tag «code-reviews»

Apa yang akan menjadi cara terbaik untuk mengelola kode sumber yang ditinjau dalam repositori kontrol sumber? Haruskah kode sumber melewati proses peninjauan sebelum diperiksa, atau haruskah peninjauan kode terjadi setelah kode tersebut dikomit? Jika ulasan terjadi setelah kode dicek ke repositori, lalu bagaimana itu harus dilacak?

12 version-control  code-reviews  svn 

Perusahaan tempat saya bekerja mengharuskan semua kode ditinjau oleh pengembang lain sebelum dikomit. Anggota tim saya sering frustrasi karena pengembang lain terlalu sibuk mengkode untuk melakukan review, terutama jika itu sangat panjang. Bagaimana Anda memberi insentif kepada pengembang lain untuk melakukan tinjauan kode tepat waktu? (Kami menggunakan git-svn sehingga kami …

12 code-reviews 

Skenario umum dalam pengembangan perangkat lunak adalah meninjau kode kode orang lain. Alat umum untuk melakukan ini adalah membuka Permintaan Tarik. Pertanyaan saya adalah, ketika masalah ditemukan dalam ulasan, haruskah perubahan itu terjadi dikomit secara terpisah (komit baru) atau jika komit yang ada diubah (dengan asumsi tidak ada yang bercabang …

11 git  code-reviews  repository 

Ditutup . Pertanyaan ini perlu lebih fokus . Saat ini tidak menerima jawaban. Ingin meningkatkan pertanyaan ini? Perbarui pertanyaan sehingga berfokus pada satu masalah hanya dengan mengedit posting ini . Ditutup 6 tahun yang lalu . Kami mencoba peninjauan kode wajib pada setiap komit - tidak ada yang masuk ke …

11 teamwork  code-reviews 

Maksud saya segalanya, bukan hanya perubahan skema. Bahkan SELECT sederhana pada kunci primer tidak dapat masuk ke produksi, meskipun telah ditinjau kode oleh pengembang lain (dalam konteks), tanpa ulasan DBA dari setiap pernyataan, diekstraksi dari kode dan dikirimkan dengan output EXPLAIN, detail seberapa sering akan dipanggil, dll, dll. Jika Anda …

11 code-reviews  dba 

Kadang-kadang ada 1% kode yang cukup intensif secara komputasi yang membutuhkan jenis optimasi tingkat rendah terberat. Contohnya adalah pemrosesan video, pemrosesan gambar, dan semua jenis pemrosesan sinyal, secara umum. Tujuannya adalah untuk mendokumentasikan, dan mengajarkan teknik pengoptimalan, sehingga kode tidak menjadi tidak dapat dipelihara dan cenderung dihapus oleh pengembang yang …

11 documentation  code-reviews  optimization 

Pertama saya sangat percaya pada proses peninjauan kode dan selalu ingin orang lain meninjau kode saya. Pertanyaan saya benar-benar terpusat pada bagaimana saya bisa melakukan pekerjaan yang lebih baik dalam melakukan tinjauan kode untuk orang lain? Saya tahu bahwa untuk melakukan tinjauan kode, Anda harus memiliki pengetahuan tentang cara kerja …

11 code-reviews 

Kami memiliki ketergantungan pada layanan pihak ketiga yang memperlihatkan antarmuka raksasa yang hanya kami butuhkan seperti 3 metode. Selain itu, antarmuka sering berubah ... Saya telah memutuskan untuk membungkus antarmuka dalam kelas di proyek kami dan hanya memaparkan metode yang kami butuhkan. Tapi saya tidak yakin bagaimana saya harus menangani …

11 c#  design-patterns  programming-practices  code-reviews  wrapper 

Saat ini saya sedang meninjau beberapa kode pengembang junior yang baru saja bergabung dengan tim saya, saya bertanya-tanya bagaimana saya harus memberikan hasil tinjauan ini: Haruskah saya memperbaiki kode sendiri? Haruskah saya memberi mereka umpan balik tentang proses peninjauan dan membiarkan mereka melakukan perbaikan sesuai dengan instruksi saya? Dan jika …

10 code-reviews  process 

Saya telah mewarisi beberapa kode buruk yang telah saya sertakan contoh singkat di bawah ini. Apakah ada nama untuk pola khusus ini? Apa sajakah rekomendasi untuk refactoring ini? // 0=Need to log in / present username and password // 2=Already logged in // 3=Inactive User found // 4=Valid User found-establish …

10 coding-style  code-quality  coding-standards  code-reviews  code-smell 

Saat ini saya sedang berupaya memperkenalkan React di perusahaan tempat saya bekerja. Sebagai satu-satunya yang telah melakukan kursus tentang topik ini, siapa yang dapat saya minta ulasan untuk memastikan kode mengikuti 'praktik terbaik' dan diterapkan dengan benar?

10 code-reviews 

Ditutup . Pertanyaan ini didasarkan pada pendapat . Saat ini tidak menerima jawaban. Ingin meningkatkan pertanyaan ini? Perbarui pertanyaan sehingga dapat dijawab dengan fakta dan kutipan dengan mengedit posting ini . Ditutup 4 tahun yang lalu . Sebagai hipotesis, jika saya akan mewawancarai seseorang untuk posisi pengembang PHP baru ketika …

10 code-quality  interview  code-reviews 

Saya tahu ada dorongan besar untuk ulasan kode dalam pengembangan komersial. Namun, apakah tinjauan kode digunakan dalam perangkat lunak sumber terbuka atau didasarkan pada kepercayaan? Jika demikian, lalu bagaimana kinerjanya? [Apakah ini komit yang ditunda, "lingkungan sebelum komit", apakah ada alat yang memungkinkan patch dikirim ke pengembang lain]? Apakah ada …

10 open-source  code-reviews  source-code 

Saya berdebat dengan kolega saya tentang kapan harus melakukan tinjauan kode - sebelum atau setelah pengujian unit. Apa praktik terbaik? Beberapa faktor yang perlu kita perhitungkan (mungkin ada lebih banyak): Ukuran perubahan kode - perubahan besar berarti lebih banyak perubahan akan dihasilkan dari tinjauan kode. Jika perubahan ini lebih besar …

10 language-agnostic  code-reviews 

Saya mencari beberapa ide di sini. Saya membaca artikel Bagaimana seharusnya ulasan kode dilakukan dan Ulasan Kode, apa kelebihannya? yang sangat informatif tetapi saya masih perlu kejelasan tentang pertanyaan di bawah ini. Pertanyaanku adalah, Menjadi pengembang target, dapatkah Anda menyarankan beberapa praktik terbaik yang dapat dimasukkan oleh pengembang sebelum kodenya …

10 c#  code-reviews