Simple PHP Question

[LPL]

<select class="input" name="day">
  <?php
    $day = 1;
    while ($day <= 31) {
    echo "<option value=" . $day . ">" . $day . "</option>";
    $day = $day + 1;
    }
  ?>
</select>

How do I set a default value for the option?
ie when the page loads it doesn't start at 1 but whatever number I set.

I tried option selected or value selected and both just set the default displayed number to 31

 

[marc2003]

selected="selected"

is the html. so you just need to implement that in your php.

echo '<option value="' . $day . '"';
if($day == 5) echo ' selected="selected"'; //edit this to whatever you want
echo '>' . $day . '</option>';

 

[LPL]

I want the counter to go from 1-31 and then have the default value of something from a database.

I tried the above last night and still it defaults to 31

 

[marc2003]

works fine here....

http://www.marc2003.ukfsn.org/test.php

full code....

<?php
echo '<select name="day">';
$day = 1;
while ($day <= 31) {
	echo '<option value="' . $day . '"';
	if($day == 5) echo ' selected="selected"'; //edit this to whatever you want
	echo '>' . $day . '</option>';
	$day++;
}
echo '</select>';
?>

 

[LPL]

Cheers I misunderstood the first time round got it now though lol

 

[nav...]

why are you not using a for loop???

<select name="day">
<?php
for($i = 1; $i <= 31; $i++)
    echo '<option value="'.$day.'"'.($day == 5 ? ' selected="selected"' : '').'>'.$day.'</option>';
?>
</select>

 

[Izi]

why are you not using a for loop???

<select name="day">
<?php
for($i = 1; $i <= 31; $i++)
    echo '<option value="'.$day.'"'.($day == 5 ? ' selected="selected"' : '').'>'.$day.'</option>';
?>
</select>

does it matter? both get same result. it sounds like he is a novice and for loops take a little bit more understanding

 

[nav...]

a for loop is hardly rocket science.

understanding a for loop will lead to understanding a foreach loop.

using less code across a project means that it will be easier to see what is doing what.

Not sure that being a novice means you can't learn stuff or use good practice.

 

[Dj_Jestar]

it's personal preference, and is nit picking.

foreach (range(1, 31) as $day) {
  echo '<option value="'.$day.'"'.($day == 5 ? ' selected="selected"' : '').'>'.$day.'</option>';
}

 

[adam83]

a for loop is hardly rocket science.

understanding a for loop will lead to understanding a foreach loop.

using less code across a project means that it will be easier to see what is doing what.

Not sure that being a novice means you can't learn stuff or use good practice.

agreed.

 

[Cuchulain]

a for loop is hardly rocket science.

understanding a for loop will lead to understanding a foreach loop.

using less code across a project means that it will be easier to see what is doing what.

Not sure that being a novice means you can't learn stuff or use good practice.

There's teaching a novice something and coming across like you're trying to belittle someone with less knowledge than yourself.

 

[nav...]

not sure where in my posts I belittled anyone. I just merely suggested a for loop and posted the accompanying code.

A for loop to me is the best way to do it, with the least amount of code.

Maybe if I'd posted in lolspeak and thrown the term n00b about then I'd understand the criticism.