Code Smell 169 — Склеенные методы

TL;DR: старайтесь быть как можно более атомарными в своих методах

Если вы называете метод с помощью «И», вы, вероятно, упускаете возможность метода извлечения и разбиения.

calculatePrimeFactorsRemoveDuplicatesAndPrintThem()
// Three responsibilities

calculatePrimeFactors();
removeDuplicates();
printNumbers();
// Three diferent methods
// We can test them and reuse them

[X] Полуавтоматический

Некоторые линтеры могут предупредить нас о методах, включающих термин «и».

При разработке методов очень важно разыграть какую-нибудь историю с резиновой уткой и сказать себе, все ли мы делаем правильно.

Code Smells — это всего лишь мое мнение.

Изучение искусства программирования, как и большинства других дисциплин, состоит в том, чтобы сначала изучить правила, а затем научиться их нарушать.

Джошуа Блох

Эта статья является частью серии CodeSmell.