2014-12-07 45 views
0

下运行MySQL查询我不知道是否有人在这里可以告诉我。我有一个页面,我正在更新数据库中的信息,从前一页我发布任何新图像(带有标题),我有一个用于删除图像的复选框。在处理页面,我有以下代码:在特定的顺序

<?php 
    session_start(); 
    $dbhost = 'removed'; 
    $dbuser = 'removed'; 
    $dbpass = 'removed'; 
    $dbname = 'removed'; 

    function reArrayFiles($file_post) { 
     $file_ary = array(); 
     $file_count = count($file_post['name']); 
     $file_keys = array_keys($file_post); 
     for ($i=0; $i<$file_count; $i++) { 
      foreach ($file_keys as $key) { 
       $file_ary[$i][$key] = $file_post[$key][$i]; 
      } 
     } 
     return $file_ary; 
    } 
    try { 
     $dbo = new PDO('mysql:host=localhost;dbname='.$dbname, $dbuser, $dbpass); 
    }catch (PDOException $e) { 
     print "Error!: " . $e->getMessage() . "<br/>"; 
    die(); 
    } 

    $query="SELECT * FROM `soldier_info` WHERE `soldier_id` = " . $_POST['soldier_id'] ; 
    foreach ($dbo->query($query) as $row) { 
     $soldier_pre_images = explode(",", $row['soldier_images']); 
    } 

    if($_POST['deletephoto']){ 
     $imagestodelete=$row['soldier_images']; 
     $captionstodelete=$row['soldier_images_captions']; 
     foreach($_POST['deletephoto'] as $todelete){ 
      $deleteexplode = explode("--",$todelete); 
      echo $deleteexplode; 
      $deleteexplodepath = $deleteexplode[0]; 
      $deletecaption=$deleteexplode[1]; 
      $imagesafterdelete= str_replace($deleteexplodepath.",","",$imagestodelete); 
      echo $imagesafterdelete; 
      $captionsafterdelete = str_replace($deletecaption."--","",$captionstodelete); 
      echo $captionsafterdelete; 
      $unlinkpath = str_replace('site url','..',$deleteexplodepath); 
      unlink($unlinkpath); 
     } 

     try { 
      $dbo = new PDO('mysql:host=localhost;dbname='.$dbname, $dbuser, $dbpass); 
      $dbo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 
      $q = $dbo->prepare("UPDATE `soldier_info` SET `soldier_images`= :image, `soldier_images_captions`= :captions WHERE `soldier_id`= :id"); 
      $q->execute(array(":image" => $imagesafterdelete, ":captions" => $captionsafterdelete, ":id" => $_POST['soldier_id'])); 
     }catch (PDOException $e) { 
      $db_error = "Error!: " . $e->getMessage() . "<br/>"; 
      die(); 
     } 
    } 

    if ($_FILES['photo']) { 
     $file_ary = reArrayFiles($_FILES['photo']); 
     $capcount = 0; 
     foreach ($file_ary as $file) { 
      $allowedExts = array("gif", "jpeg", "jpg", "png"); 
      $temp = explode(".", $file["name"]); 
      $extension = end($temp); 
      if ((($file["type"] == "image/gif")||($file["type"] == "image/jpeg")||($file["type"] == "image/jpg")||($file["type"] == "image/pjpeg")||($file["type"] == "image/x-png")||($file["type"] == "image/png"))&&($file["size"] < 9000000) && in_array(strtolower($extension), $allowedExts)) { 
        if ($file["error"] > 0) { 
        $file_error = "Return Code: " . $file["error"] . "<br>"; 
        } else { 
        $newfilename = "../assets/uploads/images/" . $_POST['soldier_id'] . "/" . $file["name"]; 
        $file_fullname = "site urlv/assets/uploads/images/" . $_POST['soldier_id'] . "/" . $file["name"]; 
        $new_caption = $_POST['image_captions'][$capcount]; 
        if (file_exists($newfilename)) { 
         $file_exists = $file["name"] . " already exists. "; 
        } else { 
         if(is_dir("../assets/uploads/images/" . $_POST['soldier_id'])){ 
          move_uploaded_file($file["tmp_name"],$newfilename); 
          if (strlen($uploaded_images)>1){ 
          $uploaded_images = $uploaded_images . ",". $file_fullname; 
          }else{ 
          $uploaded_images = $file_fullname; 
          } 
          if (strlen($uploaded_captions)>1){ 
          $uploaded_captions = $uploaded_captions . "--". $new_caption; 
          }else{ 
          $uploaded_captions = $new_caption; 
          } 
         }else{ 
          mkdir("../assets/uploads/images/" . $_POST['soldier_id'], 0777, true);  
          move_uploaded_file($file["tmp_name"],$newfilename); 
          if (strlen($uploaded_images)>1){ 
          $uploaded_images = $uploaded_images . ",". $file_fullname; 
          }else{ 
          $uploaded_images = $file_fullname; 
          } 
          if (strlen($uploaded_captions)>1){ 
          $uploaded_captions = $uploaded_captions . "--". $new_caption; 
          }else{ 
          $uploaded_captions = $new_caption; 
          } 
         } 
        } 
        if (strlen($row['soldier_images'])>1){ 
        $uploaded_images = $row['soldier_images'] . ",". $uploaded_images; 
        }else{ 
        $uploaded_images = $uploaded_images; 
        } 
        } 
       } else { 
        $file_invalid = "The file is not a jpg/gif/png file, or is larger than 20mb. Please try again."; 
       } 
       $capcount = $capcount+1; 
     } 
     try { 
      $dbo = new PDO('mysql:host=localhost;dbname='.$dbname, $dbuser, $dbpass); 
      $dbo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 
      $q = $dbo->prepare("UPDATE `soldier_info` SET `soldier_images`= :image, `soldier_images_captions`= :captions WHERE `soldier_id`= :id"); 
      $q->execute(array(":image" => $uploaded_images, ":captions" => $uploaded_captions, ":id" => $_POST['soldier_id'])); 
     }catch (PDOException $e) { 
      $db_error = "Error!: " . $e->getMessage() . "<br/>"; 
      die(); 
     } 
    } 

    $query="SELECT * FROM `soldier_info` WHERE `soldier_id` = " . $_POST['soldier_id'] ; 
    foreach ($dbo->query($query) as $row) { 
     $firstname = $row['firstname']; 
     $surname = $row['surname']; 
     $yearofbirth = $row['yearofbirth']; 
     $dateofdeath = date("d/m/Y", strtotime($row['dateofdeath'])); 
     $ageatdeath = $row['ageatdeath']; 
     $regiment = $row['regiment']; 
     $battallion_brigade = $row['battallion_brigade']; 
     $Coybty = $row['Coy/bty']; 
     $rank = $row['rank']; 
     $solider_number = $row['soldier_number']; 
     $Next_of_Kin = $row['Next_of_Kin']; 
     $CWGC = $row['CWGC']; 
     $Place_Died = $row['Place_Died']; 
     $Connection_to_Wymeswold = $row['Connection_to_Wymeswold']; 
     $Cemetery = $row['Cemetery']; 
     $LRoH = $row['LRoH']; 
     $Grave = $row['Grave']; 
     $Memorial = $row['Memorial']; 
     $Pier_Face = $row['Pier_Face']; 
     $Other_Mem = $row['Other_Mem']; 
     $Other_Details = $row['Other_Details']; 
     $soldier_images = explode(",", $row['soldier_images']); 
     $soldier_images_captions = explode("--", $row['soldier_images_captions']); 
    } 
    $dbo = null; 
    ?> 

这不是一个活的网站现在是在测试中的/ dev。但是我看到的问题是,所有的疑问都在运行,但页面加载,就好像选择了首先运行(通过我的意思是返回的记录是那些从作为处理更新之前,我可以强制使用选择所以它获得最新的记录呢?

+0

你的意思是它选择要运行多久? – krzysiej 2014-12-07 10:44:36

+0

请使用准备好的语句:http://php.net/manual/en/mysqli.quickstart.prepared-statements.php其他http://xkcd.com/327/ – idstam 2014-12-07 11:01:18

+0

@krzysiej我要运行这个最后:$查询=“SELECT * FROM'soldier_info' WHERE'soldier_id' =”。 $ _POST ['soldier_id']; – Stefan 2014-12-07 11:08:04

回答

0

我想你的最后选择,因为你正在运行在不同的连接查询。因此,他们将生活在孤立的交易“原始”的数据去年运行,这也许不是正确提交时的最后选择运行。

如果你在同一个PDO对象运行它们,我认为它会工作。

(如果你把POST参数,并将它们连接成SQ L你的服务器将被黑客入侵。这只是一个时间的问题,你有这个网址给大家看。)

+0

谢谢你的建议,我会去承担工作对我来说现在看一看,我在看今天的消毒(一旦我得到这个执行顺序排序) – Stefan 2014-12-07 11:14:20