Control Couple

Introduction

Control coupling occurs when a method or block checks the value of a parameter in order to decide which execution path to take. The offending parameter is often called a Control Couple.

Control Coupling is a kind of duplication, because the calling method already knows which path should be taken.

Control Coupling reduces the code's flexibility by creating a dependency between the caller and callee: any change to the possible values of the controlling parameter must be reflected on both sides of the call. A Control Couple also reveals a loss of simplicity: the called method probably has more than one responsibility, because it includes at least two different code paths.

You can find a good write-up regarding this problem here.

Current Support in reek

reek warns about control coupling when:

  • Control-Parameter - a method parameter or block parameter is the tested value in a conditional statement (as in the example below); or
  • Boolean-Parameter - a method parameter is defaulted to true or false.

Configuration

Control Couple supports the Basic Smell Options.