Re: <OPTION

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Robert Cummings wrote:
> On Wed, 2006-10-25 at 17:35 -0700, Paul Novitski wrote:
>> At 10/25/2006 04:09 PM, Stut wrote:
>>> Dang that's painful!! Try this...
>>>
>>> <?php
>>>     foreach (range(1, 31) as $day)
>>>     {
>>>         print '<option value="'.$day.'"';
>>>         if ($selected_day_of_month == $day)
>>>             print ' selected';
>>>         print '>'.$day.'</option>';
>>>     }
>>>
>>> ?>
>>
>> Ouch!  Gnarly mix of logic and markup.  I suggest something more like:
>>
>>          foreach (range(1, 31) as $day)
>>          {
>>                  $sSelected = ($selected_day_of_month == $day) ? ' 
>> selected="selected"' : '';
>>
>>                  print <<< hdDay
>>                  <option value="$day"$sSelected>$day</option>
>>
>> hdDay;
>>          }
> 
> Ewww, I'll take Stut's style anyday. Heredoc has its uses, but I
> wouldn't consider your above usage one of them :/ Now to add my own
> flavour...
> 
> <?php
> 
>     for( $day = 1; $day <= 31; $day++ )
>     {
>         $selected
>             = $selected_day_of_month == $day
>             ? ' selected="selected"'
>             : '';
> 
>         echo '<option value="'.$day.'"'.$selected.'>'
>             .$day
>             .'</option>';
>     }
> 
> ?>

bunch of space wasters ;-)

<?php foreach (range(1, 31) as $d) echo '<option value="',$d,'"',($d = $selDay?'
selected="selected"':''),'>',$d,'</option>'; ?>


> 
> Cheers,
> Rob.

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


[Index of Archives]     [PHP Home]     [Apache Users]     [PHP on Windows]     [Kernel Newbies]     [PHP Install]     [PHP Classes]     [Pear]     [Postgresql]     [Postgresql PHP]     [PHP on Windows]     [PHP Database Programming]     [PHP SOAP]

  Powered by Linux