+1 Savenkova Natalya [email protected] https://www.facebook.com/name.rnd
2013/12/18 ksvs <[email protected]> > Сделал бы вот так: > > 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 > >
-- Moscow.pm mailing list [email protected] | http://moscow.pm.org
