0

Избыточность алгоритма в главе "Разделение запроса и модификатора"

Никита Кузьмин il y a 1 an 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();
    }
}

В данном случае просто избавляемся от повторного вызова метода