error: end tag for "select" is not finished

varai

Hello

I validated the code below in http://validator.w3.org/
It says
end tag for "select" which is not finished
</select>✉
Most likely, you nested tags and closed them in the wrong order. For example <p><em>...</p> is not acceptable, as <em> must be closed before <p>. Acceptable nesting is: <p><em>...</em></p>

Another possibility is that you used an element which requires a child element that you did not include. Hence the parent element is "not finished", not complete. For instance, in HTML the <head> element must contain a <title> child element, lists require appropriate list items (<ul> and <ol> require <li>; <dl> requires <dt> and <dd>), and so on.

I have no idea on what is wrong coz' I have used similar codes in other pages with no prob. The only difference is here i'm using for loop and in the other pages i'm using while loop. But I think that shouldn't matter.

Can anyone please please help me? 🙁 I've spent hours on this.
The code in red is where the error occurs.

<form name="cr_card_disp2_frm" action="confirm.php" method="post">

<div class='form_label1' style='left:7%;'>Payment</div>
<div class='form_element' style='right:40%; vertical-align:middle;'>
Expiration Date:
<select name="exp_mth">
<?php 
for ($mth=1; $mth<=12; $mth++)
  {if ($mth < 10)
       echo "<option>"."0".$mth."</option>";
    else
         echo "<option>".$mth."</option>";
   }
?>
[COLOR="Red"]</select>[/COLOR]
<input type="submit" value="Continue Checkout" style="position:absolute;top:0;right:-15%;" />
</div>
</form>

thanks a lot

bradgrafelman

Show us (or give us a link to) the resulting HTML of that page.

Also, note that this:

for ($mth=1; $mth<=12; $mth++)
  {if ($mth < 10)
       echo "<option>"."0".$mth."</option>";
    else
         echo "<option>".$mth."</option>";
   }

could be reduced to this:

for ($mth=1; $mth<=12; $mth++)
  printf("<option>&#37;02u</option>\n", $mth);

varai

Hello bradgrafelman

I haven't uploaded this website yet. So,how do I show you the output of that page?

thank you.

varai

Hello bradgrafelman

I changed it to

for ($mth=1; $mth<=12; $mth++)
  printf("<option>%02u</option>\n", $mth);

but still i'm getting the same error in validator.w3.org.

When I display the page in the browser it says error on page and the form doesn't get displayed.

however i'm not getting any error for </select> in the following simple html code (that doesn't involve any php)

<select name='cr_card'>
 		 <option>Visa</option>
		 <option>Master</option>
		 <option>Discover</option>
</select>

I haven't uploaded this page to the net so how can I show u the output of the page?

thank you.

johanafm

How have you been able to run your html output through a validator if you can't do it before you take your code live? In case you actually have been using your production server to test new code, I recommend setting up an environment for development and testing.

Just install a webserver and PHP on the machine where you write your code so that you can test it before you deploy it. Ideally, your dev and/or test environment(s) should match your live environment, but even if it doesn't, it's a lot better than having no such possibilities.

varai

Hi,

I do have a development and testing environment on my laptop. I had installed WAMP Server. I am able to see the output on the browser with WAMP.

In W3 validator there are 3 tabs and I choose the 3rd one: validate by direct input. I just had to copy and paste the code and click on the check button.

thanks

varai

bradgrafelman;10960862 wrote:
Show us (or give us a link to) the resulting HTML of that page.

Also, note that this:
for ($mth=1; $mth<=12; $mth++)
  {if ($mth < 10)
       echo "<option>"."0".$mth."</option>";
    else
         echo "<option>".$mth."</option>";
   }
could be reduced to this:
for ($mth=1; $mth<=12; $mth++)
  printf("<option>%02u</option>\n", $mth);

bradgrafelman, can you please explain on what this regular expression means? %02u?

thank you.

johanafm

It's not a regular expression, it's a format specifier for printf. See sprintf for info on these. Looks to be a 2 digit width, unsigned integer, left padded with 0 if necessary.

varai

thanks a lot. 🙂 I checked out the link for sprintf function. Those formattings are very useful indeed.