Оператор if - Признак плохого кода #1

Если у тебя в коде есть if оператор, который в зависимости от условия выполняет разные действия:

void func() {
	If (type == 1) {
		// логика функции #1
		 . . . 
		 . . . 
	}
	If (type == 2) {
		 // логика функции #2
		 . . . 
		 . . . 
	}
}

Подумай, а действительно ли код построен верно? Если в твоем классе есть более одного случая, где логика кода зависит от магической переменной type, то вероятность плохого дизайна увеличивается. 

В классах не должно быть переменной, которая будет влиять на логику выполнения методов. Скорей всего нужно создать два типа класса, которые могут иметь один общий базовый. 

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

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

---

Понравилось? Кликни Лайк, чтобы я знал, какой контент более интересен читателям. Заметку пока еще никто не лайкал и ты можешь быть первым

---