DRY, KISS, YAGNI ve Demeter — Değişimi Ucuz Tutmak

Evdeki Wi-Fi şifresini değiştirdin. Buzdolabındaki not, misafir odasındaki kart ve annenin telefonu hâlâ eski şifreyi söylüyor. Kodda da aynısı olur: bir bilgi kaç yerde yazılıysa, değiştiğinde o kadar yeri bulman gerekir.

DRY: bir bilgi, bir sahip

DRY?Terimin tanımını açDRY"Don't repeat yourself": aynı sebeple değişen her bilginin sistemde tek bir sahibi olmalı. Kod satırlarıyla değil, bilgiyle ilgilidir.Sözlükte gör →, aynı satırı iki kez yazmamak değil, aynı bilginin tek bir sahibi olması demektir. KDV oranı bir bilgidir; dört dosyada yazılıysa dört sahibi vardır.

1
// OrderService, InvoicePrinter, CartSummary
2
BigDecimal vat = amount.multiply(new BigDecimal("0.18"));
3

4
// RefundService: the same fact, spelled differently
5
BigDecimal vat = amount.multiply(BigDecimal.valueOf(18)).divide(BigDecimal.valueOf(100));

1
public final class Vat {
2
    public static final BigDecimal RATE = new BigDecimal("0.20");
3
    private Vat() {}
4
}

Peki aynı görünen her şey aynı bilgi midir?

Buna yanlış soyutlama denir: iki bağımsız kuralı tek bir düğmeye bağlamak. Tekrar silinmedi, iki kural birbirine kilitlendi.

Pratik bir ölçü üç kuralı: iki benzer kodu hemen birleştirme. Üçüncüsü geldiğinde neyin gerçekten ortak olduğu görünür.

Demeter: yakınındakiyle konuş

Gazeteci çocuk parasını almak için cüzdanını cebinden çıkarıp içini karıştırmaz. Sen ödersin; cüzdanın deri mi, kartlı mı, onu ilgilendirmez.

Law of Demeter?Terimin tanımını açLaw of DemeterBir metot yalnızca kendisiyle, alanlarıyla ve parametreleriyle konuşsun; onların döndürdüğü nesnelerin içine uzanmasın. a.getB().getC() zincirleri bunu çiğner.Sözlükte gör → bunu koda taşır: bir metot kendi alanlarıyla ve parametreleriyle konuşsun, onların döndürdüğü nesnelerin içine uzanmasın.

1
String city = order.getCustomer().getAddress().getCity();

Bu tek satır üç sınıfın iç yapısını ezberliyor. Müşterinin adresi ikiye ayrıldığı gün, bu satırı yazan her dosya kırılır.

Yakın kuzeni tell, don’t ask?Terimin tanımını açtell, don't askBir nesneden veriyi isteyip kararı dışarıda vermek yerine, nesneye ne istediğini söylemek. Kural, koruduğu verinin yanında kalır.Sözlükte gör →: nesneden veriyi isteyip kararı dışarıda verme. Ne istediğini söyle, kararı nesne versin.

1
public class Account {
2

3
    private final Iban iban;
4
    private Money balance;
5

6
    public Account(Iban iban, Money openingBalance) {
7
        this.iban = iban;
8
        this.balance = openingBalance;
9
    }
10

11
    // Tell, don't ask: the rule lives next to the data it protects.
12
    public void withdraw(Money amount) {
13
        if (amount.isGreaterThan(balance)) {
14
            throw new InsufficientFundsException(iban, amount);
15
        }
16
        balance = balance.minus(amount);
17
    }
18
}

1
@Service
2
public class TransferService {
3

4
    private final AccountRepository accounts;
5

6
    TransferService(AccountRepository accounts) {
7
        this.accounts = accounts;
8
    }
9

10
    @Transactional
11
    public void transfer(Iban from, Iban to, Money amount) {
12
        accounts.get(from).withdraw(amount);   // one friend, one message
13
        accounts.get(to).deposit(amount);
14
    }
15
}

1
// Another way to write it: ask for the data, decide outside, write it back.
2
public void transfer(Iban from, Iban to, Money amount) {
3
    Account source = accounts.get(from);
4
    if (source.getBalance().isGreaterThan(amount) || source.getBalance().equals(amount)) {
5
        source.setBalance(source.getBalance().minus(amount));
6
    } else {
7
        throw new InsufficientFundsException(from, amount);
8
    }
9
    Account target = accounts.get(to);
10
    target.setBalance(target.getBalance().plus(amount));
11
}

KISS ve YAGNI: bugün gerekeni yaz

KISS (keep it simple) işi gören en sade çözümü seçmektir. YAGNI?Terimin tanımını açYAGNI"You aren't gonna need it" — gelmeyen değişime bugünden hazırlanma. SOLID ile gerçek bir gerilimi vardır ve ikisi de haklıdır.Sözlükte gör → ise onun zaman boyutu: gelmemiş bir istek için bugünden kod yazma.

Bir ekip “ileride başka alanlar da gelir” diye siparişe genel bir özel alan çerçevesi kurdu: eklenti arayüzü, kayıt defteri, ayar dosyası. Yıllarca tek bir alan eklenmedi.

İlk gerçek istek bir hediye notuydu. Düz bir alanla üç dosya sürecekti; çerçeve yüzünden yedi dosyaya dokunuldu.

Kendin gör

Simülatör aynı küçük sipariş kodunda bir istek değiştiriyor ve her dosyaya ne olduğunu gösteriyor.

1. Varsayılanla oynat. KDV dört kopya: üçü düzenleniyor, RefundService gözden kaçıyor.
2. Kodu “Sonra” yap. Tek dosya, sürpriz yok.
3. “DRY tuzağı”nı seç ve iki kodu karşılaştır. Birleştirilmiş sabit, inceleme kuralını sessizce değiştiriyor.
4. “Demeter”i seç. Önce dört dosya derlenmiyor; sonra değişiklik Order’da bitiyor.
5. “YAGNI”yi seç. Çerçeveli sürüm yedi, düz sürüm üç dosya açıyor.

Tuzaklar

Her tekrarı silmek. Testlerde biraz tekrar çoğu zaman iyidir: her test tek başına okunur.

Nokta saymak. Stream ve builder zincirleri hep aynı nesneyle konuşur; ezberlenen bir iç yapı yok, Demeter çiğnenmiyor.

Aracıyı şişirmek. Her zincir için Order’a bir iletici metot eklemek onu başkalarının işini taşıyan bir aracıya çevirir. İleticiyi değil, davranışın kendisini taşı.

YAGNI’yi bahane etmek. Yazılı, bilinen bir gereksinimi “belki gelmez” diye ertelemek YAGNI değildir. Ayrıntılı temizlik hamleleri için Code Smell ve Refactoring dersine bakabilirsin.

Kendini sına