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
