需要一个文本表单字段

Question

I've been struggling to have a text form field required. So when some one doesn't fill his name he will receive an error like 'No title filled!'

I got this now but it doesn't work that well cause when I submit it insert into the db.

if(isset($_POST['submit'])) {
    $update = "UPDATE post SET `title`='$_POST[title]', `pic`='$_POST[pic]', `youtube`='$_POST[youtube]' WHERE id = $_POST[id]";
    $db->query($update) or die($db->error);

        if($_POST['title'] == "") {
            $error = "Title is required!";
        }
        if ($_POST['pic'] == "") {
            $error = "Picture is required!";
        }
        if(isset($error)){
            echo $error;
        } else {
        echo '<p>Your post has been updated!</p>';
        } 

 }

Answer 1

You need to stop your code from being executed if an error is found, not just echo the error. All your other code that submits the data to the database should ONLY be executed if there is no error. Try something like this:

Edit: Upon seeing the update to your code, this is what you need to do:

if(isset($_POST['submit'])) {
    if(!isset($_POST['title']) || trim($_POST['title']) == "") {
        $error = "Title is required!";
    }
    if (!isset($_POST['pic']) || trim($_POST['pic']) == "") {
        $error = "Picture is required!";
    }
    if(isset($error)){
        echo $error;
    } else {
        $update = "UPDATE post SET `title`='" . mysql_real_escape_string($_POST['title']) . "', `pic`='" . mysql_real_escape_string($_POST['pic']) ."', `youtube`='" . mysql_real_escape_string($_POST['youtube']) ."' WHERE id = " . mysql_real_escape_string($_POST['id']);
        $db->query($update) or die($db->error);
        echo '<p>Your post has been updated!</p>';
    } 
}

The problem is, your data was being submitted to the database no matter what happened after with the validation - by the time you checked for errors it was too late, as the SQL had already been executed. If you do it the way shown above, it will only submit if the $error variable is not set, which is what you want.