Şimdiye kadar hiç soruna neden olmayan bir hatayı düzeltmek


20

Son zamanlarda bazı kod eskisinden daha sık çalıştırılmasına neden olan bir değişiklik yaptım. Bu, bir hatanın keşfedilmesine yol açar. Bu hata, kodun çalıştırıldığı her an gerçekleşme potansiyeline sahipti, ancak çok nadir çalıştırıldığı için asla ortaya çıkmadı.

Bunu baş geliştiricinin dikkatine sunduğumda, atasözü alıntılayan hatayı düzeltmek yerine hatayı ortaya çıkaran değişikliği geri almamı istedi, "Eğer kırılmazsa, düzeltmeyin".

Şimdiye kadar şanslı olduğumun açık olduğunu söyleyebilirim ama akıl dinlemeyecek.

Yine de düzeltmeli miyim?

Güncelleştirme

Kurşunun teknik olarak benim üzerimde hiçbir yetkisi yok. Sadece görev süresi. Bir yıl öncesine kadar birkaç yıl boyunca projenin tek geliştiricisi oldu ve bence yapıcı eleştiriyi çok iyi almıyor. Değeri için onu eleştirmedim. Sadece böcek asla gelmedi çünkü orada olmadığı anlamına gelmedi işaret etti.


İş parçacığıyla ilgili bir hata mı yoksa başka bir şey mi?
TheLQ

3
O bir nedenden dolayı patron. Kaka fanı vurduğunda kızarttığı kişi olacaktır. Eğer kavrulursa, istediği şeyi yapmadınız, o zaman büyük bir rakete ihtiyacınız olacak.
Martin York

3
Değişikliğiniz geri alındığında bile hatanın oluştuğu bir durum oluşturabilir misiniz? Değilse, belki bir hata değil, bir özellik ^ H ^ H ^ H ^ Hn belgesiz sınırlama.
Steve314

3
Hmm, "Eğer kırılmışsa, başka bir şeyi düzelt." - yeni bir yorum, ona vereceğim.
Orbling

1
"Eğer kırılmazsa, tamir etme"? Ama kırıldı.
StuperUser

Yanıtlar:


26

Hata izlemeniz varsa, gönderin. Kritik ise, kaldırın ve dikkatine sunun. Üstünlüğün onu izleyicide indirmesine izin ver. İşler ters gittiğinde, kağıt izi olacak.


9
Unutmayın: Cover Your Ass :-)
gruszczy

1
Çok şanslı değil. Her şey pantolonun koltuğu. Hata izleme yok, toplama gerekliliği yok, test yok. Yönetim konusunda ısrarcı olmasaydı, muhtemelen sürüm kontrolü bile olmazdı.
Kenneth Cochran

18
Çalışma ortamınızla ilgili açıklamanıza dayanarak, olası yazılım geliştirici size bunu düzeltmemenizi söylediğinde başını sallamalı ve gülümsemelisiniz, sonra devam etmeli ve yine de yapmak istediğinizi yapmalısınız.
Carson63000

2
Ve bir dahaki sefere böyle sorular sormayın :-)
gruszczy

2
@codeelegance: "Hata izleme yok, toplama gereksinimi yok, test yok. Yönetim ısrar etmeseydi muhtemelen sürüm kontrolü bile olmazdı." - Tanrı'nın tatlı Maria annesi !! 1 !! Bu ortam, kullanıcı adınıza tam bir ironik bir şekilde duruyor. :)
Bobby Tables

8

Şahsen ben değdi daha önemli bir miktar daha fazla çaba gerektirmedikçe, düzeltmek istiyorum. "Eğer kırılmazsa, tamir etmeyin" yazılıma uygulamak korkunçtur.

Baş geliştiriciniz patronunuzsa ve ona dokunma diyorsa, bu durumda olmaz.


2
Döngüde ne kadar geç kalıyorlar? Bu potansiyel bir intihar olabilir.
İş

8
"Eğer kırılmazsa düzeltmeyin" yazılıma uygulamak için mükemmel bir kuraldır - sadece yazılım kırıldığında değil.
Steve314

Bozulmamışsa, bozuk kod tanımının ne olduğuna bağlı olarak yazılım için geçerlidir. Oturduğunuz ve düzeltilmesi gereken koda baktığınız an kırılır. Bozuk kod için geçici çözümler uygulamaya başladığınız anda, aslında geriye doğru çalışıyorsunuz ve zamanla bozuk kodun düzeltilmesi daha zor ve zor olacak ...
Ernelli

2

Cevapların ve yorumların çoğu, bir hata raporu oluşturarak ve bir başkasının çağrı yapmasına izin vererek kararın sorumluluğunun azaltılmasını önerdi.

Ben böcek izci yok (ve eğer biz kendim dışında kimse kullanacak şüpheliyim) bir sonraki en iyi şey yaptım. Baş geliştiricinin başına geçtim. Durumu yönetime açıkladıktan sonra işleri yolumda gördüler. Bana düzgün bir şekilde düzeltmemizi ve potansiyel müşterinin talep talebini görmezden gelmemizi söylediler . Alt yarışı bulup şikayet ederse karıştırılmış tüyleri yumuşatacaklarını söylediler.

İdeal bir çözüm değil ama en azından hata düzgün bir şekilde düzeltildi.


Sen emir komuta zincirinde çalıştın, ve böylece poponuzu kapattın. Hata izleme yazılımını kullanmak şirketinizin göz önünde bulundurması gereken bir şeydir, en azından böyle şeyleri ve özellik isteklerini vb.
İzlemenize

2

Ona, "Eğer müşteri fark etmediyse, düzeltmeyin" değil, "Eğer kırılmazsa, düzeltmeyin" ifadesini hatırlatın.


1

Yaptığınız değişiklik için ne gibi bir gerekçeniz var? Kullanıcının hangi değişiklikleri yaşayacağına veya teknik borcun kaldırıldığına işaret edemezseniz, bu sadece işleri daha da kötüleştirdiği için değişikliği geri almak anlamına gelir.


Burada aklımda en az birkaç farklı seçenek var:

Sadece devam edip hatayı düzeltirseniz, karışıma geri tepebilecek daha fazla hata ekleme riskiyle karşı karşıya kalırsınız. Ne kadar deneyime sahip olduğunuza ve muhtemelen burada rehberim olacak bazı kötü sürprizlerden kaçınmaya bağlı olarak.

Size söylenenleri yaparsanız, bu sadece sorun olacak bir suçluluk mudur yoksa bundan daha fazlası mıdır? Burada ilke ve değer olarak bilinen şeylerden başka neyin yanlış olduğunu merak ediyorum. Yani biraz şaka gibi ama aynı zamanda bu fikirde neyin yanlış olduğuna dair dürüst bir nokta?


Değişiklik başka bir hatayı düzeltmek için gerekliydi. Kurşun, hata nedenini düzeltmek yerine kodun ne sıklıkta çalıştırıldığını sınırlayan şartlar koymamı önerdi. Hem düzelttiğim hata hem de bulduğum hata şov tıpaları.
Kenneth Cochran

3
Bu durumda düzgün bir şekilde sabitlenmesi gerekir. Bir sorunun etrafına sarılmış koşulları kullanmak sadece felaketi ortadan kaldırır ve yanlış gitmek için daha fazla yeni kod ekler. Kötü (aptalca) bir çözüm.
hızla

1

Benim ezici içgüdüm sorunu gizlemek değil hataları düzeltmek için olsa da, burnumu tutacak ve sorunu gizlemek senaryolar vardır.

  1. Kod dahili olarak ve bazen kullanılır, bu nedenle hatanın sonuçları şirket içinde yönetilebilir.
  2. Bugün nakliye gerektiren zorlu ticari hususlar ve bir hata düzeltmesi 2 hafta sonra asgari sonuçlarla ortaya çıkabilir.

Profesyonel olarak, bu cevapları sevmiyorum ve dahili olarak bu durumların eterinin meydana geldiğini açıklığa kavuşturacağım.


0

Nihayetinde, amirinizin açıkça söylemediği hiçbir şey yapmamalısınız. Pozisyonunuzda yapılacak en iyi şeyin, sahip olduğunuz hata izleme veritabanında bir hata raporu oluşturmak olduğuna inanıyorum. bu şekilde en azından herkes sorunun farkındadır ve daha yetki sahibi biri bununla ne yapacağına karar verebilir.


0

Buggy işlevini kopyalayın, düzeltmeyi uygulayın, yeniden adlandırın, belki biraz gizleyin ve bunun yerine çağırın.

İki showtopper hata yorumunuza dayanarak, en iyi seçiminiz yasa mektubunu takip etmek olabilir, ancak ruhunu görmezden gelmek olabilir.

Açıkçası bir kes-yapıştır kodlama dezavantajı var, ama sorunlarınız en az olacak gibi görünüyor.


1
kötü fikir, eğer programcılarımdan birini böyle bir şey yaparken yakalarsam, onu yerinde ateşlerdim, bu koda zarar vermek ve bu kodu korumak zorunda olan herkes için garantili bir yoldur.
Miki Watts

@Miki - bu durumda, bana tam olarak kötü bir fikir olmadığını söyle? Lütfen bir hatayı nasıl geri koyduğunuzu açıklayın çünkü başka bir hatayı (zaten oradaydı) daha görünür hale getirmek iyi bir şeydir. Olayda ateş etmeye gelince, geliştiricinin çok az seçenek bıraktığı için yapıcı işten çıkarmayı tartıştım.
Steve314
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.