PHP Syntax Error on Last Line?

Question

<?php
if (!isset($_POST['ign'], $_POST['email'])) {
    if($_POST['ign'] && $_POST['email']){
    echo "Please fill out all of the fields!";
        die;
}

if (empty($_POST['ign']) || empty($_POST['email'])) { 
    echo ("Please enter all of the values!"); 
    die;
}

if(filter_var($_POST['email'], FILTER_VALIDATE_EMAIL)) {
    $email = $_POST['email'];
    echo ("Thanks, " . htmlentities($_POST['ign']) . ", you will recieve an email when the site is complete!");

}

else {
    echo "Your email was invalid!";
    die;
}
?>

I'm getting a syntax error on the last line where ?> is..

Also, just a random side note, can anyone teach me how to insert this into my code?

$valid = (bool)preg_match('/^[a-zA-Z0-9]{1,30}$/', $_POST['username']);

is it just

if ($valid == TRUE) {
////////
}

or is declaring that variable already running it?

as a personal preference these are the moment when i consider the advantage of writing the opening and closing braces in the next line so as to assist my eyes in tracing which opening brace(s) is paired to which closing brace. — ianace
– ianace, Commented Jan 10, 2012 at 2:37

FtDRbwLXw6 · Accepted Answer · 2012-01-10 02:25:56Z

5

You have a missing ending brace } in the first if block.

answered Jan 10, 2012 at 2:25

FtDRbwLXw6

29.1k16 gold badges74 silver badges108 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Umbrella Over a year ago

+1 You can pretty much assume whenever you get an error on the last line of a file, it is a missing } somewhere. Possibly every time I have seen an error on the last line of a file, it has been a missing }

al01 · Accepted Answer · 2012-01-10 02:26:31Z

0

Without following the logic of your script, you have five open braces { and four closed braces }

answered Jan 10, 2012 at 2:26

al01

1221 silver badge4 bronze badges

Comments

slugonamission · Accepted Answer · 2012-01-10 02:26:51Z

0

You open two scopes (with {) around the if (!isset($_POST['ign'], $_POST['email'])) line, so you need to close the dangling one where you need to.

answered Jan 10, 2012 at 2:26

slugonamission

9,6741 gold badge36 silver badges41 bronze badges

Comments

Community · Accepted Answer · 2017-05-23 12:21:12Z

You can take the code from your last question, Validate Email Error , and add a test for length of username and check username is alphanumeric like this:

<?php
if (isset($_POST['ign'], $_POST['email'])) {//do the fields exist
    if($_POST['ign'] && $_POST['email']){ //do the fields contain data
        if (!filter_var($_POST['email'], FILTER_VALIDATE_EMAIL)) {//is the email address of valid form
            if(ctype_alnum($ign) && ($ign.length > 0) && ($ign.length <= 30)){//is ign alphanumeric and between 1 and 30 characters long
                echo ("Thanks, " . htmlentities($_POST['ign']) . ", you will recieve an email when the site is complete!");
            }
            else{
                echo ("Please enter a valid user name!");
            }
        }
        else{
            echo ("Please enter a valid email!");
        }
    }
    else {
        echo ("Please enter all of the values!");
    }
}
else {
    echo ("Error in form data!");
}
?>

Note: If my assumption that $_POST['ign'] is what you meant to say to in the line (bool)preg_match('/^[a-zA-Z0-9]{1,30}$/', $_POST['username']) ,is wrong ... post more details on what you need and I'll update my answer.

Collectives™ on Stack Overflow

PHP Syntax Error on Last Line?

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related