Привет.

Зачем ты навставлял проверок такого вида:

if (String.IsNullOrEmpty(filePath))
{
    return false;
}
...
            if (request == null)
{
    return;
}

?

Там действительно приходил null и пустая строка? Если, да, то неужели это нормальное поведение этих методов?

Если на оба последних вопроса ответ отрицательный, и этот код создан для чистоты, так сказать, то будь добр, замени его на генерацию исключений ArgumentNullException/ArgumentException.

Сразу объясняю почему так нельзя делать. Рано или поздно кто-то может ошибиться и случаяно передать в такой метод null или пустую строку. Наличие исключения или даже просто отсуствие проверки параметров приведет к тому, что мы увидим эту ошибку. А вот возврат null-ов и т.п. приведет к появлению исключений в совсем дургм месте или просто замажет ошибку (будет некорректное поведение о котором никто не будет знать).

ЗЫ

Да, ив if-ов с одним выражением внутри не стоит ставить скобки. Они соврешенно лишние. Только место занимают. Отбивки достаточно.