2013-01-22 139 views
4

忽略了mysql的明显安全漏洞,也没有sql转义字符串,有谁知道为什么我的sql表在页面重新加载时被填充为空msgs。 页面重新加载表单提交的每个时间。我很困惑,为什么会发生这种情况,我该如何阻止它?Php sql,表单在页面加载时重新提交

<?php 
    ob_start(); 
    session_start(); 
    $con = mysql_connect("localhost","username","pass"); 
    if (!$con) 
     { 
     die('Could not connect: ' . mysql_error()); 
     } 

    $dates = date('Y-m-d H:i:s'); 
    $uid = $_SESSION['user_id']; 
    $msg_id = (int) $_GET['msg_id']; 
    mysql_select_db("db_table", $con); 

    $result = mysql_query("SELECT users.first_name, users.last_name , intro.intro, intro.outro FROM intro INNER JOIN users ON intro.user_id = users.user_id WHERE intro.message_id = {$msg_id}"); 
    while($row = mysql_fetch_array($result)) 
    { 
     echo "<div id=\"start\"><div class=\"namedate\"><h1>". $row['first_name'] ." ". $row['last_name'] . "</h1><h2>test</h2></div><div id=\"holdmsg\"><div class=\"cent\"><strong>" . $row['intro'] . "</strong><br><i>" . $row['outro'] ."</i></div></div></div> " ; 
    } 

与表格相关的部分是本部分,也是页面顶部的$ _GET。

<form action="" method="post">  
<?php 

$sql="INSERT INTO messages (user_id, intro_id , msg, date) VALUES (('$uid'), {$msg_id} ,'$_POST[msg]', ('$dates'))"; 

if (!mysql_query($sql,$con)) 
    { 
    die('Error: ' . mysql_error()); 
    } 

mysql_close($con); 

?> 

    <textarea rows="2"style="float:left" name="msg" type="text"placeholder="Elaborate on your idea..."></textarea> 
    <input id="togz2" style="float:right; "type="submit" value="SUBMIT" name="submit" class="butts"> 
    </div></div> 
</form> 
+0

非常奇特。我假设它与你的帖子或获取方法有关。祝你好运。 – user1775570

+0

^这个评论如何建设性的:/ – ramr

+1

你有什么样的逻辑来包装插入,以确保它只发生在某些情况下?如果以上是你所有,那么它会插入每次页面加载,因为你正在经历 – bhttoan

回答

3

为什么不这样做呢?

if (mysql_query($sql,$con)) 
    { 
    header('location:yourpage.php'); 
}else{ 
    die('Error: ' . mysql_error()); 
} 

而且我没有看到你的插入查询 上述任何这样的说法就像

if(isset($_REQUEST['submit'])){ //to check if posted 

    $sql="INSERT ...... 
} 

除此之外,你还应该插入到数据库之前验证数据。

+0

如果我downvoted.Please让我知道为什么,不要这样做的沉默。 –

2

这里是很好的例子,如何防止在页面加载重复条目

请参考链接http://www.webhostingtalk.com/showthread.php?t=700175

<?php 
session_start(); 
$faction = $_REQUEST['faction'] ; 
if (!is_array($_SESSION['serials'])) $_SESSION['serials'] = array(); 
if ($_REQUEST['serial']){ 
    if (in_array ($_REQUEST['serial'] , $_SESSION['serials'])){ 
    // duplicate submition, nullify it 
    $faction = ''; 
    }else{ 
    $_SESSION['serials'][] = $_REQUEST['serial'] ; 
    } 
} 


if ($faction) { // form submited 
    // if faction is not set completely ignore submition 

} 
?> 

<form> 
<input type="hidden" name="faction" value="42"> 
<input type="hidden" name="serial" value="<?=rand(1000000 , 9999999)?;>"> 
.... 
the rest of the form is here 
.... 
</form> 

另一种解决方案: -

你应该重新定向到成功后插入新的页面。

相关问题