for my $artistkey (0..$#is_guest) { # Don't do anything if the artist is not a guest next if ($is_guest[$artistkey] == 0); # IF the artist exists, we need only to update some information my $sth = $dbh->prepare("SELECT FROM artists (alias) WHERE alias = +?"); $sth->execute($artist_alias[$artistkey]); # The Part In Question: if ($sth->rows >= 1) { my $sth = $dbh->prepare("UPDATE artists SET email=? WHERE alias= +?"); $sth->execute($email[$artistkey], $artist_alias[$artistkey]); } # Now for Redundancy next if ($sth->rows >= 1); my $sth = $dbh->prepare("INSERT INTO artists (alias, email, is_gues +t) VALUES (?, ?, ?)"); $sth->execute($artist_alias[$artistkey], $email[$artistkey], $is_gu +est[$artistkey]); }
I'd like to avoid that little bit about redundancy. If at all possible, i'd like to consolidate the 'if' statement and the 'next if' into one, while still doing what's in the brackets. In an effort to reduce redundancy, I made the following changes. The following change gave me problems at compile time:
next if ($sth->rows >= 1) { my $sth = $dbh->prepare("UPDATE artists SET email=? WHERE alias= +?"); $sth->execute($email[$artistkey], $artist_alias[$artistkey]); }
Any ideas?
--Coplan
In reply to Test break, but still doing by Coplan
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |