Hey guys, I need your help with this code, thanks in advance.

[pickamaterina]

<?php
$user="root";
$password="";
$database="base";
$host="localhost";
$table="com";

mysql_connect($host, $user, $password) or die("error");
mysql_select_db($database) or die("error");

$name = $_POST['name'];
$lastname = $_POST['lastname'];
$email = $_POST['email'];
$coment= $_POST['coment'];
$error = '';

 if(empty($name) || empty($lastname) || empty($email) || empty($coment))
{
      $error .= 'error. ';
}
if(!preg_match("/^[a-zA-A]+$/i", $name))
{
     $error .= 'Error ';
}

if(!preg_match("/^[a-zA-A]+$/i", $lastname))
{
     $error .= 'Error ';
}



if(!preg_match("/^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,3})$/i", $email))
{
     $error .= 'errorl. ';
}

if(!preg_match("/^[a-zA-z0-9.,?!]+$/i", $comment))
{
     $error .= 'Error ';
}

if($error == '')
{
 
$result = "INSERT INTO com (name, lastname, email, coment) VALUES('$name', '$lastname', '$email','$coment')";
    echo "<script type='text/javascript'>alert('Thanks.'); window.close();</script>";}
else{
      echo"<script type='text/javascript'>alert ('$error'); window.close(); </script>";}
 
Errors seems to be working its just it won't insert data in table in mysql database....

Thank you so much.

Edited September 29, 2013 by pickamaterina

[Ch0cu3r]

Because you're not doing anything with the query

$result = "INSERT INTO com (name, lastname, email, coment) VALUES('$name', '$lastname', '$email','$coment')";

You have just defined a query to $result. PHP doesn't see it as a query, it just sees it as a string.

 

What do you expect that line to do?

Edited September 29, 2013 by Ch0cu3r

[pickamaterina]

well I tried connecting it via if $result=".." (mysql_query($result)){
    echo "<script type='text..... and it didn't work...  sorry I am just a beginner so I can't think of any way how to connect it and leave that if or use it because it is kinda needed for validation?

[pickamaterina]

misstyped

$result=".."

if(mysql_query($result)){
    echo "<script type='text.....

[mac_gyver]

moving thread to the correct forum section...

 

Please use the forum's

bbcode tags (the edit form's <> button) when posting code.

 

 

[Ch0cu3r]

Well you need to check if mysql_query is returning any errors

$sql = "INSERT INTO com (name, lastname, email, coment) VALUES('$name', '$lastname', '$email','$coment')";

if(mysql_query($sql))
{
    echo "<script type='text/javascript'>alert('Thanks.'); window.close();</script>";
}
else
{
	echo 'Error: ' . mysql_error();
	// I have commented out the line below.
    //echo"<script type='text/javascript'>alert ('$error'); window.close(); </script>";
}

[pickamaterina]

Actually the point was to post data in form and also use this validation?

[pickamaterina]

Damnit, I meant to say to post data from form into database and still use this validation?

[Ch0cu3r]

When you perform queries on database you need to make sure they execute ok. During development you need to to output as much debug information as possible when something your expect to happen doesn't happen, if that makes sense.

 

You said earlier when you use mysql_query it didn't work. So I recommended you add echo mysql_error to see if the query returns an error. You can remove the debug code when everything is working.

 

So does your query return any errors when your run your script?

[pickamaterina]

Thanks I acutally figured it out, and how stupid I was, I should have just typed $result=mysql_query("INSERT....."); its working now thanks everyone !