Já už nějakou dobu frontend nedělám a klidně si nechám vysvětlit od zkušenějších že moje rady jsou nesmyslné. Začal bych asi tím pojmenováním názvů atributů, funkcí a tříd. Název třídy Parser? Co přesně to parsuje? Názvy atributů filed1 až field8? Za to bych dostal solidní čočku na code review :-) no a uplně nejvíce se mi nelíbí to neustále psaní kódu pro HTTP requesty - tohle chce človek typicky napsat jen na 1 řádek voláním jedné funkce takže je to první věc co bych refaktoroval. Za nás se na škole učila taková pomůcka že pokud se jedna věc vyskytuje na dvou nebo více místech je potřeba se zamyslet protože člověk dělá něco špatně. Console.log by v produkčním kódu neměl co pohledávat takže bych se před commitem ujistil že tam nikde nic takového nemám. Jako napsal jsem jen red-flagy seniorní programátoři tam toho najdrou daleko více...
15 Odpovědí
8038 Zhlédnutí