PHP MySQL подготовленный оператор в подготовленном заявлении

Question

В modelSlikeVrijednost У меня есть ссылка на первичный ключ model. ModelSlikeVrijednost может содержать много изображений (зависит от пользователя). Мне нужно удалить папку на основе modelID.

Пример пути: /home/mainSite/public_html/site/img/1/1/.

Возможно ли это?

Код:

if ($stmt = $mysqli->prepare("SELECT modelID FROM model WHERE proizvodacID='$id'")) {  
    $stmt->execute(); 

    $stmt->bind_result($modelID); 

    while ($stmt->fetch()) { 
     $path="/home/mainSite/public_html/site/img/".$id."/".$modelID."/"; 

     if ($stmt1 = $mysqli->prepare("SELECT modelSlikeVrijednost FROM modelSlike WHERE modelID='$modelID'")) {  
      $stmt1->execute(); 

      $stmt1->bind_result($slike); 

      while ($stmt1->fetch()) { 
       if(is_null($slike)){ 
        rmdir($path); 
       } 
       else{ 
        $slikePath="/home/mainSite/public_html/site/".$slike; 
        if($slikePath!=$path){ 
         unlink($slikePath); 
        } 
        rmdir($path); 
       } 
      } 

      $stmt1->close(); 

     } 
     else { 
      printf("Prepared Statement Error: %s\n", $mysqli->error); 
     } 
    } 

    $stmt->close(); 

} 

Я получаю эту ошибку : Prepared Statement Error: Commands out of sync; you can't run this command now Prepared Statement Error: Commands out of sync; you can't run this command now

Answer 1

Нет вы не можете .... вам нужно перебрать все результаты, закрыть курсор, или использовать отдельное соединение ,

Однако то, что вы пытаетесь сделать, это лучше свершившимся с объединением в любом случае ...

SELECT ms.modelSlikeVrijednost, m.modelID FROM model m, modelSlike ms 
WHERE ms.modelID= m.modelID 
AND m.proizvodacID ='$id' 

Это даст вам всю необходимую информацию в каждой строке.

Однако вы также неправильно используете подготовленные заявления. Вы не должны быть переходящие в PHP переменных непосредственно вы должны быть обязательными их в качестве параметров запроса:

$sql = 'SELECT ms.modelSlikeVrijednost, m.modelID FROM model m, modelSlike ms' 
     .' WHERE ms.modelID= m.modelID' 
     .' AND m.proizvodacID = ?'; 

if($stmt = $mysqli->prepare($sql)) { 

    // bind the $id to the parameter as an integer 
    $stmt->bind_param('i', $id); 

    $stmt->execute(); 

    // bind the fields of the result to the same variables you had before 
    $stmt->bind_result($slike, $modelID); 

    // less prone to error if we only type this manually once :-) 
    $basePath = "/home/mainSite/public_html/site"; 

    while($stmt->fetch()) { 

     $path= $basePath . "/img/".$id."/".$modelID."/"; 
     $slikePath = $basePath . "/" . $slike; 

     if(is_null($slike)){ 
      rmdir($path); 
     } else { 
      if($slikePath!=$path) { 
       unlink($slikePath); 
      } 

      rmdir($path); 
     } 
    } 
} 

Answer 2

Не используйте голую Mysqli API.
Получить себе вспомогательный класс, как safemysql
Тогда ваш код будет

$models = $db->getCol("SELECT modelID FROM model WHERE proizvodacID=?i",$id); 
foreach($models as $modelID) { 
    $path = "/home/mainSite/public_html/site/img/$id/$modelID/"; 
    $sql = "SELECT modelSlikeVrijednost FROM modelSlike WHERE modelID=?i"; 
    $sarr = $db->getCol($sql, $modelID)); 

    foreach($sarr as $silke) { 
     if(!$slike)) { 
      rmdir($path); 
     } else { 
      $slikePath="/home/mainSite/public_html/site/".$slike; 
      if($slikePath!=$path){ 
       unlink($slikePath); 
      } 
       rmdir($path); 
      } 
     } 
    } 
} 

Но да, это лучше сделать это в одном запросе, как prodigitalson сказал:

$sql = "SELECT ms.modelSlikeVrijednost, m.modelID FROM model m, modelSlike ms 
     WHERE ms.modelID= m.modelID AND m.proizvodacID=?i"; 
$sarr = $db->getCol($sql, $id); 
foreach($sarr as $silke) { 
    if(!$slike)) { 
     rmdir($path); 
    } else { 
     $slikePath="/home/mainSite/public_html/site/".$slike; 
     if($slikePath!=$path){ 
      unlink($slikePath); 
     } 
      rmdir($path); 
     } 
    } 
} 

Основная идея это получить ваши данные уже из запроса, а затем использовать его.