Checki mają mało sensu w praktycznych projektach.
Hej oglądałem twojego live, postanowiłem sprawdzić devSittera na moim głównym projekcie https://github.com/Rollczi/LiteCommands (30k liniek~, 668 plików). Z bugów to: - nie widać generyków typu MyClass prawdopodobnie zamieniają się w znaczniki html. strona: src: strona (html): dostałem tak dużo issues (1278) że nie da się tego przeglądać a strona zaczyna się bagować: No i może napiszę jakie są odczucia w praktyce… ogólnie rzecz biorąc 95% tego contentu wygląda jak warningi z sonarlinta co jest trochę bez sensu bo łatwiej już takie rzeczy analizować i poprawiać w IDE. Szczerze lubię używać lepiej nazwanych typów generycznych które coś faktycznie znaczą niż T, a komunikaty typu: Rename this generic name to match the regular expression '^[A-Z][0-9]?$'. Mogły by się stackować, lub można było by je wyłączyć. Tutaj kolejny przykład: Im dłużej widzę te analizy tym bardziej widać że jest to po prostu to samo co ma wtyczka do IDE SonarLint. Jako developer oczekiwałbym bardziej profesjonalnej analizy kodu, a nie checków jego wyglądu, które nie zawsze wpasowują się w kontekst. Tylko parę uwag jest trafnych (w praktyce), całe 1200 można wywalić do kosza (Importy, TODO, Nazwy generyków, method reference, 3x ten sam string w jednej klasie, Deprecated, Catch Exception instead of Throwable, Methods should not be empty itd.) Za chwilę ktoś mi może zarzucić “no dobra checkstyle to jedno ale jak to tak łamać konwencje i dobre praktyki”, prawda jest taka, że nie zawsze chcesz usunąć metodę z API nawet jak jest Deprecated, nie zawsze masz możliwość dokończyć TODO, bo ktoś inny ciebie blokuje, nie zawsze czytelnym będzie robienie constanta, nie zawsze metoda musi mieć content (np. u mnie w teście musi być pusta), nie zawsze chcesz łapać konkretny wyjątek (w moim case LiteCommands calluje kod user który może mi wybuchnąć tak o i muszę wszystko łapać) i nie zawsze chcesz stosować się do ogólnoprzyjętego code style. Same podpowiedzi są dobrymi praktykami ale to może pomóc w projekcie juniora co jeszcze nie pisze zgrabnego kodu. Żeby też nie było mój code style nie odbiega jakoś bardzo od tego standardowego ale to generuje ogromne ilości niepotrzebnego issues. Jako profesjonalna analiza mogło by być to np. sprawdzanie przepływu zależności między klasami (może jakieś ustawienie na platformie żeby zaznaczyć że bazujesz na heksagonalnej architekturze etc?) i łapanie złych praktyk typu w entity siedzi jakiś kod od usecase. Tutaj AI mogło by bardzo pomóc.
Norbert Dejlich over 1 year ago
Checki mają mało sensu w praktycznych projektach.
Hej oglądałem twojego live, postanowiłem sprawdzić devSittera na moim głównym projekcie https://github.com/Rollczi/LiteCommands (30k liniek~, 668 plików). Z bugów to: - nie widać generyków typu MyClass prawdopodobnie zamieniają się w znaczniki html. strona: src: strona (html): dostałem tak dużo issues (1278) że nie da się tego przeglądać a strona zaczyna się bagować: No i może napiszę jakie są odczucia w praktyce… ogólnie rzecz biorąc 95% tego contentu wygląda jak warningi z sonarlinta co jest trochę bez sensu bo łatwiej już takie rzeczy analizować i poprawiać w IDE. Szczerze lubię używać lepiej nazwanych typów generycznych które coś faktycznie znaczą niż T, a komunikaty typu: Rename this generic name to match the regular expression '^[A-Z][0-9]?$'. Mogły by się stackować, lub można było by je wyłączyć. Tutaj kolejny przykład: Im dłużej widzę te analizy tym bardziej widać że jest to po prostu to samo co ma wtyczka do IDE SonarLint. Jako developer oczekiwałbym bardziej profesjonalnej analizy kodu, a nie checków jego wyglądu, które nie zawsze wpasowują się w kontekst. Tylko parę uwag jest trafnych (w praktyce), całe 1200 można wywalić do kosza (Importy, TODO, Nazwy generyków, method reference, 3x ten sam string w jednej klasie, Deprecated, Catch Exception instead of Throwable, Methods should not be empty itd.) Za chwilę ktoś mi może zarzucić “no dobra checkstyle to jedno ale jak to tak łamać konwencje i dobre praktyki”, prawda jest taka, że nie zawsze chcesz usunąć metodę z API nawet jak jest Deprecated, nie zawsze masz możliwość dokończyć TODO, bo ktoś inny ciebie blokuje, nie zawsze czytelnym będzie robienie constanta, nie zawsze metoda musi mieć content (np. u mnie w teście musi być pusta), nie zawsze chcesz łapać konkretny wyjątek (w moim case LiteCommands calluje kod user który może mi wybuchnąć tak o i muszę wszystko łapać) i nie zawsze chcesz stosować się do ogólnoprzyjętego code style. Same podpowiedzi są dobrymi praktykami ale to może pomóc w projekcie juniora co jeszcze nie pisze zgrabnego kodu. Żeby też nie było mój code style nie odbiega jakoś bardzo od tego standardowego ale to generuje ogromne ilości niepotrzebnego issues. Jako profesjonalna analiza mogło by być to np. sprawdzanie przepływu zależności między klasami (może jakieś ustawienie na platformie żeby zaznaczyć że bazujesz na heksagonalnej architekturze etc?) i łapanie złych praktyk typu w entity siedzi jakiś kod od usecase. Tutaj AI mogło by bardzo pomóc.
Norbert Dejlich over 1 year ago
Completed
Registration problem
After clicking on "Link to e-mail address verification" got web page with address https://devsitter.app/dashboard?continue and content: Whitelabel Error Page This application has no explicit mapping for /error, so you are seeing this as a fallback. Fri May 24 20:06:43 UTC 2024 There was an unexpected error (type=Not Found, status=404). - - - The email address has been successfully verified. Web browser: Brave, Version 1.66.113 Chromium: 125.0.6422.76 (Official Build) (64-bit) Few adblockers enabled.
Dawid Bielecki about 2 years ago
Completed
Registration problem
After clicking on "Link to e-mail address verification" got web page with address https://devsitter.app/dashboard?continue and content: Whitelabel Error Page This application has no explicit mapping for /error, so you are seeing this as a fallback. Fri May 24 20:06:43 UTC 2024 There was an unexpected error (type=Not Found, status=404). - - - The email address has been successfully verified. Web browser: Brave, Version 1.66.113 Chromium: 125.0.6422.76 (Official Build) (64-bit) Few adblockers enabled.
Dawid Bielecki about 2 years ago
Drag and Drop Source Files
Add functionality allowing direct dragging of source code files into the browser window for review.
Przemek Bykowski about 2 years ago
Drag and Drop Source Files
Add functionality allowing direct dragging of source code files into the browser window for review.
Przemek Bykowski about 2 years ago