Kodları bir fonksiyona veya yönteme koymak mantıklı mı ve iyi bir kural nedir? Ben dört satır veya daha büyük bir başparmak kuralını kullanarak ve daha sonra iki kez daha görünür, o zaman bu kodu içeren basit bir işlev / yöntem yapmak. Daha iyi bir uygulama düşünebilir veya herhangi bir işaretçi sunabilir misiniz? Bu, dile özgü bir sorudan ziyade genel bir tasarım modeli sorusudur.
Yanıtlar:
İşlevleri kısmen kodu belgelemenin bir yolu olarak kullanıyorum. Anlamlı bir isimle bir fonksiyon çağırmak kodu anlamayı kolaylaştırır. Bazı durumlarda, tek satırlı bir işlev bile mantıklıdır.
Örneğin, "Temiz Kod" da, Robert C. Martin aşağıdaki örneği vermektedir: Hangisini görmeyi tercih edersiniz? Bu:
// Check to see if the employee is eligible for full benefits
if ((employee.flags & HOURLY_FLAG) &&
(employee.age > 65))
Veya bu?
if (employee.isEligibleForFullBenefits())
Ona her zaman katılmıyorum, ama bu durumda kabul ediyorum. Kod, yalnızca yazdığınızda ve her ayrıntıyı bildiğinizde değil, başka bir kişinin kodundaki hataları düzeltmeniz gerektiğinde saat 9'da okunabilir olmalıdır. Uzun bir koşulda bakmak ve tüm çift negatifleri anlamaya çalışmak önerilmez. Üzerine sadece bir isim koyabilirseniz (sadece koşullar değil, yazdığınız her kod parçası), çok daha basit hale gelir.
Bir işleve bir şey koymaktan hiç pişman olmadım ve performans konusunda endişeleniyorsanız, önce profil yapın.
İşlev çağrılarının yalnızca tekrarlayan kod bölümlerinden kaçınmak için yapılması gerektiği konusunda yaygın bir yanlış anlama vardır. Temel kuralım, herhangi bir mantıksal iş biriminin, yalnızca tek bir yerde kullanıldığında bile bir işleve dönüştürülmesi gerektiğidir. Bu genellikle daha iyi bir okunabilirliğe yol açar ve işlev adlarının yorumların yerini aldığı ve ne yaptığınızı açıklayan ek yorumlar yazmanıza gerek kalmayan kendi kendini belgeleyen kod yazmanıza izin verir.
Den fazla kullanılan geçtiyse bir yerde ve
daha sonra bir işlev veya yöntem yapın. Deneyimlerime göre, uzun tekrarlanan kod parçaları doğal olarak bu kategorilerden birine girecektir (genellikle ilk olan, ancak kategoriler çok fazla örtüşür;). Tabii ki, arayüzde olması gereken her şey kendi başına bir işlev / yöntemdir.
float x, int yve double densitysonra bu hesaplamaları bir C işlevi olarak ayarlamak, üç değeri de elde etmek için bir yol tasarlamanız gerektiğinden, kodu tekrarlamaktan daha zor olabilir. Tekrarlanan hesaplamaların kendileri önemsizse, bazen bunları satır içinde bırakmak daha iyidir.
Neredeyse her zaman, özellikle de her bir kopya aynı işlemi kavramsal bir bakış açısından temsil ediyorsa. Aynı şekilde, ancak farklı türlerde gerçekleştirilirse, genel bir uygulama yapın.
Düşünebilmemin tek nedeni bakımlardan biridir: bazen, ayrı bir şeyler arasında bir bağımlılık yaratmaktan kaçınmak, hatta bazı çoğaltma pahasına bile daha uygun olabilir.
"Şeklinde bir arama üstlenmeden " Bu çok yaygın süreci için sanayi "en iyi uygulamalar" için pek çok kaynağa götürecektir. Bir zamanlar meşhur olan " Once and Only Once" makalesi, bazılarının sorunuzun dile getirdiği endişeler için "en iyi uygulamalar" olarak gördüklerini açıklayan harika bir tarihsel referanstır. Ayrıca, daha genel bir kavram Kendinizi Tekrarlama (DRY) olarak bilinir . Sorunuzun cevaplar gerçekten derinlemesine kümesi için okumak Martin Fowler 'ın büyük klasik, yeniden düzenleme: Mevcut Kanunun Tasarımı iyileştirilmesi için en iyi bilinen tavsiye bazı kapsar üstlenmeden sezgisel olarak gerçekleştirmek için çalışıyoruz ne olduğu, !
Kod bir kereden fazla bir yerde tam olarak tekrarlanıyorsa ve tekrarlanan bölüm yakın gelecekte değişmeyecekse, bir fonksiyona ayırırım.
Bu, tekrarlanan kodun uyumunun doğasına bağlıdır. Kodun tekrarlanan bölümü belirli bir işlevi yerine getiriyorsa, kısmen DRY ilkesi nedeniyle, kısmen işlevin optimize edilmesi veya düzeltilmesi gerekiyorsa, sadece bir bölüm olduğu için bir yönteme dönüştürülmek için mükemmel bir adaydır. başa çıkmak için kod.
İlişkilendirme rastlantısalsa, kodu bir yönteme dönüştürmek yerine tekrarlamak daha iyidir. Bu snippet'in kullanımlarından birini karşılamak için kod dizilerinden birinin ortasına bir şey eklemeniz gerekiyorsa, bir yöntemdeyse, yaptığınız değişiklik o yöntemin diğer kullanımlarını etkileyebilir.
Kod uyumu kavramı hakkındaki Wikipedia makalesine bakın .
Yapısal programlama anlamındaki fonksiyonlar ve bir sınıfın yöntemleri arasındaki farkı ayırt etmelisiniz.
Örneğinizde, gösterdiğiniz şey , bu şekilde bir satır içi kodlanmaması gereken bir yöntemdir .
bir sayı olup olmadığını görmek için bir dizeyi doğrulamanız gerekebilir, bu durumda bir işlev kullanırsınız ve önceki yanıtların çoğu uygulanır.
Bu ayrım özellikle büyük projelerde önemlidir.
Mümkün olduğunca, iş kurallarını (yöntemler) bilgisayar algoritmalarından (saf programlama işlevleri) ayırmaya çalışın.