Kodları Refactor Edelim - ihan3t

[ihan3t]

Merhaba arkadaşlar, bu konumda kötü yazılmış bir kodu refactor edeceğiz,
yani daha iyi şekilde yeniden yazacağız.

Man man = new Man('Adam');
Woman woman = new Woman('Eve');

if(woman.isBeautiful()){

  man.propose(woman);
  
  if(man.isRich()){
    woman.acceptProposal(man);
    man.insertInto(woman);
  }else{
    man.fap();
  }
} else{
  if(man.isDrunk()){
    man.propose(woman);

    if(man.isRich()){
      
      woman.acceptProposal(man);
      man.insertInto(woman);
      //next day
      man.ohGodMy();

    }else{
      man.fap();
    }
  
  }

}

Şimdi burada bir yazılımcı sayfasında gördüğüm, eğlencesine yazılmış bir kod var. Bu kodun bir yazılımcı sayfasında paylaşılması büyük bir ayıp. Çünkü bu kod gerçekten çok kötü yazılmış bir kod. Buradaki kod resmen kokuyor. Çürük kokuyor.

Burada yapılan sözde işlem şudur, kadın güzelse adam teklifte buluyor, kadın güzeldeğilse eğer adam sarhoşsa gene teklifte bulunuyor. İki durumda da eğer adam zenginse kabul ediliyor.


Şimdi bu kodu ilk izlenimde kısaltacak daha okunur hale getirecek işlem şudur :

if(woman.isBeautiful() || man.isDrunk()){
  man.propose(woman)

  if(man.isRich()){
    woman.acceptProposal(man);
    man.insertInto(woman);
  }else{
    man.fap();
  }

}

Kadın güzelse veya adam sarhoşsa teklifte bulun, adam zenginse/değilse işlemi yap.

Burada çok ufak bir ayrıntı var, eğer adam zenginse teklif kabul ediliyor. Yukarıdaki kodda eğer kadın çirkinse man.ohGodMy() metodu çalışıyor. Birincisi bu metodun ekleniş yeri yanlış. Bu yüzden direkt refactor işleminden çıkarıyoruz. Bu metod gerçekten çağrılması gereken yerde zaten çağrılır.

Böylelikle daha uzun ve çirkin bir kodu refactor etmiş oluyoruz.

Herkese iyi forumlar, ihan3t..

 

[ByZehirx]

mantikli okudum
elin sagljik

 

[qRunt11]

Eline sağlık

 

[Leopic]

Ellerinize sağlık .

 

[ihan3t]

Teşekkürler, faydalı olması dileğiyle...

 

[Hacknology]

Ellerine sağlık abim, ilk başladığımds çok fazla çektiğim bir sorun -hatta bazdn hala bunla uğraşırım- gerçekten öğrenmek tecrübe istiyor

 

[xdebron]

bilinçaltım kan ağlıyor, eline sağlık :trl

 

[ihan3t]

Ellerine sağlık abim, ilk başladığımds çok fazla çektiğim bir sorun -hatta bazdn hala bunla uğraşırım- gerçekten öğrenmek tecrübe istiyor

Teşekkürler dostum

bilinçaltım kan ağlıyor, eline sağlık :trl

Onca yazılan kokulu kodlar gözünün önüncen geçti dimi