PHP Если выбрана радиокнопка, заполните текстовое поле

Question

Я пытаюсь понять эту небольшую проблему, и я готов вытащить свои волосы из LOL. Я больше дизайнер, чем программист, но я создал форму для отправки в php для своей школы. Вот тест форма Я работаю на:

http://www.inrf.uci.edu/php-form-test/

То, что я пытаюсь сделать, это когда человек выбирает внешний пользователь, они должны быть обязаны заполнить их название компании. Когда кто-то выбирает Внутренний пользователь, они должны заполнить свое имя группы/PI. Ну, это не работает. Когда я выбираю Internal, для него требуется только имя группы/PI, а не название компании. Буду признателен за любую помощь. Спасибо!

Вот мой PHP код здесь:

// internal selected (group pi) 

if ($userType == 'internal') { 
    $internalChecked = ' checked="checked" '; 
} else { 
    if(trim($_POST['grpPI']) === '') { 
     $grpPIError = '<span class="error">Please enter your group or PI name.</span>'; 
     $hasError = true; 
    } else { 
     $grpPI = trim($_POST['grpPI']); 
    } 
} 

    } 

// external selected (company name) 

if ($userType == 'external') { 
    $externalChecked = ' checked="checked" '; 
} else { 
    if(trim($_POST['companyName']) === '') { 
     $companyNameError = '<span class="error">Please enter your company name.</span>'; 
     $hasError = true; 
    } else { 
     $companyName = trim($_POST['companyName']); 
    } 
} 

и HTML здесь:

<table class="usertype" id="usertype"> 

         <tr> 
          <td colspan="2"> 
          <input type="radio" name="userType" value="external" <?php echo $externalChecked; ?>>External 
          <select name="externalSelect" size="1"> 
           <option selected="selected">Select</option> 
           <option>Academic</option> 
           <option>Non-Profit</option> 
           <option>Commercial</option> 
          </select><?php if($externalError != '') { ?><span class="error"><?=$externalError;?></span><?php } ?></td> 
      </tr> 
         <tr> 
          <td colspan="2"><label for="companyName">Company Name:</label> 
          <input type="text" size="40" name="companyName" id="companyName" value="<?php if(isset($_POST['companyName'])) echo $_POST['companyName'];?>" /> 
       <?php if($companyNameError != '') { ?> 
       <span class="error"><?=$companyNameError;?></span><?php } ?></td> 
         </tr> 
         <tr> 
          <td colspan="3"><input type="radio" name="userType" value="internal" <?php echo $internalChecked; ?>>Internal &nbsp;&nbsp;&nbsp;&nbsp;<input title="Enter Account String. Required for Internal Users." type="text" size="30" name="ucfund" value="<?php if(isset($_POST['ucfund'])) echo $_POST['ucfund'];?>" placeholder="KFS or Account (eg. xxxxxxx)" /><?php if($ucfundError != '') { ?><span class="error"><?=$ucfundError;?></span><?php } ?>&nbsp;-&nbsp; <input title="Enter Fund String. Required for Internal Users." type="text" size="25" name="ucfund2" value="<?php if(isset($_POST['ucfund2'])) echo $_POST['ucfund2'];?>" placeholder="Fund (eg. xxxxx)" /><?php if($ucfund2Error != '') { ?><span class="error"><?=$ucfund2Error;?></span><?php } ?><br /><span class="small">(Please enter KFS number or Accound Fund)</span> 
</td> 
         </tr> 

         <tr> 
          <td colspan="3">Optional&nbsp;&nbsp;&nbsp;&nbsp;<input type="text" size="30" name="sub" value="<?php if(isset($_POST['sub'])) echo $_POST['sub'];?>" placeholder="Sub" /><?php if($subError != '') { ?><span class="error"><?=$subError;?></span><?php } ?>&nbsp;-&nbsp; <input type="text" size="25" name="proj" value="<?php if(isset($_POST['proj'])) echo $_POST['proj'];?>" placeholder="Project" /><?php if($projError != '') { ?><span class="error"><?=$projError;?></span><?php } ?> 
</td> 
         </tr> 
         <tr> 
          <td colspan="3"><label for="grpPI">Group/PI:</label> 
          <input type="text" size="40" name="grpPI" id="grpPI" value="<?php if(isset($_POST['grpPI'])) echo $_POST['grpPI'];?>" /> 
       <?php if($grpPIError != '') { ?> 
       <span class="error"><?=$grpPIError;?></span><?php } ?></td> 
       </tr> 

     </table> 

Answer 1

Примечание логику:

if ($user == 'internal') { 
    ... 
} else { 
    ... check for empty field ... 
} 

Таким образом, если пользователь выбирает 'внешний', то поле «только для внутреннего» будет пустым, и вы вынудите ошибку. почему должно быть поле «только для внутреннего», поскольку пользователь выбирает «внешний».

Аналогично для вашего external проверьте, что поле только для внутреннего использования будет пустым и вы будете вынуждать другую ошибку.

Вам нужно else положения, чтобы связать эти два if() заявления вместе, так что только один из двух путей коды выполняет:

if ($user == 'internal') { 
    ... do internal-only checks ... 
} else if ($user == 'external') { 
    ... do external-only checks ... 
} else { 
    ... $user is undefined type, "zomg how'd you get here" error handling. 
}