id	summary	reporter	owner	description	type	status	priority	component	resolution	keywords	cc
1081	Предлагаю писать лаконичнее	alx	Denis_N	"Не первый раз уже [source:/base/lib/main.lib.php@154:40-45#L30 встречаю] конструкцию типа

{{{
  if (expression)
  then
     return true;
  else
     return false
  endif
}}}

По форме - правильно, а по сути - изысканное издевательство над читателем. :)

Ну почему нельзя написать просто:

{{{
  return (expression);
}}}

?

А в фрагменте по ссылке еще и четыре вызова `preg_match()` напрашиваются на замену одним:

{{{#!sh
function serialPregMatch($serial) {
  return preg_match('/^([A-Z]?\d{5}|Р\d{6}|[А-Я]\d{4})$/u', $serial);
}
}}}

Предлагаю избегать неоправданно ""длинных"" конструкций с использованием ненужных операторов (как оператор if в примере выше), а использовать только необходимое для выражения мысли - так заложенная в коде мысль будет восприниматься намного легче."	улучшение	new	trivial	БД изделий АДС