Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Jakub Wojt:
Będziemy rozmawiać o konkretach, czy też temat wątku pozostanie "otwarty" ? :)

private Translator translator;

private Translator getTranslator()

{  if (translator == null)

     translator = new Translator();



  return translator;

}

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Mi naprawdę nie chodzi o złożony kod, czy też istotny z punktu widzenia logiki. Mam na myśli raczej proste rzeczy jak np.

private Translator translator;

private Translator getTranslator()

{  if (translator == null)

     translator = new Translator();



  return translator;

}

Ja twierdzę, że usuwanie tego typu konstrukcji za wszelką cenę to zwyczajna strata czasu.

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Wprowadźmy licznik Dubla:
Niech N oznacza liczbę skopiowanych linijek kodu. Niech M oznacza liczbę wystąpień N skopiowanych linii kodu. Jeśli N>3 i M>1, wówczas taki kod należy zamknąć w odrębną jednostkę w ramach danego zasięgu.

Może być?

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Jakub Wojt:

Mi naprawdę nie chodzi o złożony kod, czy też istotny z punktu widzenia logiki. Mam na myśli raczej proste rzeczy jak np.

private Translator translator;

private Translator getTranslator()

{  if (translator == null)

     translator = new Translator();



  return translator;

}

Ja twierdzę, że usuwanie tego typu konstrukcji za wszelką cenę to zwyczajna strata czasu.

Usuwanie akurat takich "dubli" może być dla programisty bardzo niebezpieczne.
Jeśli mi nie wierzysz - spytaj architekta / lead programisty projektu w którym pracujesz...

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Jakub Wojt:

Wszystko wskazuje na to (powyższy przykład) że chodzi o przerabianie singletonów na multipletony (?) ;) więc zapewne formalizacja nie będzie potrzebna ;)

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Rafał Głowiński:

Jakub Wojt:
Wszystko wskazuje na to (powyższy przykład) że chodzi o przerabianie singletonów na multipletony (?) ;) więc zapewne formalizacja nie będzie potrzebna ;)

Ale to nie jest singleton. "private Translator translator;" - ta zmienna nie jest statyczna... wiec traktowałbym to tylko i wyłącznie jalo lazy-init (?) pola "translator"...

Tak czy inaczej, cały ten temat chyba się zdegenerował przez brak sensownych przykładów ze strony autora wątku.

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Sebastian Malaca:
Mi naprawdę nie chodzi o złożony kod, czy też istotny z punktu widzenia logiki. Mam na myśli raczej proste rzeczy jak np.

private Translator translator;

private Translator getTranslator()

{  if (translator == null)

     translator = new Translator();



  return translator;

}

Ja twierdzę, że usuwanie tego typu konstrukcji za wszelką cenę to zwyczajna strata czasu.

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Jakub Wojt:
Usuwanie akurat takich "dubli" może być dla programisty bardzo niebezpieczne.

Sławomir Orłowski:.
Ale czy chodzi tu o to, że ktoś chce napisać "uniwersalną metodę", która ominie pisanie w geterze tego "if"?

Rafał Głowiński:
Tak czy inaczej, cały ten temat chyba się zdegenerował przez brak sensownych przykładów ze strony autora wątku.

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Dariusz Wawer:
Ogólna zasada, która - wydaje mi się - dobrze mi służy to:

Jeśli musisz użyć tego samego kawałka kodu drugi raz, skopiuj go. Jeśli musisz go użyć po raz trzeci - refactoruj w oddzielną funkcję/metodę.

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Dariusz Wawer:

Mateusz Herych:
Programiści często niezbyt chętnie podchodzą do małych klas, które mają jedną, bardzo małą odpowiedzialność - wszak o wiele 'łatwiej' dodać jest prywatną metodę do klasy już istniejącej - takie podejście nie jest jednak do końca dobre, bo często prowadzi do złamania SRP, a i taki kod z wieloma prywatnymi metodami nie jest zbyt łatwo testowalny.

A potem mamy aplikacje, które w powtarzanej tysiące razy pętli alokują dziesiątki mini obiektów. A GC umiera.

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Dariusz Wawer:
Mogłem się spodziewać, że moja wypowiedź zostanie tak odebrana. Wszyscy pewnie macie rację, na dodatek popartą doświadczeniem. Powiedzcie mi jednak, czy nie widzieliście nigdy czegoś takiego:

JakisTamDb dbinfo = getDb();

while (dataAvaialable()){

  data = getData();

  for (DataObj do : data) {

    new DataProcessor(dbinfo, data).processAndAddToDb();

  }

}

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Piotr Jasiulewicz:
Co do tego, ze w PHP pisze sie brzydko, a w Javie ladnie.. to prawda, PHP pozwala na pisanie brzydko. Jest tak samo jak z pistoletem - mozna sobie strzelic w kolano z wlasnego pistoletu, ale zalezy to glownie od strzelca ;)

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Link do wypowiedzi

Temat: Dublowanie kodu - bezwzględnie usuwać, czy może nie?

Link do wypowiedzi