unexpected T_ELSE

[bravo14]

Hi guys

I have the following error

Parse error: syntax error, unexpected T_ELSE in /home/sites/maypolejuniors.com/public_html/subscribe.php on line 171

from the following code

<?php
 include_once('includes/connect.php');
//assign form entries from subscription form to variables
$email=$_POST["form_email"];
$name=$_POST["form_name"];
$status=$_POST["form_subscribe2"];
echo ("status ".$status);
		if($status=="1"){
				$result="$sql = 'SELECT * FROM `email_table` where `email`=$email";
	//is the email address already subsribed
				if(mysql_num_rows($result) > 0) 
						{ 
						//The email address already exists in the table, output message
   						echo('The email address '.$email.' is already subscribed to the Maypole Juniors newsletter'); 
						} 
		}
		else //if not then enter into database
			{ 
				$sql="INSERT INTO `email_table` (`name`,`email`)	VALUES ('".$name."', '".$email."')";

			if (!mysql_query($sql,$con)){

				die('Error: ' . mysql_error());
				}
						else
				{
			echo($name. ' who has '.$email.' as their email address has successfully subscribed to the Maypole Juniors newsletter');
			}
		}
		else //line 171
		{
		echo("Do something else");
		$sql="DELETE FROM email_table WHERE email='".$email."'";
		if (!mysql_query($sql,$con)){

				die('Error: ' . mysql_error());
				}
				else
				{
			echo($name. ' who has '.$email.' as their email address has successfully unsubscribed from the Maypole Juniors newsletter');
		}
		}

mysql_close($con);
 ?>

Any ideas what I have done wrong

[NogDog]

Looks like you have an else following another else. (You can only have one else in any if/elseif/else construct.)

[bravo14]

I don't mean to sound thick, but what should the code look like?

[Weedpacket]

bravo14 wrote:

but what should the code look like?

A lot tidier, for a start (then it would be easier for you to find the problem). Something like

<?php
include_once ('includes/connect.php');
//assign form entries from subscription form to variables
$email = $_POST["form_email"];
$name = $_POST["form_name"];
$status = $_POST["form_subscribe2"];
echo ("status " . $status);
if($status == "1")
{
	$result = "$sql = 'SELECT * FROM `email_table` where `email`=$email";
	//is the email address already subsribed
	if(mysql_num_rows($result) > 0)
	{
		//The email address already exists in the table, output message
		echo ('The email address ' . $email . ' is already subscribed to the Maypole Juniors newsletter');
	}
}
else //if not then enter into database
{
	$sql = "INSERT INTO `email_table` (`name`,`email`)    VALUES ('" . $name . "', '" . $email . "')";

if(!mysql_query($sql, $con))
{

	die('Error: ' . mysql_error());
}
else
{
	echo ($name . ' who has ' . $email . ' as their email address has successfully subscribed to the Maypole Juniors newsletter');
}
}
else //line 171
{
	echo ("Do something else");
	$sql = "DELETE FROM email_table WHERE email='" . $email . "'";
	if(!mysql_query($sql, $con))
	{

	die('Error: ' . mysql_error());
}
else
{
	echo ($name . ' who has ' . $email . ' as their email address has successfully unsubscribed from the Maypole Juniors newsletter');
}
}

mysql_close($con);
?> 

[NogDog]

Without knowing exactly what logic you want implemented, I cannot give an exact answer. As Weedpacket's post shows, you have two consecutive top-level else blocks. Either you need to convert the first into an elseif() (with an applicable condition), or else the logic is just wrong and things need to be reorganized in some manner.