C
C#2y ago
KidKai25

✅ What's your view on tool like Sonarqube?

Should I learn from the code smells it captures?
29 Replies
Unknown User
Unknown User2y ago
Message Not Public
Sign In & Join Server To View
KidKai25
KidKai25OP2y ago
like it telling me about reducing "Cyclic Complexity"
Unknown User
Unknown User2y ago
Message Not Public
Sign In & Join Server To View
KidKai25
KidKai25OP2y ago
so I become better?
Unknown User
Unknown User2y ago
Message Not Public
Sign In & Join Server To View
KidKai25
KidKai25OP2y ago
right
Unknown User
Unknown User2y ago
Message Not Public
Sign In & Join Server To View
KidKai25
KidKai25OP2y ago
code smell is subjective, right?
Unknown User
Unknown User2y ago
Message Not Public
Sign In & Join Server To View
MODiX
MODiX2y ago
KidKai25#7728
like it telling me about reducing "Cyclic Complexity"
Quoted by
React with ❌ to remove this embed.
Unknown User
Unknown User2y ago
Message Not Public
Sign In & Join Server To View
KidKai25
KidKai25OP2y ago
You use it?
Unknown User
Unknown User2y ago
Message Not Public
Sign In & Join Server To View
KidKai25
KidKai25OP2y ago
I think should I get a sonarqube linter
Unknown User
Unknown User2y ago
Message Not Public
Sign In & Join Server To View
KidKai25
KidKai25OP2y ago
so you will use it?
Unknown User
Unknown User2y ago
Message Not Public
Sign In & Join Server To View
KidKai25
KidKai25OP2y ago
I thought there might be there
Unknown User
Unknown User2y ago
Message Not Public
Sign In & Join Server To View
KidKai25
KidKai25OP2y ago
sorry it's sonarlint
Unknown User
Unknown User2y ago
Message Not Public
Sign In & Join Server To View
Becquerel
Becquerel2y ago
i think metrics like cyclic complexity can be useful hints but they're inherently prone to false-positives sometimes you have a good, maintainable function that's complex to computers but not humans switch cases kill cyclic complexity but are usually ok to understand for humans. though they have other problems i think using tools like that is good for highlighting specific code problems (use this language feature instead of that one, etc.) but for broad metrics stuff just use them as a general guideline for areas that need attention, don't take them religiously
LPeter1997
LPeter19972y ago
I fell into the trap of listening to SonarQube suggestions/reports In the end, became way less productive and code quality didn't improve at all because I tried to make arbitrary metrics happy
Unknown User
Unknown User2y ago
Message Not Public
Sign In & Join Server To View
LPeter1997
LPeter19972y ago
So you can def. look at what it says, it might has some merit to stuff. But take it with a pinch of salt, don't handle it as an oracle
Unknown User
Unknown User2y ago
Message Not Public
Sign In & Join Server To View
Auger
Auger2y ago
Cognitive complexity is also a useful measurement as well, that I think they came up with
Accord
Accord2y ago
Was this issue resolved? If so, run /close - otherwise I will mark this as stale and this post will be archived until there is new activity.
KidKai25
KidKai25OP2y ago
Hey I hope you are taking care of yourself!
Want results from more Discord servers?
Add your server