«code-reviews» etiketlenmiş sorular

Bu etiket, kod incelemesi ve kod kılavuzları hakkında sorular içindir. Mevcut, çalışan kodun yorumları için lütfen http://codereview.stackexchange.com adresine bakın.

4
Çekme isteğinde bir TODO nasıl kullanılır?
Bu soru edildi göç o Yazılım Mühendisliği Stack Exchange'de yanıtlanabilir çünkü Yazılım Kalite Güvence ve Test Stack Exchange. Geçen sene göç ettim . Bir çekme isteğindeki değişiklikleri gözden geçirdiğimde, bazen aşağıdakilerden dolayı farklı nedenlerle bulunabilecek bir "TODO" notu olan bir yorumumda yanıldım: Bir problemi çözmek için kullanılan çözüm geliştirilebilir, ancak …

10
Bir uygulamanın kötü bir kod temeli üzerine inşa edildiğini nasıl ispat edersiniz?
Şu anda daha önce işimde çalışan bazı geliştiriciler tarafından oluşturulan bir sistemi inceliyorum. Sistem, kullanıcının bakış açısından oldukça iyi çalışıyor, ancak kod incelemesine geçerken çok karışık bir durum. Kullanımın yüksek artışlarına rağmen, uygulamanın oluşturulma şeklinin gelecekteki güncellemeler için geçerli olmayacağına ikna oldum. Sorun şu ki, ne kadar kötü olduğunu bildiğimden …

4
Yalnız programcılar için Başvuru / Kod incelemeleri?
'Uygun bir fiyata' başvurular hakkında iyi ve teknik tavsiyeler verecek herhangi bir hizmet var mı? Bir çok projede genellikle tek geliştiriciyim ve bazen verimliliğin, daha iyi MVC etkileşimlerinin, vb. İçin çalışmalarımın iyileştirilmesi gerektiğine inanıyorum. böyle yorumlar yapacağım

6
Kodumu minimum süre incelemesi için nasıl belgeleyebilirim? [kapalı]
Kapalı . Bu sorunun daha fazla odaklanması gerekiyor . Şu anda cevapları kabul etmiyor. Bu soruyu geliştirmek ister misiniz? Soruyu, yalnızca bu yazıyı düzenleyerek tek bir soruna odaklanacak şekilde güncelleyin . Geçen sene kapalı . Kodumu belgelemek istiyorum, aylar sonra tekrar kodu okumak ve taramak için minimum gereksinim olacak. Farklı …


3
Github olmadan Github benzeri "çekme istekleri"
Veri hassasiyeti nedeniyle bulutta herhangi bir veri depolamayacak bir finansal kurum için analist olarak çalışıyorum. Ancak, ekibimin Git'i kod yönetimi için kullanmasını sağlamada biraz başarılıyım. Kendi sunucumuzda Github benzeri çekme isteklerini uygulamanın herhangi bir yolu olup olmadığını merak ediyordum. İlgilendiğim özellik , belirli bir dalda birleştirilmeden yorumlar için bir değişiklik …
21 git  code-reviews 

17
Programcıları temel kuralları izlemeye ikna etme
Programcılardan çok sık takip etmelerini istemem gereken birkaç kural var. Kod yazarlar ve eğer çalışırlarsa, onlar için iş yapılır. En temel kurallar şunlar olabilir: Değişikliklerin yerine getirilmesi Görünüm veya Denetleyicilere Model sorunları yazmama Sabit kodlamadan kaçının Bana deneyiminden bahseder misin? Bunu nasıl yönetiyorsunuz?

9
Patronun kod incelemesinin iyi bir şey olduğuna ikna etme konusunda ipuçları [kapalı]
Kapalı . Bu sorunun daha fazla odaklanması gerekiyor . Şu anda cevapları kabul etmiyor. Bu soruyu geliştirmek ister misiniz? Soruyu, yalnızca bu yayını düzenleyerek tek bir soruna odaklanacak şekilde güncelleyin . 4 yıl önce kapalı . Diyelim ki, nadiren projelerde birlikte çalışan birkaç geliştiriciye sahip olan varsayımsal bir şirkette çalışıyor …

4
Daha az deneyimli bir programcıdan kod düzeltmeyi nasıl yapmalıyım?
Küçük bir arka plan: 10 kişilik bölümümüz için iki programcıdan biriyim (geri kalanı sanatçılar ve yönetim). İkimiz, şeylerin iyi akması ve ortaya çıkan projeleri geliştirmesi için gereken tüm kodlamaları yapıyoruz. Yaklaşık 4 yıldır programlıyorum, bu onun ilk "gerçek" işi (koyduğu gibi). Genellikle herhangi bir zamanda farklı projeler üzerinde çalışıyoruz. Birkaç …
19 code-reviews  bug 

5
Standart kod incelemesi ne içerir?
Şirketimde, hangi özelliğin uygulandığına ve kodu yazan kişi tarafından ne tür bir hatanın düzeltildiğine dair bir e-posta tartışması. Ve postayı alan gözden geçiren, kodu gözden geçirecek ve kodun kalitesini ve kodun nasıl düzenleneceğini tartışacak. Standart kod incelemesi ne içerir?

5
Kodun “olması” kötü olduğunda gözden geçiren olarak ne yapmalıyım?
Zorunlu kod incelemelerinin birkaç ay önce tanıtıldığı, kötü tasarlanmış, yapılandırılmış bir projede çalışıyorum. Yeni bir özellik uygulayan önemli bir kod parçasını gözden geçirmem istendi. Kod tabanımızın geri kalanıyla aynı kusurları taşır. Büyük ölçüde, bu kusurların yeni koda sürüneceğini anlıyorum. uyumluluğunu korurken kötü tasarlanmış bir sınıftan miras almak veya kötü tasarlanmış …

3
İyi çalışsa bile yaşlı ve patronlu programları gözden geçirmek iyi mi?
Şirketimde, herhangi bir projeyi teslim etmeden önce, patronum yaşlılardan benden veya diğer ekip üyeleri tarafından yazılan programları gözden geçirmelerini ister veya bazen patron da gözden geçirilmek üzere bizimle birlikte oturur. Bilgi edinmenin iyi bir yolu olduğunu düşünüyorum, ancak bazen programlar iyi çalıştığında, incelemeden sonra aynı şekilde çalışmazlar ve programıma tekrar …

12
Tüm kodumuzu gözden geçirmeye çalışmalı mıyız?
Şu anda geliştirme sürecini değiştiriyoruz ve taahhütlerimizin% 100'ünün akranını gözden geçirmeye çalışıp çalışmadığımızı merak ediyorum. Kod incelemeleriyle ilgili deneyiminiz nedir? Onlara çok fazla zaman harcıyor musunuz (günde 1/2 saat diyelim), ya da sadece 5/10 dakika boyunca yağma mı yapıyorsunuz? Günlük / hafta / sprint / proje için harcayacağınız sabit bir …

2
Mercurial ile kod incelemeleri için önerilen işlem
Genelde Perforce ve SmartBear'ın Kod Ortak Çalıştırıcısını kullandık Big Corpve şimdi bazı projeler için Mercurial kullanacağız. Code Collaborator desteği Mercurial (sürüm 5'i kullanıyoruz) ve kod inceleme için en iyi zamanın (kesin / sunucuya gönderme sırasında) işleminin en iyi / verimli zaman olduğunu belirlemeye çalışıyorum Teşekkürler

5
Serbest bırakma ateşi sırasında etkili kod incelemesi nasıl yapılır?
Serbest bırakma için son tarih yarın, kolejiniz sonunda bu sürüm için çok önemli olan görevini bitirdi, proje yöneticisi omzunuzun üzerinde duruyor ve sonunda bir yapı yapmanız için baskı yapıyor ve inceleme sırasında kolej kodunuzda bir kusur fark edersiniz. Kritik değil, ama yarın serbest bırakılmasaydı gitmesine izin vermeyeceğiniz bir şey. Ve …

Sitemizi kullandığınızda şunları okuyup anladığınızı kabul etmiş olursunuz: Çerez Politikası ve Gizlilik Politikası.
Licensed under cc by-sa 3.0 with attribution required.