2015-08-26 47 views
-2

我有一個功能連接onclick按鈕。 它通過ajax發送一些數據到另一個php文件。 其實一切工作正常,但我試圖現在添加到'成功'一個簡單的IF語句,這將從外部文件得到真或假。 我想這裏的一些語法可能是錯誤的,因爲錯誤消息說我的主要功能沒有定義。 請問有人可以在這裏找到一些錯誤嗎?語法困難在ajax

主文件(在這裏取得成功:我已經添加的if else語句只)

   $.ajax({ 
       type: 'get', 
       url: '/editcomment', 
       data: {newComment: newComment, 
          id: id, 
         userId: userId}, 

       success:function(data){ 
        if(data.success){ 
         $(".newCommentForm" +id).replaceWith("<span id='" + id + "' class='limitedText'>" + newComment + "</span>"); 
         $(thisId).data("PinComment", newComment); 
        } 
        else { alert('error');}, 

       error:function(){ 
         alert('error'); 
        } 
      });      
     } 

外部文件(我這裏補充返回數組)

public function editComment(){ 

    $userId = Input::get('userId'); 

    if ($userId == Auth::id() or Auth::user()->hasRole("admin")){ 

     $id = Input::get('id'); 
     $newComment = Input::get('newComment'); 
     DB::update('UPDATE `comments` SET f_text = ? WHERE h_id = ?', array($newComment, $id)); 
     return array('success' => true); 

    } else { 
     return array('success' => false); 
    }  
} 
+0

你定的代碼和說明不清晰,完整 –

+0

這是很難解釋整個項目:DI只是有一些語法問題在阿賈克斯 – UberProCoder

+0

即使這個簡單的返回數據,我建議你使用'echo json_encode(array('success'=> false));'並將數據作爲json字符串返回。它使一切都更容易在JavaScript中使用。還要在''.ajax'參數列表中添加'dataType:'json''。 – RiggsFolly

回答

1

您在這裏有一個錯字else { alert('error');},,您需要另一個右大括號來關閉成功功能。

+1

而且'''所以你需要'})' – RiggsFolly

+0

嗯,我已經編輯我的帖子給這個ajax coz更多的代碼大括號已經在那裏。 但是,謝謝你的回答! – UberProCoder

0

擺脫逗號。在else塊試試這個

jAlert('No Success', 'alert box');