Problem populating a radio button

[jimdavidson]

On my update page I'm tring to display the value of a radio button that is stored in a table. The value stored is "yes" When I load the form the radio button field doesn't highlight either button. Can someone tell me what I'm doing wrong? Here's the code, I know it's probably sloppy, but I 'm new at this and any help will be appreciated.

<td><div align="left">
<input 
<?php 
   if (!$_POST && !(strcmp($row_GetMember['county_relief'],"yes"))) {
     echo "checked=\"checked\"";
   }
  elseif ($_POST && !(strcmp($_POST['county_group'],"yes"))){
     echo "checked=\"checked\"";
  } 
?>
name="county_group" type="radio" class="text_background" value="yes" id="county_yes"/>
                     <label for="county_yes">Yes</label>
<input 
<?php 
   if (!$_POST && !(strcmp($row_GetMember['county_relief'],"no"))) {
      echo "checked=\"checked\"";
   }
   elseif ($_POST && !(strcmp($_POST['county_group'],"no"))){
      echo "checked=\"checked\"";
   }
 ?>
name="county_group" type="radio" class="text_background" id="county_no" value="no" />
                     <label for="county_no">No</label></div>
                 </td>

[EDIT: added

 tags around code -- MOD][/color]

[NogDog]

This might not be the exact solution, but I think something structured like this will (a) ensure that one or the other radio button is checked, and (b) by separating the PHP logic from the HTML output, it will make it easier for you to debug the logic if it is wrong.

<?php
if( ( !$_POST && !strcmp($row_GetMember['county_relief'],"yes") ) ||
    ( $_POST && !strcmp($_POST['county_group'],"yes") ) )
{
   $check1 = 'checked="checked';
   $chekc2 = '';
}
else  // if yes is not checked, the no must be:
{
   $check1 = '';
   $check2 = 'checked="checked"';
}
?>
<td><div align="left">
<input <?php echo $check1; ?>
name="county_group" type="radio" class="text_background" value="yes" id="county_yes"/>
<label for="county_yes">Yes</label>
<input <?php echo $check2; ?>
name="county_group" type="radio" class="text_background" id="county_no" value="no" />
<label for="county_no">No</label></div>
</td>

[jimdavidson]

Thanks NogDog,

That works for me!

Thanks again!🙂