search

Imdapro / commons-rdf

Mirror of Apache CommonsRDF
Other
0 stars 1 forks source link

Statikus analízis eszköz futtatása és jelzett hibák javítása #2

Closed Imdapro closed 7 years ago

Imdapro commented 7 years ago

A kódon statikus ellenőrzést kell csinálnunk.

Imdapro commented 7 years ago

Készítek egy wiki oldalt: Wiki A SonarQube-t használva futtatok egy vizsgálatot.

TitanillaGats commented 7 years ago

Commit: "made attribute transient"@2017. 04. 19. 14:46:29 Status: OK

Az RDFParser valóban nem szerializálható, így jobb, ha jelezzük a fordítónak is, hogy ne próbálja meg szerializálni adott esetben.

TitanillaGats commented 7 years ago

Commit: "replaced double quotes to simples"@2017. 04. 19. 14:47:53 Status: OK

Különösebben nem zavart, hogy egy karakter nem aposztrófban van, hanem idézőjelek között. Mivel a SonarQube szólt érte, megérte átírni. Érdemben nem változott ettől semmi.

TitanillaGats commented 7 years ago

Commit: "replaced Optional<> nullcheck to isPresent() check"@2017. 04. 19. 14:49:10 Status: BROKEN

Szerintem itt a SonarQube tévedett. Az Option elnevezés is azt sugallja, hogy megengedett az üres Option() is, amit például az empty() függvény is szolgáltat. Így most az üres Option<> nem megengedett, pedig a null vizsgálattal az volt.

@Imdapro meg tudnád nézni, hogy nem néztem-e be valamit? Köszi!

TitanillaGats commented 7 years ago

Commit: "replaced null return to Optional.empty()"@2017. 04. 19. 14:50:20 Status: BROKEN

Szerintem ennél is fennáll, ugyanaz, mint az előzőnél. Esetünkben a null és az Option nem egyenértékű (nyilván), és ezzel logikailag változtattunk a kódon. Nem tudhatjuk, hogy hol számítanak Option-ra, ahol a null hibát okoz.

@kcsombi esetleg rá tudnál nézni? Nem ártana egy másodvélemény. Köszi!

TitanillaGats commented 7 years ago

Commit: "removed unnecessary IllegalArgumentException declarations"@2017. 04. 19. 14:56:18 Status: OK

Szép! A legjobb kód, amit meg sem írtak, szóval támogatom a felesleges (és félrevezető) dolgok kiszedését.

TitanillaGats commented 7 years ago

Commit: "replaced lambdas with method references"@2017. 04. 19. 14:57:52 Status: OK

Ez igazán csak szépítés: lambdák vagy metódus referenciák. Érdemben nem változott semmi.

TitanillaGats commented 7 years ago

Commit: "moved variables to their right places"@2017. 04. 19. 14:59:15 Status: OK

Igen, az osztály-szintű tagváltozók igenis legyenek az osztály elején definiálva.

TitanillaGats commented 7 years ago

Commit: "removed commented-out code"@2017. 04. 19. 15:20:59 Status: OK

Kikommentezett kódnak nincs helye, csak nagyon indokolt esetben és akkor is max rövid távon. Jó, hogy kikerült.

TitanillaGats commented 7 years ago

Commit: "removed unnecessary TODO"@2017. 04. 19. 15:21:58 Status: OK

Nem tudom, hogy egy TODO hogy lehet felesleges. Természeténél fogva azt jelzi, hogy ott valamit még csinálni kell. Így hacsak a SonarQube nem tudja értelmezni a szöveges igényt ami alapján meggyőződött arról, hogy a kódban megtörtént a kért változtatás, aligha dönthet úgy, hogy egy TODO felesleges. Rosszabb nem lett, mondjuk információt vesztettünk.

TitanillaGats commented 7 years ago

Commit: "removed commented-out code #2"@2017. 04. 19. 15:22:40 Status: OK

Nem használt kódnak nincs helye.

TitanillaGats commented 7 years ago

Commit: "removed unnecessary IllegalStateException declaration"@2017. 04. 19. 15:32:59 Status: OK

Korábban már volt egy ilyen commit másik osztályon. Támogatom hogy a nem valós állítás kikerült.

micskeiz commented 7 years ago

OK.

Jó, hogy nem fogadtátok el vakon a Sonar összes jelzését.

Ha végeztetek a feladattal, akkor zárjátok le az issue-t.

Imdapro commented 7 years ago

Kijavítottuk a hibákat, amiket valóban hibának véltünk és ki tudtunk javítani. A kódminőségre már megfelelőt ír a Sonar.