Code Smell 319 - Hardcoded Stateless Properties

Wait 5 sec.

Don't turn collaborators into permanent roommates TL;DR: You should avoid storing stateless utility classes as instance variables initialized with new.Problems 😔Hardcoded dependenciesTesting difficultiesHigh couplingHidden side effectsRigid designMisleading intentPremature OptimizationStack clutterSolutions 😃Use dependency injectionPass as parameterUse static methodsInline the logicUse local variablesInline object creationRefactorings ⚙️https://hackernoon.com/refactoring-024-replace-global-variables-with-dependency-injection?embedable=truehttps://hackernoon.com/refactoring-030-how-to-avoid-accidental-redundancy?embedable=truehttps://hackernoon.com/refactoring-007-the-refactor-that-reveals-missing-concepts?embedable=trueContext 💬Hardcoding a stateless class in the constructor creates permanent coupling.Even if the class is cheap to instantiate, you lose the ability to swap it.Stateless objects shouldn't be part of the object's internal state.You confuse readers by making a tool look essential to the object's identity.It makes testing harder because you can't mock the hardcoded dependency.Sample Code 💻Wrong 🚫class UserProcessor { private provider: MockDataProvider; constructor() { // You hardcode the dependency here. // This makes the class harder to test. this.provider = new MockDataProvider(); } process(data: any) { return this.provider.format(data); }}Right 👉interface DataProvider { format(data: any): any;}class UserProcessor { // You inject the dependency via constructor. // Now you can swap it or mock it easily. constructor(private readonly provider: DataProvider) {} process(data: any) { return this.provider.format(data); }}// Simpler but coupled class UserProcessor { constructor() { // Empty } process(data: any) { return new MockDataProvider().format(data); } }Detection 🔍Look for the new keyword inside constructors.Watch for private properties instantiated directly in the constructor rather than passed as parameters.Most linters flag this pattern automatically when you create instances and assign them to private fields.Tags 🏷️Premature OptimizationLevel 🔋[X] BeginnerWhy the Bijection Is Important 🗺️Software should mimic a MAPPER of the real world.In reality, a worker might use a tool to complete a task.The tool is not a permanent physical attachment to the worker.When you refactor to use dependency injection, you respect the bijection by treating collaborators as external entities, not internal state.This keeps your simulation flexible and accurate.AI Generation 🤖AI generators frequently create this smell.They often suggest code that just works by instancing dependencies directly in the constructor to save time.AI Detection 🧲AI can easily detect this smell without explicit instructions.When you show AI a class with new keywords in the constructor, it recognizes the pattern as hardcoded coupling.AI identifies that stateless utility classes should be injected rather than instantiated internally.The detection is straightforward because the pattern is syntactically obvious and semantically harmful.Try Them! 🛠Remember: AI Assistants make lots of mistakes Suggested Prompt: remove the cached attributeWithout Proper Instructions 📵ChatGPTClaudePerplexityCopilotYouGeminiDeepSeekMeta AIGrokQwenWith Specific Instructions 👩‍🏫ChatGPTClaudePerplexityCopilotYouGeminiDeepSeekMeta AIGrokQwenConclusion 🏁Storing stateless dependencies as instance variables makes your code rigid.When you inject these dependencies instead, you improve testability and keep your objects focused on their true purpose.Relations 👩‍❤️‍💋‍👨https://hackernoon.com/code-smell-06-trying-to-be-a-clever-programmer?embedable=truehttps://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-iv-7sc3w8n?embedable=trueMore Information 📕https://hackernoon.com/coupling-the-one-and-only-software-designing-problem-9z5a321h?embedable=trueDisclaimer 📘Code Smells are my opinion.Credits 🙏Photo by Possessed Photography on Unsplash Coupling is the enemy of changeRich Hickeyhttps://hackernoon.com/400-thought-provoking-software-engineering-quotes?embedable=trueThis article is part of the CodeSmell Series.https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-i-xqz3evd?embedable=true\