Refaktoryzacja kodu - szachy

pytanie zadane 12 lipca 2020 w JavaScript przez Arcywojak Początkujący (370 p.)

Witam, jakiś czas temu napisałem szachy w javascripcie. Kod, który napisałem jest moim zdaniem bardzo zagmatwany i skomplikowany. Chciałbym ten kod zrefaktoryzować, aby był bardziej czytelny dla innych i bardziej skalowalny. Niestety nie mam pomysłu od czego zacząć. Na razie nie chcę bardziej rozwijać tego projektu, a tylko uporządkować kod.

Link do szachów: https://github.com/Arcywojak/Chess

1 odpowiedź

odpowiedź 12 lipca 2020 przez Tomek Sochacki Ekspert (227,490 p.)

Zanim zaczniesz robić refaktor to pokryj kod testami, to Ci zapewni większe bezpieczenstwo, ja gdy mam zrefaktoryzowac jakis kawalek kodu to zawsze najpierw analizuje pokrycie testami ut, integracyjnymi i e2e aby móc potem faktycznie bezpiecznie robic refaktor. Dodaj testy e2e chociaz na podstawowe happy path i kilka.corner case j wtedy wez.sie za refactor.

komentarz 12 lipca 2020 przez Ehlert Ekspert (213,900 p.)

Ja się z tym nie zgadzam. Testy, a szczególnie te wykraczające poza warstwę jednostkową, często przed refactorem są ciężkie do utrzymywania. Mam świadomość że taka praktyka jest stosowana, ale w systemach w których należy zapewnić 100% niezmienności działania przy refactorze.

komentarz 12 lipca 2020 przez Tomek Sochacki Ekspert (227,490 p.)

Ale refaktor to właśnie zachowanie niezmiennosci zachowania biznesowego i to warto otestowac aby mieć pewność że nic nie popsules. Refaktor to nie zmiana logikj biznesowej, nie mylmy tych rzeczy. Miałem okazje robic sporo refaktorow czy nawet migracji apek np. Vanillajs do react jak zaczynały się rozrastac i wolę naprawdę poświęcić np. 1-2 dni na analize testow e2e i dodanie ewentualnych testow dla kluczowych procesow biznesowych niz potem co chwile recznie klikac całą apke... serio jestes zwolennikiem manualnego klikania milion razy czegos, co moze zrobić np. Cypress, testcafe itp.? Ja tam raczej wolę automatuzacje :) potem gdy siadasz do apki jaką robił ktoś inny to takie testy tez naprawde potrafia pomoc i czasami wykryc problemy... ale oczywiście dużo zależy od podejscia do pisania apki... ja staram sie patrzeć nie tylko na krotkotrwaly development ale tez na dalsze utrzymanie apki i nowe ficzery... dlatego u mnie w zespole zawsze przy ficzerach wyznaczamy sobie czas na odpowiednie testy reczne i dodanie odpowiednich automatow. To się naprawdę oplaca.

komentarz 12 lipca 2020 przez Arcywojak Początkujący (370 p.)

@Tomek Sochacki, a czy w sytuacji, gdy chciałbym później zmienić strukturę, rozmieszczenie lub kolejność wywołynia poszczególnych funkcji to czy testy wcześniej napisane nie stałyby się bezużyteczne i trzeba by było napisać je od nowa?