Question

我有一个php页面，它从包含电子邮件addy的dbms中提取数据。这有效。然后它显示电子邮件addy和其他存储的数据数据。然后，用户可以选择在名为emailselected的php页面中设计的字段中放置“X”。这也有效。我现在想要根据存储的电子邮件addy使用新字段更新dbms，但更新语句不起作用。请帮忙。这部分php文件就在这里。

    include("db.php");
if (isset($_POST['ssubmit']))
    {
    $id_save = $test['id'];
    $emailselected_save = $_POST['emailselected'];
    $email_save = $test['email'];

    mysql_query("UPDATE emails SET selected='$emailselected_save' WHERE id = 'id'");
    echo "<input type='button' value='Email(s) sent' onclick='goBack()' />";
mysql_close($conn);
    } else {echo "hello";}
?>
</head>
<form method='post'>
    <div id='headd'>
    <br />
    <input type='button' value='Close this window without Sending' onclick='goBack()' />
    <input type='submit' name='ssubmit' id='ssubmit' value='Send Email Now!!!' />
    <p>Place an "X" in the emails you wish to send!!!</p>
    </div>
    <body>
        <br /><br/>

<?php
    include("db.php");  
    $result = mysql_query("SELECT * FROM emails WHERE unsubscribe  != 'x' ORDER BY lastname ASC");
            while($test = mysql_fetch_array($result))
            {
?>
    <table border='1' width='78%'>
    <tr align=\"left\">
        <td width='4%'><font color='black'><input type='text' size='1' id='emailselected' name='emailselected' /></font></td>
        <td width='15%'><font color='black'><?php echo $test['lastname']?></font></td>
        <td width='15%'><font color='black'><?php echo $test['firstname']?></font></td>
        <td width='40%'><font color='black'><?php echo $test['email']?></font></td>
        <td width='4%'><font color='black'><?php echo $test['id']?></font></td>
    </tr>
</table>
<?php
            }
?>
</form>

表单顶部是include语句。这是有效的，因为它提取电子邮件addy并将其显示在php文件中。

我做错了什么？

Answer 1

如果数据没有更新，您几乎可以使用以下可能性：

声明失败了;或
声明没有影响行;或
该声明影响了错误的行。

要找出哪个，您应该检查mysql_query的返回代码，看看它是否失败：

$rc = mysql_query('my hovercraft is full of eels');
if (!$result) {
    die('What?: ' . mysql_error());
}

如果没有失败，那么您应该检查它受mysql_affected_rows()影响的行数：

$num = mysql_affected_rows();
printf("Updated %d rows\n", $num);

检查返回它们的函数的返回值是很好的编程习惯。

输出或记录您在SQL语句中使用的所有变量也可能是个好主意，以确保它们符合您的想法。

而且，虽然现在应该不言而喻，mysql_*函数的使用是discouraged，您应该使用mysqli_* (or PDO)代替。

我只是注意到你的HTML元素层次结构似乎不太合适。你有效：

<head>
    :
</head>
<form>
    :
    <body>
        :

对我来说不凝胶。我不确定form是否有效在之外。根据{{3}}，html元素必须包含：

头部元素后跟一个body元素。

根据用户输入和现有dbms数据更新dbms

1 个答案: