Skip to main content

You are not logged in. Your edit will be placed in a queue until it is peer reviewed.

We welcome edits that make the post easier to understand and more valuable for readers. Because community members review edits, please try to make the post substantially better than how you found it, for example, by fixing grammar or adding additional resources and hyperlinks.

Required fields*

3
  • 1
    I guess I'm just trying to figure out where the line is. In a recent task, I had to perform a fairly simple operation, but it was in a codebase without much existing scaffolding or functionality. As such, everything I needed to do was very simple, but all pretty unique and didn't seem to fit in shared classes. In my case, I needed to save a document to a network drive, and log it to two separate database tables. The rules surrounding each step were fairly particular. Even the filename generation (a simple guid) had a few classes to make testing more convenient. Commented Jul 9, 2019 at 14:14
  • 3
    Again, @JDDavis, choosing multiple classes over a single one purely for testability purposes is putting the cart before the horse, and it goes directly against the SRP, which calls for grouping cohesive functionalities together. I can't advise you on particulars, but the problem that individual functional changes require modifying many files is an issue that you should be addressing (and attempting to avoid), not one that you should be trying to justify. Commented Jul 9, 2019 at 14:23
  • Agreeing, I add this. To quote Wikipedia, "Martin defines a responsibility as a reason to change, and concludes that a class or module should have one, and only one, reason to be changed (i.e. rewritten)." and "he more recently stated "This principle is about people."" In fact, I believe this means that the "responsibility" in SRP refers to stakeholders, not functionality. A class should be responsible for changes required by only one stakeholder (person requiring that you change your program), so that you change as FEW things as possible in response to different stakeholders demanding change. Commented Jul 11, 2019 at 19:49