Problems

• Coupling
• Break encapsulation

Solutions

• Create intermediate methods.
• Think about Law of Demeter.
• Create higher level messages.

Sample Code

Wrong

class Dog {
	   constructor(feet) {
	     this.feet = feet;    
	  }
	  getFeet() {    
	    return this.feet;
	  }  
	}

	class Foot {
	  move(){    
	    //..
	  }
	}

	feet = [new Foot(), new Foot(), new Foot(), new Foot()];
	dog = new Dog(feet);

	for (var foot of dog.getFeet()) {
	  foot.move(); 
	}

Right

class Dog {
	   constructor(feet) {
	     this.feet = feet;    
	  }
	  walk(){
	    for (var foot of this.feet) {
	      foot.move(); 
	    }
	  }
	}

	class Foot {
	  move(){    
	    //..
	  }
	}

	feet = [new Foot(), new Foot(), new Foot(), new Foot()];
	dog = new Dog(feet);
	dog.walk();

Detection

Automatic detection is possible using parsing trees.

Also Known as

• Law of Demeter

Tags

• Declarative
• Encapsulation

Conclusion

Avoid successive message calls. Try to hide the intermediate collaborations and create new protocols.

#clean-code #software-development #object-oriented #code-smells #programming