循环中的查询花费太多时间来执行

Question

我正在尝试使用 Ajax 和 PHP 更新数据。以下脚本更新数据，但运行时间过长。 PHP

$groupId = $_POST['group_id'];
$Id_array = $_POST['Id'];
$result_array = $_POST['result'];

$data = array();
if(count($_POST['data']) > 0 && !empty ($_POST['data'])){  
    foreach($_POST['data'] as $key => $array){
        $row = array();
        $row['team_id'] = intval($array['team_id']);
        $row['Note'] = strip_tags(trim(strval($array['Note'])));
        $data[$key] = $row;
    }

    for ($i = 0; $i < count($Id_array); $i++) {
        $Id = intval($Id_array[$i]);
        $result = strip_tags(trim(strval($result_array[$i])));

        $sql1 = $db->prepare("UPDATE teams SET result = :result WHERE id = :id ");
        foreach($data as $key => $array){
            $sql1 ->execute(array(':result' => $result, ':id' => $Id));
        }


        $sql2 = $db->prepare("UPDATE teams SET note = :note WHERE team_id = :teamid AND group_id = :group_id");

        foreach($data as $key => $array){
            $sql2->execute(array(':note' => $array['Note'], ':teamid' => $array['team_id'], ':group_id' => $groupId ));
        }

网络时序

Answer 1

我会使用事务而不是在循环中自动提交。 所以，

//PDO::beginTransaction ( void )
$dbh->beginTransaction();
foreach () {
    $smt->exec();
}
//PDO::commit ( void )
$dbh->commit();

此外，由于您将变量绑定在执行函数中，因此您只需要准备一次语句，因此请尝试将任何准备语句排除在循环之外。

这应该有助于减少循环中查询的一些开销。

Answer 2

你不需要这个循环：

    foreach($data as $key => $array){
        $sql1 ->execute(array(':result' => $result, ':id' => $Id));
    }

它会重复更新同一个 ID，因为 $Id 在循环中不会改变。做吧：

    $sql1->execute(array(':result' => $result, ':id' => $Id));

一次。

您还可以通过以下方式获得一些小的改进：

$sql1 = $db->prepare("UPDATE teams SET result = :result WHERE id = :id ");
$sql2 = $db->prepare("UPDATE teams SET note = :note WHERE team_id = :teamid AND group_id = :group_id");

只有一次，在任何循环之前。

另一个问题是你有这个循环：

    foreach($data as $key => $array){
        $sql2->execute(array(':note' => $array['Note'], ':teamid' => $array['team_id'], ':group_id' => $groupId ));
    }

在for() 循环内，但它不使用任何在循环中每次都会改变的变量。因此，它正在为$Id_array 中的每个 ID 重新执行所有相同的查询。

把它排除在循环之外。

进行所有这些更改后，代码现在如下所示：

$groupId = $_POST['group_id'];
$Id_array = $_POST['Id'];
$result_array = $_POST['result'];

$sql1 = $db->prepare("UPDATE teams SET result = :result WHERE id = :id ");
$sql2 = $db->prepare("UPDATE teams SET note = :note WHERE team_id = :teamid AND group_id = :group_id");

$data = array();
if(count($_POST['data']) > 0 && !empty ($_POST['data'])){  
    foreach($_POST['data'] as $key => $array){
        $row = array();
        $row['team_id'] = intval($array['team_id']);
        $row['Note'] = strip_tags(trim(strval($array['Note'])));
        $data[$key] = $row;
    }

    foreach ($Id_array as $i => $Id) {
        $Id = intval($Id);
        $result = strip_tags(trim(strval($result_array[$i])));
        $sql1 ->execute(array(':result' => $result, ':id' => $Id));
    }

    foreach($data as $key => $array){
        $sql2->execute(array(':note' => $array['Note'], ':teamid' => $array['team_id'], ':group_id' => $groupId ));
    }
}