this post was submitted on 06 Jul 2023

16 points (100.0% liked)

Programming

13375 readers

9 users here now

All things programming and coding related. Subcommunity of Technology.

This community's icon was made by Aaron Schneider, under the CC-BY-NC-SA 4.0 license.

founded 1 year ago

MODERATORS

alyaza@beehaw.org

Hexorg@beehaw.org

remington@beehaw.org

16

Refactoring in a nutshell (webcomicname.com)

submitted 1 year ago by jmanes@beehaw.org to c/programming@beehaw.org

2 comments fedilink hide all child comments

you are viewing a single comment's thread
view the rest of the comments

[–] benark@beehaw.org 2 points 1 year ago (1 children)

Refactoring, without test coverage, in a nutshell.

Definitely adding this one to the dev guidelines on unit and integration tests. 👍🏼

[–] MagicShel@programming.dev 2 points 1 year ago

I don't think I've met a codebase that had tests that would work with the level of refactoring that would need to be done. I mean let's be honest here, if it was well architected, refactoring wouldn’t be like the comic. It’s probably a mess of entangled concerns and bad inheritance.

For that matter the code most in need of refactoring probably has shit tests and that just assertNotNull on the happy path to appease sonar and be done with it.