on Mon, 12 Aug 2002 13:45:21 GMT, [EMAIL PROTECTED] (Sudarshan
Raghavan) wrote: 

> Instead of these multiple conditions why not create a hash of files 
> that cannot be  checkedout. Something like,
> my %cannot_co = ( comEPRHelp => 1, ....);
> 
> After this your condition will be
> if (($check_out == 0) && (exists ($cannot_co{$var5}))) { 
>     die "...." 
> } 
> 
> Makes it more readable and a lot easier to maintain.

If you initialize your hash-values with '1', you don't need the 
'exists', which makes the statement even more readable:

    if (!$check_out && $cannot_co{$var5}) { 
        die "...";
    }

-- 
felix

-- 
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to