PHP PDO MySQL struktura Kod transakcji

Question

Próbuję skonfigurować swoją pierwszą transakcję w MySQL za pomocą PHP/PDO ...

mam tylko krótkie pytanie, co jest najlepszym sposobem, aby ustalić, czy poprzednie zapytanie było udane czy nie? Oto, co mam teraz, ale wolałbym znaleźć sposób na przetestowanie zapytania za pomocą instrukcji if.

To jest prawie makiety kod, aby spróbować uzyskać działający model .. Wiem, że wyniki $ nie jest skuteczne testowanie, jeśli coś było dobre lub złe .. mam go tam bardziej jako miejsce na prawdziwą ofertę kiedy przyjdzie czas ..

if ($_POST['groupID'] && is_numeric($_POST['groupID'])) { 
    $sql = "SET AUTOCOMMIT=0"; 
    $dbs = $dbo->prepare($sql); 
    $dbs->execute(); 

    $sql = "START TRANSACTION"; 
    $dbs = $dbo->prepare($sql); 
    $dbs->execute(); 

    $sql = "DELETE FROM users_priveleges WHERE GroupID=:groupID"; 
    $dbs = $dbo->prepare($sql); 
    $dbs->bindParam(":groupID", $_POST['groupID'], PDO::PARAM_INT); 
    $dbs->execute(); 

    try { 
     $sql = "DELETE FROM groups WHERE GroupID=:groupID LIMIT 1"; 
     $dbs = $dbo->prepare($sql); 
     $dbs->bindParam(":groupID", $_POST['groupID'], PDO::PARAM_INT); 
     $dbs->execute(); 

     $results["error"] = null; 
     $results["success"] = true; 

     try { 
      $sql = "DELETE FROM users WHERE Group=:groupID"; 
      $dbs = $dbo->prepare($sql); 
      $dbs->bindParam(":groupID", $_POST['groupID'], PDO::PARAM_INT); 
      $dbs->execute(); 

      $results["error"] = null; 
      $results["success"] = true; 

      $sql = "COMMIT"; 
      $dbs = $dbo->prepare($sql); 
      $dbs->execute(); 
     } 
     catch (PDOException $e) { 
      $sql = "ROLLBACK"; 
      $dbs = $dbo->prepare($sql); 
      $dbs->execute(); 

      $results["error"] = "Could not delete associated users! $e"; 
      $results["success"] = false; 
     } 
    } 
    catch (PDOException $e) 
    { 
     $sql = "ROLLBACK"; 
     $dbs = $dbo->prepare($sql); 
     $dbs->execute(); 

     $results["error"] = "COULD NOT REMOVE GROUP! $e"; 
     $results["success"] = false; 
    } 
} 

Answer 1

PDO komunikat za execute() Zwraca TRUE w przypadku sukcesu, FALSE w przypadku porażki, dzięki czemu można przetestować zwracanej wartości z poprzedniego execute() w if.

$pdo_result = $dbs->execute(); 
if ($pdo_result) { 
    // handle success 
} else { 
    // handle failure 
    // you can get error info with $dbs->errorInfo(); 
} 

Powiedział, że jak słusznie @Bill Kerwin się (w swojej odpowiedzi, że jestem całkowicie upvoting bo to dokładnie poprawne), lepiej byłoby użyć PDO::beginTransaction(), PDO::commit() i PDO::rollback().

Answer 2

Nie będę przygotowywać & do wykonywania transakcji transakcji. Użyłbym PDO::beginTransaction(), PDO::commit() i PDO::rollback().

PDO :: przygotować() i PDO :: execute() zwróci FALSE, jeśli wystąpi błąd, lub też wyrzucą wyjątek PDOException, jeśli jesteś setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION).

W programie obsługi wyjątku należy sprawdzić PDO::errorInfo() i podać charakter błędu. Najlepszą praktyką jest rejestrowanie surowych informacji o błędzie, ale przekazanie użytkownikowi bardziej przyjaznego komunikatu.

Nie powtarzaj dosłownego komunikatu o błędzie w interfejsie użytkownika - może to dać użytkownikowi niewłaściwą wiedzę na temat zapytania SQL i schematu.

Answer 3

Kilka uwag: Nie używaj bindParam() chyba zastosować procedurę, która modyfikuje wartość parametru Dlatego use bindValue(). bindParam() przyjmuje wartość argumentu jako zmienną referencyjną. Oznacza to, że nie możesz wykonać $stmt->bindParam(':num', 1, PDO::PARAM_INT); - powoduje to błąd. Ponadto PDO ma swoje własne funkcje do kontrolowania transakcji, nie trzeba wykonywać kwerend ręcznie.

przepisałem kod nieznacznie rzucić światło na sposób PDO można stosować:

if($_POST['groupID'] && is_numeric($_POST['groupID'])) 
{ 
    // List the SQL strings that you want to use 
    $sql['privileges'] = "DELETE FROM users_priveleges WHERE GroupID=:groupID"; 
    $sql['groups']  = "DELETE FROM groups WHERE GroupID=:groupID"; // You don't need LIMIT 1, GroupID should be unique (primary) so it's controlled by the DB 
    $sql['users']  = "DELETE FROM users WHERE Group=:groupID"; 

    // Start the transaction. PDO turns autocommit mode off depending on the driver, you don't need to implicitly say you want it off 
    $pdo->beginTransaction(); 

    try 
    { 
     // Prepare the statements 
     foreach($sql as $stmt_name => &$sql_command) 
     { 
      $stmt[$stmt_name] = $pdo->prepare($sql_command); 
     } 

     // Delete the privileges 
     $stmt['privileges']->bindValue(':groupID', $_POST['groupID'], PDO::PARAM_INT); 
     $stmt['privileges']->execute(); 

     // Delete the group 
     $stmt['groups']->bindValue(":groupID", $_POST['groupID'], PDO::PARAM_INT); 
     $stmt['groups']->execute(); 

     // Delete the user 
     $stmt['users']->bindParam(":groupID", $_POST['groupID'], PDO::PARAM_INT); 
     $stmt['users']->execute(); 

     $pdo->commit();  
    } 
    catch(PDOException $e) 
    { 
     $pdo->rollBack(); 

     // Report errors 
    }  
}