jQueryの関数は、PHPスクリプト

Question

を実行しているとき、私は基本的に、あなたは、現時点では

を「アップ投票」ボタンをクリックすると、それはので、私は

clients.php

を持って作っています定義されていないと言っています

<?php 

$clientInfo = "SELECT * FROM Clients ORDER BY Client ASC"; 
$stmt = sqlsrv_query($conn, $clientInfo); 

echo "<div style='width: 100%; display: inline-block;'>"; 

while ($client = sqlsrv_fetch_array($stmt, SQLSRV_FETCH_ASSOC)) 
{ 
    echo "<div class='clientid' style='height: 50px; font-size: 18px; vertical-align: middle; display: inline-block; width: 100%'>" . 

      " 
      <div style='width: 35%;'></div> 

      <div onclick=\"voteUp()\" style='display: inline-block; width: 5%;'>  
       <span style='font-size: 20px;' class='hover-cursor fa fa-chevron-up vote-up'></span> 
      </div>" . 

       "<div class='hover-cursor hvr-underline-reveal' style='width: 20%; display: inline-block;' data-clientid='{$client['ClientID']}'>" . $client['Client'] . "</div>" . 

      "<div onclick=\"voteDown()\" style='display: inline-block; width: 5%;'> 
       <span style='font-size: 20px; text-align: right;' class='hover-cursor fa fa-chevron-down vote-down'></span> 
      </div> 

      <div style='width: 35%;'></div> 

     </div> 
     <br />"; 
} 

echo "</div>"; 

?> 

上記のように、voteUp（）関数のonclickがあります。私のページの一番下にある

私は

<script type="text/javascript" src="scripts/scripts.js"></script> 

とscripts.jsには持って

（私は、PHPファイルを実行するためのdivをクリックする上でのチュートリアルからvoteUp（）のためのビットを得ました）

$(document).ready(function() 
{ 
    $('.pull-me').click(function() 
    { 
     $('.panel1').slideToggle('fast') 
    }); 
    $('.pull-me').click(function() 
    { 
     $('.panel2').slideToggle('fast') 
    }); 
    $('.pull-me').click(function() 
    { 
     $('.panel3').slideToggle('fast') 
    }); 

    $("#selection-box1").change(function() 
    { 
     document.location.href = $(this).val(); 
    }); 

    $(".output").click(function() 
    { 
     var noteid = $(this).data("noteid"); 
     var templatenoteid = $(this).data("templatenoteid"); 
     var variablenoteid = $(this).data("variablenoteid"); 

     if ($(this).data("noteid")) 
     { 
      $("#right-box").load("noteContent.php", {noteid: noteid}); 
     } 

     else if ($(this).data("templatenoteid")) 
     { 
      $("#right-box").load("templateNoteContent.php", {templatenoteid: templatenoteid}); 
     } 

     else 
     { 
      $("#right-box").load("variableContent.php", {variablenoteid: variablenoteid}); 
     } 
    }); 


    var modal = document.getElementById('login-box'); 

    window.onclick = function(event) 
    { 
     if (event.target == modal) 
     { 
      modal.style.display = "none"; 
     } 
    } 

    //This script votes up and down for the client 

    function voteUp() 
    { 
     $.get("voteup.php"); 
     return false; 
    } 

    function voteDown() 
    { 
     $.get("votedown.php"); 
     return false; 
    } 

}); 

そして最後にでvoteup.php

<?php include 'connectionDetails.php'; ?> 

<?php 

if (isset($_POST['clientid'])) 
{ 
    $voteup = "UPDATE Clients SET Pos = (SELECT MAX(Pos) FROM Clients) + 1 WHERE ClientID = " . $_POST['clientid']; 

    $stmt = sqlsrv_query($conn, $voteup); 
} 
else 
{ 
    echo "No ClientID"; 
} 

?> 

私は「にReferenceErrorを：voteUpが定義されていません」と言ってコンソールにエラーが出るのはなぜ私はonclickの=「voteUp（）」

とDIVで投票Upボタンをクリックして

Answer 1

あなたが持っていますメソッドをスコープで指定します。

代わりに、どの場所からでもアクセスできるようにウィンドウにアタッチすることができます。

window.voteUp = function() 
{ 
    $.get("voteup.php"); 
    return false; 
} 

window.voteDown = function() 
{ 
    $.get("votedown.php"); 
    return false; 
} 

Answer 2

Why do I get an error in the console saying "ReferenceError: voteUp is not defined"

を

スコープ。

voteUp()とvoteDown起動ブロックで定義されている（$(document).ready(function() ...の{}内ので、このブロックの外側では使用できませんあなたが外の二つの機能を移動する必要があり

：。

$(document).ready(function() { 
    ... 
}); 

function voteUp() 
{ 
    $.get("voteup.php"); 
    return false; 
} 

function voteDown() 
{ 
    $.get("votedown.php"); 
    return false; 
} 

Answer 3

なぜインラインJSの代わりにjQueryを使用しないのですか？あなたのdivのクラスを与えます。そして、

<div class="voteup" data-clientid="{$client['ClientID']}">...</div> 

、それは非常にクリーンなように代わりonclick()のjQueryのを使用します。

$(document).on('click', '.voteup', function(){ 
    var clientid = $(this).data('clientid'); 
    $.get("voteup.php", { clientid: clientid }); 
    return false; 
}) 

すでに与えて、これはあなたのマークアップクリーナーを維持するのに役立ちますjQueryのを使用しているので、あなたはあなたのマークアップとあなたのJavaScriptにもっと簡単に変更を加えることができます。

さらに、スコープの問題がなくなるため、名前空間を追加する必要がありません。

他の唯一の問題は、$_POST['clientid']をvoteup.phpに渡していないということです。クエリが実行されることはありません。あなたはdiv宣言でdata-client-idを含めることができますが、ことをあなたのPHPでテストを変更する必要があります。

if (isset($_GET['clientid'])) 

あなたがGETリクエストwwithに沿って値を送信されるように。