my php form keeps duplicating rows in mysql - php

Question

Hi im trying to create a subscribe to mailing list widget. its kind of working but keeps adding a blank row after every insert. I would like to jsut be able to let users add their name and email address to 2 inputs and click submit, then the details should be saved into the database, avoiding spam and duplications etc.
here is where am at so far hopefully someone could give a few pointers im very new to php so any help is appreciated thanks!
<?php
// Connect to server and select database.
//mysql_connect("$subsc_hostname", "$subsc_username", "$subsc_password")or die("cannot connect");
//mysql_select_db("$subsc_database")or die("cannot select DB");
$mysqlconn= new PDO('mysql:dbname=' .$subsc_database . ';host=' . $subsc_hostname, $subsc_username, $subsc_password);
$subsc_name = $_POST['sub_FullName'];
$subsc_email = $_POST['sub_EmailAddress'];
// Insert data into mysql
$sql="INSERT INTO subscribers(subsc_name, subsc_email)VALUES('$subsc_name', '$subsc_email')";
$result=mysql_query($sql);
// close connection
mysql_close();
?>
<form method="post" action="">
<div class="form-group">
<label>Full Name:</label>
<input class="form-control" type="text" name="sub_FullName" />
<br>
<label>Email:</label>
<input class="form-control" type="text" name="sub_EmailAddress" />
<div class="pull-right">
<input type="submit" value="Subscribe" />
</div>
</div>
</form>

Answer 1

You may want to move the last } in your file:
} else {
mail( $to, $subject, $message, $headers );
echo "<div class='alert alert-success alert-dismissable'><h4>Subscribed!</h4><p>Thanks $contact_name,<br /><br />you will start receiving emails via $contact_email very soon.</div>";
// FROM HERE
// Insert data into mysql
$sql="INSERT INTO subscribers(subsc_name, subsc_email)VALUES('$subsc_name', '$subsc_email')";
$result=mysql_query($sql);
// TO HERE
}
To add to the comments to your question, you may want to take a look at lightweight frameworks like Silex and to database abstraction layers, like Doctrine. Using such tools, you greatly reduce the risks of SQL injections and you don't need to code every single user provided data control by hand.