PHP Если выбран переключатель, заполните текстовое поле
Я пытался решить эту маленькую проблему и собираюсь вырвать мои волосы, смеется. Я больше дизайнер, чем программист, но я создал форму отправки в php для своей школы. Вот тестовая форма, над которой я работаю:
http://www.inrf.uci.edu/php-form-test/
То, что я пытаюсь сделать, это когда человек выбирает Внешнего пользователя, он должен указать свое Название компании. Когда кто-то выбирает Внутреннего пользователя, он должен заполнить свое имя группы / PI. Ну, это не работает. Когда я выбираю Внутренний, он должен требовать только имя группы / 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 <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 } ?> - <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 <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 } ?> - <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>
Решение
Обратите внимание на вашу логику:
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.
}
Другие решения
Других решений пока нет …