Kopyalama / yapıştırma kalıbı nasıl düzeltilir?

Çalıştığım yerde, insanlar (danışmanlar) özellikleri olabildiğince hızlı yayınlamak için baskı altında hissediyorlar. Bu nedenle, işleri doğru şekilde nasıl yapacağınızı düşünmek için çok fazla zaman harcamak yerine, hiçbir şeyi kırmak istemedikleri için, kod farklı modüllerden kopyalanır ve değiştirilir.

Kod tabanı tüm şirkete açık olduğundan bunu önlemek kolay değildir. Birçok insan bu konuda çalışıyor.

Artık karışıklık zaten var, bu fazlalıkları çok fazla kırmadan kaldırmanın en iyi yolu nedir?

Cevabın bir kısmı Yeniden Düzenleme .

Öncelikle, değişikliklerinizle yanlışlıkla bir şey kırmamanız için birim testleri yazmaya başlayın. Ardından, tasarımı geliştirmeye, kopyaları vb. Küçük adımlarla kaldırmaya, her adımdan sonra birim testlerinizi çalıştırmaya, testlerden herhangi biri başarısız olursa herhangi bir sorunu çözmeye veya kolayca çözebileceğinizden daha büyük bir sorunla karşılaşırsanız hemen geri dönmeye başlayın.

Diğer kısım eğitimdir .

İnsanlara kötü kodları geride bırakmamaları öğretilmelidir. Bu kesinlikle uzun vadeli bir savaştır, çünkü alışkanlıkların ve düşünce süreçlerinin değiştirilmesi zor (bazen imkansız) . Bununla birlikte, onsuz, yeniden düzenlenmesi için çığlık atan sonsuz bir kötü kod kaynağı almaya devam edeceksiniz.

İyi ve kötü kodlama alışkanlıkları hakkında tartışma açmak ve birincisinin yararlarını yaymak için grup kodu incelemeleri yapmayı seçebilirsiniz. "Böyle kod yazmalısınız (söylememelisiniz) demek yeterli değildir, insanları mantık ve zor gerçeklerle ikna etmeniz gerekir. Gibi "Eğer varsa yöntemin bu parça kod tabanına üzerinde çoğaltılamaz n defa, sen şansını bir hata olduğunu yöntemde bulunursa, bu yöntem kodunun her kopyasında sabit olacağını ne düşünüyorsunuz?"

Şirketinizin ayrıca danışmanlar için teşvik ve kabul kriterlerini gözden geçirmesi gerekebilir - özensiz kod yazarak kurtulabilirlerse, kesinlikle daha kolay yolu seçmeye devam edecektir. Şirket uzun vadeli sürdürülebilirlik üzerinden "hızlı teslimat" değer vermeye devam ederse, hiçbir şey değişmeyecek: --( Bu yüzden yönetimi ile de tartışmak gerekebilir. Onları anlamak için bir yolu şudur: yeniden düzenleme kodu temiz tutmak, Yeniden düzenleme yapmamak, kredi kartınıza borç toplamak gibidir .. Bir süre bununla kurtulabilirsiniz, ancak satın alma alışkanlıklarınızı ve borçlarınızı aktif olarak yönetmiyorsanız, bir gün omuzlarınızda kaçınılmaz olarak parçalanacaktır. Bir yazılım projesinin hayatında iflas, projenin sürdürülemez hale geldiği zamandır: sıfırdan yeniden yazmak, mevcut kod tabanına yeni bir özellik eklemekten daha kolay hale gelir. Ya da kullanıcılar, sadece rekabete geçecekleri düşük seviyedeki destek ve özelliklerden bıkmış olurlar.

@Peter gibi eğitimin bir parçası olarak, PMD gibi bir kopyala ve yapıştır dedektörü tanıtabilir ve kodlama standartlarınızın bu kısmını uygulamaya yardımcı olmak için bunu çevriminizin bir parçası olarak kullanabilirsiniz.

Standartları kodlayan projelerinizin bu modeli kapsadığından emin olun, böylece tartışmaya başlamak için bir taban çizginiz olur.

insanlar (danışmanlar) özellikleri mümkün olan en kısa sürede yayınlamak için baskı altında hissediyorlar

Teknik bir sorunun yok, sosyal bir sorunun var. Gerçekten de bir yönetim sorununuz var.

Kod tabanı tüm şirkete açık olduğundan bunu önlemek kolay değildir. Birçok insan bu konuda çalışıyor.

"Kod tabanı tüm şirkete açık" bir sorun değil. Önemli değil.

Önemli olan kopyala ve yapıştır için bir yönetim ödül sisteminin mevcut olmasıdır. Temel neden, insanların kopyala-yapıştır için ödüllendirilmesidir (yani, ücretli veya övülmüş veya terfi ettirilmiş veya uzatılmış).

Kültürü, "özellikleri olabildiğince hızlı bir şekilde serbest bırakmak için bastırılmış" durumundan "uygun, iyi test edilmiş kod tabanı değişikliklerini yaptığınız için ödüllendirilen" e değiştirmeden bunu kıramazsınız.

Yapman gerek

1. En üste, ödülleri güçlendiren yöneticilerle başlayın. Mevcut uygulamayı ortaya çıkarmalı ve maliyetleri ve riskleri belgelemelisiniz. Maliyetleri ve riskleri azaltan bir alternatif önermelisiniz.

2. Bu kuruluştaki görev sürenizin geri kalanı için maliyet ve riski acımasızca belgelemek ve ortaya koymak zorundasınız. Amansız. Gerçek tabanlı. Maliyet ve Risk. Her hafta daha fazla maliyet ve kopyala yapıştır nedeniyle daha fazla risk.

3. Yöneticilerin, iyi görünmelerini sağlayacak yeni yaklaşımı kredilendirmelerine yardımcı olmanız gerekir ve göz ardı edilirsiniz.

Kopyala ve yapıştır işlemlerini azaltmak çok önemlidir. Ancak bir kurumun kültürünü değiştirmek zordur. Çok fazla gerçek vermelisiniz ve davayı sizinle aynı fikirde olmayan yöneticilere tekrar tekrar vermelisiniz.

Şimdi bu çürümeye başlamıştı bir kod tabanı var. Temel olarak diğer modüllerde aynı statik işlevlerle özdeş olan modül başına 10'dan fazla statik işlevim vardı. Her biri davranmış sadece mümkün olduğunca çabuk şeyler yapmanın uğruna yeni enkarnasyona gerektirecek farklı yeter.

Bugün başka bir özellik daha eklemek zorunda kaldım ve daha fazla dayanamadım. Yeni bir kütüphane oluşturdum, 100 + işlevlerini bit bayraklarına göre davranışlarını hafifçe değiştiren 10 yeniden işlevle birleştirdim ve daha sonra bu kütüphanede yapılan herhangi bir değişikliğin başka hiçbir şeyi bozmadığından emin olmak için bir dizi test yazdım.

Toplam harcanan süre: 4 saat. Gerekirse 20 saatlik bir maratona gitmeye hazırdım ve ne kadar çabuk kontrol altına aldığımı görünce şaşırdım. Bonus olarak, daha sonra bir grup başlık bağımlılığı sorununu düzeltmek daha kolaydı. Ayrıca, özel mülklerimizin çoğu artık bağlantı için statik nesnelerde bulunduğundan, kaynak koduna daha önce sahip olduğumuzdan daha fazla erişen müşterilerimize verebiliriz.

Benim tavsiyem: şimdi yapmadan önce mermiyi ısır ve tekrar et, gerçekten kötü olur . Muhtemelen düşündüğünüz kadar uzun sürmez, ancak her ihtimale karşı kendiniz için yeni bir şube oluşturun.

Ayrıca, yine kapıdan özellikleri almak yapıştırmak / kopyalayabilirsiniz ederken temel sorunu düzeltmek. İşiniz bittiğinde, yapıştırılan öğeleri yırtın ve bunun yerine yeni kitaplığı kullanın.

Şimdiye kadar verilen cevaplara katılıyorum. Malısın:

• birim testleri oluşturma
• refactor
• Eğitim vermek
• kodlama standartlarına çaba göstermek ve ihlalleri tespit etmek

Ancak diğer taraftan, insanların bunu kopyalayıp yapıştırmasına neyin sebep olduğuna bakmanız gerekir.

• insanlar kodu çok iyi bir şekilde kullandığından kodu iyi bir şekilde kullanamayabilir
• insanlar kullanabilecekleri bir kütüphane olduğunu bilmeyebilirler
• Kütüphane kodu yeterince genel olmayabilir ve kendi sürümünüzü oluşturmak mevcut bir kütüphaneyi kullanmaktan çok daha kolaydır
• İyi bir sürüm oluşturma (kaynak kontrolü değil) stratejisi olmayabilir ve genel bir kitaplığın değiştirilmesi, diğer birçok uygulamanın da test edilmesine neden olabilir.

Bu yüzden yeniden kullanımı kolaylaştırmak için ihtiyacınız olan kopyala / yapıştır desenini durdurmayı düşünüyorum.

• kütüphaneleri bulunabilir ve iyi belgelenmiş hale getirme
• kütüphaneleri her şeyden bağımsız hale getirin
• iyi bir versiyonlama stratejisi düşünün
• geriye dönük uyumluluk sağlamak
• kütüphanelerin kolay genişletilebilirliğini düşünün

Çerçeve Tasarımı Yönergeleri'ni okuyun

Bu yardımcı olur umarım.

Güçlü bir "kopya pastası zararlı kabul edilir" tutumu vardır. Bence iyi, ama biraz fazla ileri gidiyor. Üçgenleştirme sürecindeki bir adım olarak iki yöntem veya sınıf arasındaki benzerlik ve farklılıkları keşfetme alıştırması olarak kopya yapıştırma sağlıklıdır. Ancak, tam bir üçgenleme eksikliği - kopyala yapıştırma ile getirilen kopyayı ortadan kaldırmak - gerçekten zararlıdır.

Bu daha nüanslı tutumu kullanmanın yollarını bulabilirseniz, geliştiricilere "bu kötü değil!" Değil, daha ziyade "eksik, yeniden düzenlemeyi tamamlamak için benimle çalışabilir misiniz?"

Burada aynı sorunla ilgileniyorum ve benim üstümde: Açıktan kaçınmaya çalışmayın, çok kötü olduğunda sadece refactor.

Şu anda başlangıçta başka bir modülün kopyası olarak çalıştığım modül, şimdi farklı olması gereken her şeyi değiştiriyorum. Bu yapıldıktan ve yeni modül bittiğinde, onu orijinal modülle karşılaştıracağım ve hangi parçaların az çok değişmediğini ve bir kütüphaneye, soyut ebeveyn sınıfına vb. Taşınması gerektiğini öğreneceğim.

Sorumlu kim hatalı. Bir kişinin her kod satırını gözden geçirmesi beklenemez, ancak standartları ve zaman çerçevelerini belirler.

Yükleniciler (veya bir projede kısa vadeli olanlar), sadece ilk kez çalıştıkları için telafi edildikleri pozisyona getirilebilirler. En kısa zamanda yapılmasını sağlamak için bazı teşvikler vardır. Kopyalanan kodun hiçbir zaman değiştirilmesi gerekmeyebilir ve bu kod onlar tarafından değiştirilmez.

Onları kendi zamanlarında düzeltmeye zorlayabilirsin. Sonra bunu en baştan yapmaya başlayacaklar, ama sonra işleri halletmek için çok zaman harcayacaklar. AmmoQ'nun sorunlara neden olan şeyleri yeniden düzenleme konusunda doğru fikri olduğunu düşünüyorum.

Kopyala / yapıştır kodunu ortadan kaldırmanın tek yolu (IMHO) kod incelemeleridir, kodu kontrol etmek için bir kişi (veya tercihen daha fazla) bulundurun ve bir kopyala / yapıştır eyleminden gibi görünen kodu bulduklarında programcıyı yeniden düzenleyin.

Önerildiği gibi bu esas olarak organizasyonda bir sorundur. İnsanları eğiterek başlayın (konumunuzun üstündeki doğrudan yönetim katmanını unutmayın). Treninize bir veya iki kişi girmeye ve virüsün yayılmasına izin vermeye çok yardımcı olur. Çoğunluk bunun iyi bir fikir olduğunu düşündüğünde, tarifini alın ve bu şekilde kalmasını garanti etmek için incelemeleri tanıtmaya çalışın. Bu çok yavaş ve sıkıcı bir süreçtir, ancak hızlı bir şekilde değişemez. İlk başta ekstra zamana mal olacak, bu yüzden yönetimin uzun vadeli hedefi bilmesi ve desteklemesi önemlidir.

@Anders K. Yorumlar, uygulamayı yerinde tutmak için iyi bir araçtır. İnsanları kod yazmaya zorlarken, buna inanmıyorlar çok fazla sürtünme yaratıyor. Mümkün olan en kısa sürede eski alışkanlıklarına geri dönecekler. İvme kazanmak için eğitime başlamanız gerektiğine inanıyorum.