At 3:19 PM +0100 6/13/07, Richard Davey wrote:

 <?php

 $filter['flags'] = 0;

 > if( $allow_fraction )
 {
 >     $filter['flags'] |= FILTER_FLAG_ALLOW_FRACTION;
 }

 > if( $allow_thousand )
 {
 >     $filter['flags'] |= FILTER_FLAG_ALLOW_THOUSAND;
 }

 > if( $allow_scientific )
 {
 >     $filter['flags'] |= FILTER_FLAG_ALLOW_SCIENTIFIC;
 }

?>>

I don't think it's *terribly* verbose, as it has good sentence structure
to it, but your version is certainly more efficient, hence I've
swapped to that. Any other takers? ;)

Rich:

How about?

switch (1)
        {
        case $allow_fraction:
        $filter['flags'] = FILTER_FLAG_ALLOW_FRACTION;
        break;

        case $allow_thousand:
        $filter['flags'] = FILTER_FLAG_ALLOW_THOUSAND;
        break;

        case  $allow_scientific:
        $filter['flags'] = FILTER_FLAG_ALLOW_SCIENTIFIC;
        break;
        }

Would that not work?

Cheers,

tedd
--
-------
http://sperling.com  http://ancientstones.com  http://earthstones.com

--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to