Det kunde vara värt att kolla på om det finns något bra verktyg för att testa komplexitet automatiskt. Jag har ingen förhoppning om att det ska fånga allt perfekt, men det skulle kunna vara bra som ett steg innan manuell granskning. Stötte t.ex. på Code Climate som gör bl.a. detta och är "Free for open-source.".
Ämne på Diskussion:Developer guidelines
Hoppa till navigering
Hoppa till sök
Jag har använt Code Climate för vissa projekt (hostade på Github). Det jag tyckte att den var väldigt bra på var att flagga nästan-duplicerad kod vilket ger en indikation på var refaktorering kan vara bra. Jag vill minnas att det var lite strul med att köra över default-inställningarna men annars är det inte ett dåligt verktyg att ha i lådan. Går även att integrera med webhooks i github så att man får feedback innan man mergar en PR.
Kollade som hastigast och det ser ut som att en del saker där kan vara användbara. Främst tycker jag att varningar för duplicering (som du nämner ovan) och cognitive complexity ser intressanta ut. Hårda gränser för antalet rader i funktioner vet jag inte om de är lika användbara, men kan funka som varningsflaggor.
Jo jag reagerade också på att långa filer/funktioner får så pass stränga straff.
Ska bli intressant hur exemplet ovan påverkas av refactor-js patchen.
Jag får inte "Trends" att fungera. Det är bara en laddningssnurra, oavsett vilken typ jag väljer.
Då är det nog något i din webbläsare:
- https://screenshots.firefox.com/z6zHo0GylGe6dxIA/codeclimate.com
- https://screenshots.firefox.com/ALPWjJvBp0e2noqA/codeclimate.com
- https://screenshots.firefox.com/8euAd8TFGeAC27GD/codeclimate.com
OBS skärmbilderna raderas automatiskt 2018-04-27