Parse error: syntax error, unexpected 'else' (T_ELSE)

dekon

could someone please help me solve the following code error thanks

<?php
//create_cat.php
include 'mysql.php';
include 'header.php';

if($_SERVER['REQUEST_METHOD'] != 'POST')
{
	//someone is calling the file directly, which we don't want
	echo 'This file cannot be called directly.';

else
{
	//check for sign in status
	if(!$_SESSION['loggedIn'])
	{
		echo 'You must be signed in to post a reply.'. mysql_error();
	}
	else
	{
		//a real user posted a real reply
		 $topicid = mysql_insert_id(); 
		$sql = "INSERT INTO 
                            post(content, 
                                  date, 
                                  topic, 
                                  postby)
				  VALUES 
                            ('" . mysql_real_escape_string($_POST['content']) . "', 
                                 NOW(),
								  " . $topicid . ",
						" . mysql_real_escape_string($_GET['id']) . ",
						" . $_SESSION['user_id'] . ")";
		$result = mysql_query($sql);

	if(!$result)
	{
		echo 'Your reply has not been saved, please try again later.'. mysql_error();
	}
	else
	{
		echo 'Your reply has been saved, check out <a href="topic.php?id=' . htmlentities($_GET['id']) . '">the topic</a>.';
	}
}
}
}
include 'footer.php';
?>

Lord_Yggdrasill

Apparently you are missing a right bracket } right after your first if statement, the script is expecting this right bracket but instead it detects an else keyword, this is why it throws an error.

dekon

Your reply has not been saved, please try again later.You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ' 4)' at line 10

this is the error i am now getting can you help?

Derokorian

you're trying to insert 5 things into 4 columns.

dekon

i have deleted " . $topicid . ", and now get this error

traq

Why don't you read the error message, look at the line it references, and see if you can notice what is wrong?

(hint: you can try printing your [font=monospace]$sql[/font] variable to see what the query you send to MySQL actually looks like.)