in my three files i am getting only one error,Parse error: syntax error, unexpected $end

Question

1.00/5 (1 vote)

See more:

i try to improve it this time so it can become more readable. and may be i mismatch {} this. so thats why i am writing a hint after every { closing in which line, and after this } started at this line, it will make it more clear.

thx
and its showing error in the line 39, and the error is
Parse error: syntax error, unexpected T_ELSE in

1.

PHP

<?php
2. session_start();
3. include ("connect.php");
4. include ("html_codes.php");

5. if (isset ($_POST ['submit'])) { closing at line 49m before php closing statement.
6.    $error = array ();

7. if (empty ($_POST ['username'])) { closing at line 9
8.    $error [] = 'please enter a username.';
9.    } started at line 8
10.    else if (ctype_alnum($_POST['username'])) { closing at line 12
11.        $error[]= 'username must consist of letters and numbers only';
12.            }started at line 10

13. if (empty ($_POST ['email'])) {closing at line 15
14.    $error [] = 'please enter your email.';
15.    }started at line 13

16.    else if  (preg_match ("/^ ([a-zA-Z0]) +([a-zA-Z0-9_-])+([a-zA-Z0-\._-]+)+$/", $_POST ['email']))17. { closing at line 20
18.        $email=  mysql_real_escape_string($_POST ['email']);
19.        $error[]='your e-mail adress is invalid';
20.    } started at line 17

21.    if (empty ($_POST ['password'])) { closing at line 23
22.        $error[]='please enter your password';
23.    } started at line 21

24.    else { closing at line 38
25.        if (empty ($error)){ closing at 28
26.        $result =mysql_query ("SELECT * FROM users WHERE email = '$email' OR username = '$username'")
27.        or die (mysql_error());
28.        } started at 28

29.    if (mysql_num_rows ($result)==0){closing at 34
30.    $activation = md5 (uniqid (rand(), true));
31.  $result2 =mysql_query ("INSERT INTO tempusers (user_id,username,email,password,activation) VALUES ('', 32.   '$username', '$email', 'password', '$activation')")
33.            or die (mysql_error());
34.    }started at 29

35. if (!result2) {closing at 37
36. die ( 'could not insert into database:' .mysql_error()); 
37. }

38.    i moved one curly bracket from here to line 

39. else{ closing at 45
40.    $message="To activate your account, please click on this link:\n\n";
41. $message="http://lisa1986.com". '/activae.php?email= '.urlencode ($email). "&key= 
42. $activation";
43. mail($email, 'Registration Confirmation', $message);
44. header ('Location: prompt.php?x=1');
45. }started at 39
45a.  } this is the curly bracket which i moved from line 38
46. else {closing at 46b
46a. header ('Location: prompt.php?x=2');
46b. } new curly bracket
46c. <pre>$error_message = '<span class = "error">';

46b.

foreach ($error as $key=> $values){

47. $error_message='</span><br/><br/>';
48. }started at 46
49. }started at 5

50. ?>

Posted 3-Jun-12 18:19pm

arifsuhail

Updated 4-Jun-12 22:36pm

v5

Add a Solution

Comments

Sergey Alexandrovich Kryukov 4-Jun-12 0:23am

Did you look at your post after submission? It's not readable!
--SA

arifsuhail 4-Jun-12 1:06am

yah your are right i am posting it again sorry

thanks by

Mohibur Rashid 4-Jun-12 3:24am

Please post the Question again, if you can handle the change, someone will fix it for you, may be me.

arifsuhail 4-Jun-12 23:22pm

i have posted it again, first i try to see weather i am mismatching {}, may be thats the case that's why i write the hint when the curly bracket is opening { closing at this line .......... and when the curly bracket is closed } started it this line. it also help me to check, and the result which i wrote here that where they started and where they closed, i get that result from notepad++, thx

Mohibur Rashid 5-Jun-12 1:48am

tell me this, did you put those line number at the beginning of the line by yourself? by they why this thread says solved?

arifsuhail 5-Jun-12 1:58am

i dont know why its says solved, and yah i put those no. my in order so i can explain my question. may its say solved cause i accepted on solution may, but i am not sure cause i am very new the forum, can i make again unsolved??

arifsuhail 5-Jun-12 2:00am

any way i am rejecting the solution now it may show unsolbed again,,, may it work

arifsuhail 5-Jun-12 2:01am

and it was your solution i accepted last time lolz, thanks

Mohibur Rashid 5-Jun-12 1:55am

Check my edited answer

arifsuhail 5-Jun-12 1:59am

i did not understand check my edited answer,

arifsuhail 5-Jun-12 2:38am

can i borrow for a while so i can solve my question my solve, the which i understood you are saying that i started one if on line 35 and gave it two else statement on line 39 and on line 46 which is wrong?? did i understand you correctly?? and you are saying closing statement on line 38 should move down, move to where after closing last else statement?? and one more thing if,, statement if cant have to else statement thats mean i am missing one if statement before line 46, if true what if statement that's going to be, thanks

arifsuhail 7-Jun-12 22:50pm

how are you, i have one more problem. and this problem is also silly, as you know i am fully capable of doing stupid mistake, if you have time please try to solve it for me thanks and the link for the new question is http://www.codeproject.com/Questions/399964/failed-to-open-file-in-the-directory-i

thanks

arifsuhail 7-Jun-12 23:21pm

thats solved my one problem now its showing the complete path thanks for the help

Mohibur Rashid 5-Jun-12 2:00am

read my answer again. and yes you can reject answer

arifsuhail 5-Jun-12 2:09am

oh yah i got now i am trying to understand it thanks i will post you, as you know i am new to the coding and to the forum so i did not understand where you are referring me to, i was reading your reply again and again instead of edited answer, and even your reply say read my edited answer, i think i need brain surgery , or brain replacement lol

Mohibur Rashid 5-Jun-12 2:11am

No problem, you can have one from em if you want. but it will cost you and surgery wont be successful

arifsuhail 5-Jun-12 2:39am

can i borrow for a while so i can solve my question my solve, the which i understood you are saying that i started one if on line 35 and gave it two else statement on line 39 and on line 46 which is wrong?? did i understand you correctly?? and you are saying closing statement on line 38 should move down, move to where after closing last else statement?? and one more thing if,, statement if cant have to else statement thats mean i am missing one if statement before line 46, if true what if statement that's going to be, thanks

Mohibur Rashid 5-Jun-12 2:44am

issue one: you would find out what is right, i am telling you what is wrong.

At Line 25 you started an if, then there is else statement at line 29 then there is another else statement at line 47. if you follow the brackets carefully you will understand

arifsuhail 5-Jun-12 4:49am

i solved it but i am still a bit confused, that's why i am asking you this question again, please see this in a updated question not in answer. last time you told me two important things, follow the bracket and one if statement cant have to else, but if you look the updated quesion, one if statement started on line 36, than one else started on line 39 and another one in line 46, you told me it cant happen and second thing you told me i should follow the bracket carefully, i think answer lie their, cause i am not saying you are wrong, but i wanna understand it more. that's why i am asking you how can i follow the bracket properly, so i can know preciously where one statement is opening and where i am closing it. and last time as you mention that } which was on line 38 should move down, yah i have to move it down on line 45a and it works. i did not want to change the no. in the updated question that's why i gave the no. like 45a, 45b like this. thanks for the help

2 solutions

Solution 1

Even though, I didn't see your code. but it is very much possible that you are mixing up with curly bracket. try to found the combination. Use Notepad++. this is an excellent tool for coding php. it will also help you to find out bracket miss management.

[adding]
Look at your line 35, an if started. at line 38 there is closing from the line 24, it is a closing of an else

then another else started. and if condition cannot have two else statement. if the next else is related with your line 35 if then the closing bracket at line 38 will move down.

Posted 3-Jun-12 21:26pm

Mohibur Rashid

Updated 4-Jun-12 19:54pm

v2

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

arifsuhail · Accepted Answer · 2012-06-04T22:37:00

PHP

<?php
session_start();
include ("includes/connect.php");
include ("html_codes.php");

if (isset ($_POST ['submit'])) {
    $error = array ();

if (empty ($_POST ['username'])) {
    $error [] = 'please enter a username.';
    }
    else if (ctype_alnum($_POST['username'])) {
        $username = $_POST ['username'];
    }else{
        $error[]= 'username must consist of letters and numbers only';
            }
 if (empty ($_POST ['email'])) {
    $error [] = 'please enter your email.';
    }

    else if  (preg_match ("/^ ([a-zA-Z0]) +([a-zA-Z0-9_-])+([a-zA-Z0-\._-]+)+$/", $_POST ['email'])){
        $email=  mysql_real_escape_string($_POST ['email']);
        $error[]='your e-mail adress is invalid';
    }

    if (empty ($_POST ['password'])) {
        $error[]='please enter your password';
    }
    else{ $password=  mysql_real_escape_string($_POST ['password']);
    }

        if (empty ($error)){
        $result =mysql_query ("SELECT * FROM users WHERE email = '$email' OR username = '$username'")
        or die (mysql_error());
        if (mysql_num_rows ($result)==0){
    $activation = md5 (uniqid (rand(), true));
  $result2 =mysql_query ("INSERT INTO tempusers (user_id,username,email,password,activation) VALUES ('', '$username', '$email', 'password', '$activation')")
            or die (mysql_error());
    }
if (!result2) {
die ( 'could not insert into database:' .mysql_error());
}

else{
    $message="To activate your account, please click on this link:\n\n";
$message="http://lisa1986.com". '/activae.php?email= '.urlencode ($email). "&key= $activation";
mail($email, 'Registration Confirmation', $message);
header ('Location: prompt.php?x=1');
}
        }


else {
    header ('Location: prompt.php?x=2');
}
$error_message = '<span class = "error">';
foreach ($error as $key=> $values){
    $error_message='</span><br/><br/>';
    }


}


?>