我的查询有问题吗?

时间:2012-06-04 20:41:57

标签: php mysql sql database

我正在开发一个包含3个表的数据库,其中一些表有重叠的信息。用户可以通过我正在创建的Web应用程序更新每个表中的几列。但是......有一个问题。不确定它是什么,但我的更新没有发生。我想知道我的查询是否有问题。 (实际上,经过调试,我确信有)。

    if (empty($errors)) {   
    $query1 = "UPDATE owner SET
            name = '{$name}'
        WHERE ownerId= '{$ownerId}'";
    $query1_result = mysql_query($query1);
    if (mysql_affected_rows()==1) { 
        $query2 = "UPDATE queue_acl SET
                    date_expires = '{$date_expires}' 
                WHERE user_id='{$ownerId}'";
        $query2_result = mysql_query($query2);
        if (mysql_affected_rows()==2) {
            $query3 = "UPDATE ownerOrganization SET
                    orgId = {$orgId}
                WHERE ownerId = '{$ownerId}'";
            $query3_result = mysql_query($query3);
            if (mysql_affected_rows()==3) {
                    $_SESSION['name'] = $name;
                    $_SESSION['updates_occurred'] = true;
            }
        }
    }

对不起,如果它是微不足道的;我之前从未使用过多个表。

2 个答案:

答案 0 :(得分:1)

以您的方式更新表格并不是一个好习惯。如果更新以某种方式相关,您可能需要考虑创建事务。事务确保执行所有更新(如果没有,则完成回滚(这意味着不会执行更新)):

// disable autocommit 
mysqli_autocommit($dblink, FALSE);

// queries
$query1 = mysqli_query($dblink, "UPDATE owner SET name = '{$name}' WHERE ownerId= {$ownerId}'");
$query2 = mysqli_query($dblink, "UPDATE queue_acl SET date_expires = '{$date_expires}' WHERE user_id='{$ownerId}'");
$query3 = mysqli_query($dblink, "UPDATE ownerOrganization SET orgId = {$orgId} WHERE ownerId = '{$ownerId}'");

if($query1 && $query2 && $query3)
{
   mysqli_commit($dblink);
   $_SESSION['name'] = $name;
   $_SESSION['updates_occurred'] = true;
}
else
   mysqli_rollback($dblink);

我没有测试过,但它认为它应该可行。此外,您应该查看mysqli或预处理语句,因为不推荐使用mysql_。

答案 1 :(得分:0)

第一个问题可能与范围有关。你的if条件错了吗?

If (result ==1)
{
  if(result == 2)
  { 
   ...
  }
}

因此,如果您的第一个结果超过1,那么将跳过所有内部条件。

如果我理解应该是:

if(result ==1)
{
}
elseif(result ==2)
{
}
...(other conditions)...
else
{ 
}

如果您的结果超出预期,我会建议您使用基本案例。

第二个问题可能是您引用除orgId = {$ orgId}之外的所有数据。这应该是orgId ='{$ orgId}',如果id是一些随机字符串,那么不引用会导致问题。

最后一个问题是检查ownerId。如果由于某种原因它是空白的,那么你的查询将失败,因为id = 0(假设你有自动增量)将永远不会成立。放一个if(!empty(ownerId)条件。