Сделал бы вот так:
my $sql = posts_sql_view();
my @where = ();
do { push @where, 'ty.id=?'; push @params, $type } if $type;
do { push @where, 'p.id=?'; push @params, $id; $limit = 0 } if $id;
$sql .= join ' AND ', @where if @where;
$sql .= ' ORDER BY id DESC' unless $id;
$sql .= " LIMIT $limit" if $limit;
On Wednesday, 18 December 2013, 1:14, Nick Knutov <[email protected]> wrote:
У меня в коде в некоторых функциях, в зависимости от входных параметров,
конструируются разные sql запросы. Пример:
my $sql = posts_sql_view();
$sql .= ' WHERE ' if $type or $id or $main;
do { $sql .= ' ty.id=? '; push @params, $type;} if $type;
do { $sql .= ' AND p.id=? LIMIT 1'; push @params, $id } if $id;
$sql .= ' ORDER BY id DESC ' unless $id;
do { $sql .= ' LIMIT ? ' ; push @params, $limit } if $limit and not $id;
Мне не нравится как это визуально выглядит. Как бы вы это переписали,
чтобы было понятнее и читабельнее?
ps: в некоторых других случаях можно было бы использовать && вместо
do{}, но не тут.
pps: ORM предлагать и обсуждать не надо. Как минимум потому, что запросы
сложные и сложность/стоимость проверки того, что генерирует ORM
многократно выше любого профита от ORM в данной ситуации.
--
Best Regards,
Nick Knutov
http://knutov.com
ICQ: 272873706
Voice: +7-904-84-23-130
--
Moscow.pm mailing list
[email protected] | http://moscow.pm.org--
Moscow.pm mailing list
[email protected] | http://moscow.pm.org
