The issue is the other conditional.
If the dog is barking let him in otherwise give him a bone, but only if it is raining.
This sentence reads nicer if we place the "if it is raining" earlier in the sentence. I however, would force the truth of $country
my $country = $card->country || 'gbr'; $country = $country eq 'gbr' ? '' : uc "[$country]";
Unfortunately, this breaks when we need to check definedness (until we have a reliable // operator). In that case (assume "0" is a valid country) I would probably go with something like:
my $country = $card->country; $country = (!defined($country) or $country eq 'gbr') ? '' : uc "[$coun +try]";
Which is just as complex as the original, but the full conditional is easier to find.
Good Day,
Dean
In reply to Re^3: How Much Is Too Much (on one line of code)?
by duelafn
in thread How Much Is Too Much (on one line of code)?
by Ovid
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |