Есть сервер, в котором вот такой код (псевдо-код)
if(someCondition or foo.regexMatches("pattern1|pattern2|pattern3"))
then (expression; return)
Я подумал, что если разнести его вот так
if(someCondition) then (expression; return)
if(foo.regexMatches("pattern1")) then (expression; return)
if(foo.regexMatches("pattern2")) then (expression; return)
if(foo.regexMatches("pattern3")) then (expression; return)
, то производительность увеличится, поскольку, если someCondition == true, то дальнешйие проверки не понадобяться. И дальше то же самое.
Мне это все не нравиться, ибо экономия на спичках, а компактность и логичность кода теряется. Но, с другой стороны, этот код будет выполняться при каждом запросе (хотя, с третьей стороны, все запросы обрабатываются асинхронно)
Стоит ли тут ваще думать в этом случае о мелких оптимизациях? Че то не хватает опыта, чтобы оценить, насколько это критично, например, при высоких нагрузках?