[RESOLVED] Error message in unfilled out form boxes

[Sacreed]

I was wondering how to make it where that if a user is registering/filling out a form and he/she leaves a spot empty and it is required to be filled in how would I make it where that it would refreshes the page and says something like:

Error: Username,password,email must be filled in!

Thanks for all your guys help! 😃

[dagon]

basically

if (empty($_POST['username'])){
$username_error='Fill it in';
}

[Sacreed]

It doesn't seem to work. I even put it in the <?php ?> but it does what it normally even with the fields filled in..Am not sure if it is suppose to be in a certain spot or not though..😕 Sorry really new to php :o

[dagon]

do you do anything with the variable $username_error ?

[bradgrafelman]

There are many examples/methods out there on how to do this, but here's another...

$errors = array();

if(empty(trim($_POST['first'])))
    $errors[] = 'You must supply your first name!';

if(empty(trim($_POST['last'])))
    $errors[] = 'You must supply your last name!';

// etc.

// then, check if any errors occured:
if(!empty($errors)) {
    // errors occurred! output each error message and stop processing the form
}

[Sacreed]

Sorry about not replying sooner, bradgrafelman it says this:

Fatal error: Can't use function return value in write context in C:\wamp\www\directory\register.php on line 54

[NogDog]

The argument to empty() must be a variable, not any other expression. You can instead do:

if(trim($_POST['first']) === '') { // use === so that "0" is valid entry.

[bradgrafelman]

Oops! This is what happens when I spend too much time in Java and try to switch over to PHP. 🙁

[Sacreed]

NogDog you have { there..am pretty sure thats not suppose to be there because I get: Parse error: parse error in C:\wamp\www\directory\register.php on line 52

But when I remove that, I get: Notice: Undefined index: username in C:\wamp\www\directory\register.php on line 35

If any of you can tell me a good tut for php(i've looked up). I have found one though:

http://www.w3schools.com/php/php_variables.asp

but not sure if there is any better one.

Sorry if am causing a lot of trouble lol 😉

[dagon]

it was a code snippet, not the full thing, you put your code and the closing } in, of course {} are optional for one line conditional statements.

[Sacreed]

I was think that tbh..I should of went with my gut lol. Thanks.

[Sacreed]

Headache....can't think right now. I've tried putting code in it and it doesn't seem to work.

[Sacreed]

Took some work but guess what? what? I did it (= Thanks for all your guys help, but this is what really worked:

if (isset($_POST['submit']))
			{
			$username = htmlentities($_POST['username']);
			$password = htmlentities($_POST['password']);
			$email = htmlentities($_POST['email']);
			$name = htmlentities($_POST['name']);
			$confirmpassword = htmlentities($_POST['confirm password']);
			if ($username == '')
			{
			echo 'You did not enter a username!<br>';
			}

All you do now is just add each if for each $variable lol, but thanks again for all your guys help! I really appreciate it!

Edit:

Just wanted to add that for the confirm password it looks like this:

if ($confirmpassword !== $password)
			{
			echo 'Your passwords did not match up!<br>';
			}

[dagon]

why are you using htmlentities on all the post vars?

[Sacreed]

Ya I took that off myself. I've changed that code up a lot since I posted it lol.