Pertanyaan yang diberi tag «code-reviews»

Hanya berpikir keras-keras - kami programmer suka semua voting / badges / hal rep ini sehingga skema seperti ini dapat dimasukkan ke dalam proses peninjauan kode perusahaan untuk mendorong pengkodean yang lebih baik. Sesuatu seperti Anda (atau orang lain atas nama Anda) dapat memposting ulasan (dapat berupa cuplikan, komit tunggal …

17 code-quality  code-reviews  motivation 

Saya cukup baru dalam review kode, tetapi saya telah mengkode selama bertahun-tahun selama PhD - yang tidak selalu membuat Anda seorang programmer yang baik! Jika reviewer mengubah kode Anda dan melewatinya dengan Anda baris demi baris, apa yang Anda lakukan jika Anda tidak setuju? Kadang-kadang Anda membuat pilihan X dan …

16 teamwork  code-reviews 

Perusahaan saya baru-baru ini mulai melakukan tinjauan kode formal. Prosesnya seperti ini: Anda mengirimkan ke github, meminta permintaan tarik, kode ditinjau oleh sekitar tiga orang, lalu jika semua lewat, kode Anda masuk. Prosesnya tampak adil, namun tiga orang yang melakukan tinjauan kode tampaknya tidak adil. Saya perhatikan bahwa ketika saya …

16 teamwork  code-reviews 

Prasyarat Tim menggunakan DVCS IDE mendukung parsing komentar (seperti TODO dan lain-lain) Alat seperti CodeCollaborator mahal untuk anggaran Alat seperti gerrit terlalu rumit untuk dipasang atau tidak dapat digunakan Alur kerja Penulis menerbitkan suatu tempat di cabang fitur repo pusat Reviewer mengambilnya dan memulai review Dalam hal beberapa pengulas pertanyaan …

16 teamwork  code-reviews  workflows  dvcs 

Tutup. Pertanyaan ini di luar topik . Saat ini tidak menerima jawaban. Ingin meningkatkan pertanyaan ini? Perbarui pertanyaan sehingga sesuai topik untuk Rekayasa Perangkat Lunak Stack Exchange. Ditutup 4 tahun yang lalu . Sunting: Justin Cave menyatakan dengan baik bahwa komunikasi semacam ini harus ada di depan dalam kutipan / …

16 learning  project-management  freelancing  code-reviews  billing 

Beberapa tim di perusahaan saya mempraktikkan alur kerja tinjauan kode yang belum pernah saya lihat sebelumnya. Saya mencoba memahami pemikiran di baliknya, dengan gagasan bahwa ada nilai dalam membuat seluruh perusahaan konsisten. (Saya berkontribusi pada banyak basis kode dan telah tersandung oleh perbedaan di masa lalu.) Penulis kode mengajukan permintaan …

16 code-reviews  pull-requests 

Mengembangkan aplikasi web yang memungkinkan Pengguna menjadwalkan janji temu berdasarkan TimeZone mereka. Dan saya menyimpan datetime yang dijadwalkan Pengguna sebagai datetime server ke dalam bidang basis data. Sementara menampilkan informasi jadwal mengambil nilai dari Database dan diubah menjadi timzone pengguna. memproses dalam basis kode Saya mengonversi DateTime berdasarkan zona waktu …

16 c#  asp.net  code-reviews 

Kami baru saja pindah ke proses scrum dan sedang mengerjakan tugas dan kisah pengguna di dalam sprint. Kami ingin sering melakukan tinjauan kode untuk membuatnya tidak terlalu menakutkan. Kami berpikir untuk melakukannya pada tingkat cerita pengguna tetapi tidak yakin bagaimana cara memasukkan kode kami ke akun ini. Kami menggunakan VS …

15 agile  code-reviews  scrum 

Sebagai pemimpin tim yang mengelola sekelompok pengembang tanpa pengalaman (dan tidak perlu melihat) dalam tinjauan kode dan pengujian unit, bagaimana Anda bisa memajukan tinjauan kode dan praktik pengujian unit? Bagaimana Anda akan membuat cara agar ulasan kode dan pengujian unit agar secara alami sesuai dengan alur pengembang? Salah satu hambatan …

15 unit-testing  code-reviews 

Tim kami baru-baru ini mulai melakukan tinjauan kode terhadap setiap checkin. Saat tim memimpin saya mencoba menemukan keseimbangan antara memberikan terlalu banyak saran, mengganggu pengembang dan mengurangi output tim, dan melepaskan kode yang saya tulis berbeda. Adakah bukti, studi, atau panduan dari sumber-sumber terkenal yang menyarankan pendekatan yang membantu?

14 code-quality  code-reviews  clean-code  team-leader  process-improvement 

Untuk menghindari angka ajaib, kita sering mendengar bahwa kita harus memberikan nama yang bermakna secara literal. Seperti: //THIS CODE COMES FROM THE CLEAN CODE BOOK for (int j = 0; j < 34; j++) { s += (t[j] * 4) / 5; } -------------------- Change to -------------------- int realDaysPerIdealDay = …

14 coding-style  code-quality  clean-code  code-reviews 

Dalam proses Agile kami, kami memiliki Sprint 2 minggu. Tugas dikirimkan setiap hari (harian dibangun), dan Tim Uji menyelesaikan pengujian mereka segera pada hari berikutnya atau bahkan pada hari yang sama. Kami juga memiliki ulasan kode Dev, yang memerlukan waktu (1-2 jam), sehingga dijadwalkan 3 kali seminggu: Senin-Jumat-Jumat. Pengembang berkumpul …

14 project-management  agile  teamwork  code-reviews  sdlc 

Dalam proyek pemrograman ekstrem, pemrogram melakukan pairing pemrograman hampir sepanjang waktu. Karena pasangan ini juga berputar, yaitu, Anda memasangkan program dengan orang yang berbeda, dan ada rasa kepemilikan bersama, kode sumber sering ditinjau dan diperbarui. Jadi, apakah ada kebutuhan untuk ulasan kode? Maksud saya, hentikan pemrograman dan sebenarnya hanya melakukan …

14 agile  code-reviews  pair-programming  extreme-programming 

Saya adalah bagian dari tim yang terdiri dari tujuh pengembang di perusahaan perangkat lunak kecil dan saya mencoba memperkenalkan kode grup dan ulasan desain. Kami telah melakukan beberapa ulasan di masa lalu, tetapi sudah sporadis. Saya ingin menjadikannya hal yang lebih biasa. Saya telah membaca Kode Lengkap dan sumber daya …

14 code-reviews 

Adakah yang mempraktikkan proses "review kode" untuk tes fungsional? Apakah menurut Anda bermanfaat? Cara majikan saya saat ini melakukan SCRUM, kami menyertakan pengujian fungsional sebagai bagian dari hal-hal "yang harus dilakukan" dalam setiap sprint yang diberikan.

14 scrum  code-reviews  quality