0
Избыточность алгоритма в главе "Разделение запроса и модификатора"
ИМХО, но данный рефакторинг не самый оптимальный. Возможно, это не относится к главе, но если здесь речь идёт о выделении метода DoSendAlert(), то стоит написать его оптимальнее.
public void CheckSecurity(string[] people) { string found = FindCriminal(people); DoSendAlert(found); SomeLaterCode(found); } public void DoSendAlert(string criminal) { if (!String.IsNullOrEmpty(criminal)) { SendAlert(); } }
В данном случае просто избавляемся от повторного вызова метода
Customer support service by UserEcho