2017-04-24 24 views
0

我正在嘗試創建一個名字/姓氏改變的用戶面板。MySQL/PHP/jQuery - 問題與查詢

看起來好像$ajax表格處理做得很好,因爲我可以console.log(data)並參見{fname: "Damian", lname: "Doman", id: "20"}

此外,php類似乎運行得太多,因爲我發送表單後,我得到print 'Your name has been changed! To go back, please click <a href="userpage.php">here</a>';,因爲它的意思是。

最後,查詢本身(UPDATE members SET fname = ?, lname = ? WHERE id = ?)在SQL控制檯中使用值而不是問號手動觸發時也可以完美無缺。

儘管如此,整個腳本不會工作,我得到的消息說它是成功的,但仍然不會得到UPDATE

HTML:

<div class="changename-form-wrapper"> 
    <h1>Define your first and last names here:</h1> 
    <div class="form-group"> 
     <input type="text" name="fname" id="fname" tabindex="3" class="form-control ui-autocomplete-input" placeholder="First name..." value=""> 
    </div> 
    <div class="form-group"> 
     <input type="text" name="lname" id="lname" tabindex="4" class="form-control ui-autocomplete-input" placeholder="Last name..."> 
    </div> 
    <div class="form-group"> 
     <input type="text" name="id" id="id" tabindex="4" class="form-control ui-autocomplete-input" style="display:none;" value="<?=$_SESSION['user']['id']?>"> 
    </div> 
    <div class="form-group"> 
     <div class="row"> 
      <div class="col-sm-6 col-sm-offset-3"> 
       <input type="button" name="name-submit" id="name-submit" tabindex="4" class="form-control btn btn-login" value="Apply"> 
      </div> 
     </div> 
    </div> 
</div> 
<div class="alert-space"> 
    <div class="ui-widget" style="display:none;"> 
     <div class="ui-state-error ui-corner-all"> 
      <div class="p"><span class="ui-icon ui-icon-alert"></span></div><div class="p"><div class="ui-state-error-text"></div></div> 
     </div> 
    </div> 
</div> 

的jQuery/AJAX:

$("#name-submit").click(function(){ 
       var data = { "fname": $('#fname').val(), "lname": $('#lname').val(), "id": $('#id').val() }; 
       if($("#fname").val() != "" && $("#lname").val() != ""){ 
        $.ajax({ 
         method: "POST", 
         url: 'changename.php', 
         data: data, 
        }).done(function(msg) { 
         console.log(data); 
         if(msg !== ""){ 
          $(".ui-widget").show(); 
          $(".ui-state-error-text").html(msg); 
         }else{ 
          window.location = "userpage.php"; 
         } 
        }); 
       }else{ 
        $(".ui-widget").show(); 
        $(".ui-state-error-text").html("<strong>Error:</strong> Please, fill in both the first and last name."); 
       } 
      }); 

CHANGENAME.PHP表單處理程序:

<?php 
    require_once '../../class/user.php'; 
    require_once '../../config.php'; 

    $id = filter_input(INPUT_POST, 'id', FILTER_SANITIZE_STRING); 
    $fname = filter_input(INPUT_POST, 'fname', FILTER_SANITIZE_STRING); 
    $lname = filter_input(INPUT_POST, 'lname', FILTER_SANITIZE_STRING); 

    if($user->userUpdate($id, $fname, $lname)) { 
     print 'Your name has been changed! To go back, please click <a href="userpage.php">here</a>'; 
     die; 
    } else { 
     $user->printMsg(); 
     die; 
    }; 

最後的userUpdate($id, $fname, $lname)類代碼:

public function userUpdate($id,$fname,$lname){ 
     $pdo = $this->pdo; 
     if(isset($id) && isset($fname) && isset($lname)){ 
      $stmt = $pdo->prepare('UPDATE members SET fname = ?, lname = ? WHERE id = ?'); 
      if($stmt->execute([$id,$fname,$lname])){ 
       return true; 
      }else{ 
       $this->msg = 'User information change failed.'; 
       return false; 
      } 
     }else{ 
      $this->msg = 'Provide a valid data.'; 
      return false; 
     } 
    } 

我在做什麼錯?目前我在控制檯中沒有任何錯誤。

+0

爲什麼你不把表單放在jquery中? –

+2

'if($ stmt-> execute([$ id,$ fname,$ lname])){'看起來像那些查詢的錯誤順序。 –

+0

您不會收到任何錯誤,因爲查詢將正確執行,它只是不會更新任何內容。 –

回答

1

您給予execute的數組的值爲錯誤順序,應該是$fname, $lname, $id以匹配查詢中的順序。

但恕我直言,最好使用命名的佔位符,而不是有序的佔位符。那麼順序並不重要。

$stmt = $pdo->prepare('UPDATE members SET fname = :fname, lname = :name WHERE id = :id'); 
$stmt->execute([':id' => $id, ':fname' => $fname, ':lname' => $lname]); 
+0

感謝您的迴應!就像我在評論部分寫的那樣 - 簡直不敢相信。所以讓我把它正確的 - 在'execute'中的順序必須與這些名稱/變量出現在'MySQL query'中的順序相同?這個佔位符的詭計在未來的事情上似乎也爲我節省了很多麻煩,感謝它! –