«
»

Uh-oh, this can’t be good

Can this lead to anything good when I’m starting to write lines like this myself?

my $filter = join ‘ and ‘, map { “agr.$_ = $temp.$_” } split(/, /, $fields);

I know, it’s not that scary, and does not involve even a single regexp, but still…

Posted on 2008-08-06 11:21 under random. Comments feed here. Comments are currently closed.

3 Responses to 'Uh-oh, this can’t be good'

  1. David Helgason on 2008-08-06 11:58 :

    Well, strictly speaking /, / is a (simple) regexp :)

    d.

  2. Ignas on 2008-08-06 13:51 :

    agr ?

  3. Fredrik Alstromer on 2008-08-06 15:13 :

    Looks like he’s building an SQL statement (or where clause rather), so I’m guessing agr is a table name.