phpのヒアドキュメント内でのJavaScript関数の実行について

Question

現在、最終的にはhtmlでonClickしたらphp関数のlog_bbs()を実行するという処理を行いたいと考えております。

そのため、onClickしたらajaxで非同期処理をおこないlog_form()を実行したいのですが、
そもそもonClickしたらjsの関数を実行してくれません。

phpのヒアドキュメント内に書いているのですが何が間違っているのでしょうか。
お力をお貸しください。

下記にコードを記載致します。

echo <<<EOM
 <script language="javascript" type="text/javascript">
      function logout() {
          alert('OK');//実行されず
          $.ajax({
            type: "POST",
            url: "bbs.php", //PHPを呼び出す
            success: log_bbs(){//成功処理
              $("#bbs_form").html('正常に処理されました');
            },
            error:log_bbs(){//失敗処理
              $("#bbs_form").html('処理に失敗しました');
            }
          });
      }
  </script>
  <form action="{$script}" method="POST" id="bbs_form">
  <input type="button" value="ログアウト" onClick="logout()">
  省略・・・

EOM;


 function log_form(){
 省略・・・
 }

Comments

[shi_ue]

ヒアドキュメント内のscriptタグは、なぜヒアドキュメント内になければならないんですか？
普通に書けばいいのに・・・

[mpyw]

PHPの使い方を間違っている感じですね．正しい使い方をしていれば，基本的にechoやprintを生で書くことはありません．

http://detail.chiebukuro.yahoo.co.jp/qa/question_detail/q11153985376

[kakkydayo]

htmlを見やすくするためにまとめて書いておりまいた。忠告ありがとうございます。美しくないですもんね。
またCertaiNさんありがとうございます。echoを省略できること初めて知りました。

Answer 1

JavaScriptの構文が間違っています。success: log_bbs(){ ... }という記述は不正であり、success: function(){ ... }が適切でしょう。

よって、正しくはこうではないでしょうか？

function logout() {
  alert('OK');//実行されず
  $.ajax({
    type: "POST",
    url: "bbs.php", //PHPを呼び出す
    success: function(){//成功処理
      $("#bbs_form").html('正常に処理されました');
    },
    error:function(){//失敗処理
      $("#bbs_form").html('処理に失敗しました');
    }
  });
}

これを修正してもalert文が動作しない場合、HTMLの属性も誤っているのかもしれません。
onClickではなくonclickと記述してください。

---

※以降の文章は余談となります

jQuery.ajaxのその使い方は既に古くなっています。現在は、done()やfail()を使用した書き方の方が分かりやすいです。

jQuery モダンAjaxな書き方を目指して 〜Deferredを使ったAJAX〜 - Hack Your Design! #jQuery 1.8以上の書き方

また、Ajax送信により特定のPHPの関数を実行したいという説明から不安を感じます。
何故なら、Ajax送信は通常のページ読み込みと同様に、対応するPHPファイルのPHPスクリプト全体が実行されるものであるため、特定の関数だけを実行する場合はURLルーティングなどの記述をおこなう必要があるためです。

全体の実装を見ていないため、的はずれな指摘ならば申し訳ないのですが、
提示したPHPコードがbbs.phpの一部であり、かつ、質問のecho文が条件分岐などで分岐されずに直接記述されている場合、正しくログアウト処理が動作しない可能性があります。

Comments

[kakkydayo]

特定の関数だけを実行したいのですが、URLルーティングの記述とはどういった書き方をすればいいのでしょうか？

[sounisi5011]

書き方には複数ありますし、また要件によっても異なります。
ここでいう「URLルーティング」とは、URLの記述によって処理内容を分岐させる手法を意味しています。なので、URLの構造などが決まっていないとなんとも…

いずれにせよ、余談の範囲であるため質問の本筋から外れること、コード全体が分からないため的はずれである可能性があること、回答内容の記述が多くなる事などが予想されるため、ここのコメントでは示せません。別の質問として投稿されれば（そしてそれを私が見つけられれば）、回答できるかもしれません…

[mpyw]

「Slim3などのマイクロフレームワークを使ってルーティングしましょう」

http://www.slimframework.com/

といえばそれまでなんですが，明らかに回答の範疇を超えているので…（笑