Can anyone suggest a nicer way to do this? I particularly dislike the use of the $success flag. In my real code, I have many conditions to check and have to put the flag setting code and the "last" statement after each one, which feels redundant.my $success = 0; foreach my $try (@options) { if (exists $hash{$try}) { do_something($try); $success = 1; last; } } if (!$success) { log_failure(); }
In reply to improve ugly flow control by perrin
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |