Form

[Nickosssss]

Hello i have this form and i want when the two text fields are empty to stop the process and not to load the php file on the action....But it always process no matter if the text field is empty or not

<?php

$top1 = $POST['top1'];
$top2 = $POST['top2'];

$top1 = trim($top1);
$top2 = trim($top2);
if (!$top1 || !$top2)
{
echo 'You have not entered search details. Please go back and try again.';
exit;
}
?>

<form class = "myform" action="choices.php" method="POST"> 
		<div style = "position:relative; top:0%; left:35">
				<font size = "4"><b> Please fill out the form </b></font><br><br>
				<table border="1" cellpadding="7" cellspacing="0" style="border-collapse: collapse" bordercolor="#111111" width="600" id="AutoNumber1">
				<tr>
						<td rowspan = "4" width = "20%"><strong>Project Topic 1</strong></td>
					</tr>				
					<tr>
						<td align = "right" width = "30%"> Topic Name: </td>
						<td width = "50%"><input class = "input" type = "text" name = "top1" value = "<?echo $_POST['top1'];?>" maxlength = "100" size = "50"></td>
					</tr>
					<td width = "50%"><input class = "input" type = "text" name = "top2" value = "<?echo $_POST['top2'];?>" maxlength = "100" size = "50"></td>
					</tr>

[halojoy]

I do not understand why.
But try this, and see what happens:

<?php

if ( $_POST['submit'] ){

$top1 = trim( $_POST['top1'] ); 
$top2 = trim( $_POST['top2'] );

if ( empty($top1) || empty($top2) ){
die( 'Please enter all fields');
exit;
}
else{
die( 'Thank you!');
exit;
}

}
?>
<form ..... ... ...... ....method="POST">
.....
.....
<input class="input" type="submit" name="submit" value="SUBMIT">
</form>

[Nickosssss]

loll i did a really silly stupid but thank you mate....urs works as well

thank u

[halojoy]

Okay.
Maybe change this bit of my code to

if ( empty($top1) || empty($top2) ){
  echo "Please enter all fields<br><br>";
  // After this continues down automatically to form again
  // To fill in missing data.
}
else{
  // Thank you, because have got all data wanted. Form complete!
  // And when you get sometthing, you say thanks
  echo "Thank you! <br>";
  echo "You entered:<br>$top1<br>$top2<br><br>";
  echo "<a href="index.php">Home</a>";
  //Here this user can return to HOME. Maybe after login.
  // So we exit this page.
  exit;
}

?>
<!-- HTML form here -->
<form>
<input type=submit value=SUBMIT>
</form>

I think that would be super!

[harmor]

Actually wouldn't the "exit" be in the "if" not the "else"

if ( empty($top1) || empty($top2) ){
  echo "Please enter all fields<br><br>";
exit;
}
else{
  echo "Thank you! <br>";
  echo "You entered:<br>$top1<br>$top2<br><br>";
  echo "<a href="index.php">Home</a>";
}

And if you only wanted the fields that they typed in display then you can use this code

if ( !empty($top1) || !empty($top2) ){
  echo "You entered:<br>$top1<br>$top2<br><br>";
  echo "<a href="index.php">Home</a>";
}

Notice the exclamination point.

[halojoy]

Hmmmm

Comments to harmor

if ( empty($top1) || empty($top2) ){
  echo "Please enter all fields<br><br>";
  // After this continues down automatically to form again
  // To fill in missing data.
}
else{
  // Thank you, because have got all data wanted. Form complete!
  // And when you get sometthing, you say thanks
  echo "Thank you! <br>";
  echo "You entered:<br>$top1<br>$top2<br><br>";
  echo "<a href="index.php">Home</a>";
  //Here this user can return to HOME. Maybe after login.
  // So we exit this page.
  exit;
}

?>
<!-- HTML form here -->
<form type=POST>
<input type=submit value=SUBMIT>
</form>

Should work nice!

Remember this is only the changes,
from my first suggestion in this thread.

But there are plenty of ways to do such a form script.
This is just one of them. But it works!

[Nickosssss]

ok mates thanks 🙂