質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.35%

  • PHP

    25496questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • jQuery

    8757questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

  • Ajax

    1407questions

    Ajaxとは、Webブラウザ内で搭載されているJavaScriptのHTTP通信機能を使って非同期通信を利用し、インターフェイスの構築などを行う技術の総称です。XMLドキュメントを指定したURLから読み込み、画面描画やユーザの操作などと並行してサーバと非同期に通信するWebアプリケーションを実現することができます。

  • Laravel

    1272questions

    LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

  • セキュリティー

    555questions

    このタグは、コンピューターシステムの安全性やデータの機密性に関連したトピックの為に使われます。

非同期いいね実装 HTMLの切り替えとJSONの受け渡しについて

解決済

回答 1

投稿

  • 評価
  • クリップ 3
  • VIEW 283

osheton

score 7

非同期のいいね機能を jQuery を使って実装しました。
サーバーサイドは PHP(Laravel) です。 

Twitterの「いいね」のように、非同期で処理してボタンが変わる仕組みを実装しました。

発生した問題

  • 時々エラーが出て動かなくなる
  • HTMLのクラス名を innerHTML で動的生成している部分など、キュリティ的に問題があるかもしれない

コメントに対して「いいね」できる仕組みです。
ボタンの変化は、Font Awesome のアイコン切り替えで表現します。
つまりHTMLの要素(クラス名)を動的に変化させます。

<i class="far fa-heart"></i> <!-- 未いいねの際表示 -->
<i class="fas fa-heart"></i> <!-- いいね済みの際表示 -->


切り替えるのは far と fas の部分だけです。

また「いいね」された数も表示します。

PHP側からビューに渡すものは以下になります。

<画面表示時>

  • ユーザーが既に「いいね」しているかを保持した配列 

<ボタンが押された時(JSONで渡す)>

  • Font Awesome のクラス名(ボタンを変化させるため)
  • 更新された「いいね」の数

ほぼ期待通りの動作をしますが、時々以下のエラーで動かなくなります。
Chromeではぼエラーはでませんが、Edgeだと10回に1回ほど出ます。

Uncaught SyntaxError: Unexpected end of JSON input // Chrome 
JSON.parse Error: Unexpected input // Edge


JSON.parse() に JSON以外の値を渡していることが原因だと思います。
エラー時に、Ajaxでサーバーから返された値を console.log してみると、
空文字でした。しかしPHPからはjson_encodeした値しか返していないので、なぜ空文字になるのかわかりません。

以下実装したコードです。

コメント一覧をforeachで回し、その中にボタンを設置します。
POST時、コメントIDを判別するのにカスタムデータ属性を利用します。

// @foreach($comments as $comment) の中

@if(is_null($already[$comment->id]))  //まだいいねしてなければ
  <button type="button" class="fav" data-fav="{{$comment->id}}">
      <i class="far fa-heart"></i> 
      {{$comment->cnt}} //いいね数
  </button>
@else  //既にいいねしていれば
  <button type="button" class="fav" data-fav="{{$comment->id}}">
      <i class="fas fa-heart"></i>  
      {{$comment->cnt}}  //いいね数
  </button>
@endif
//「いいね」したコメントのIDを保持
let comment_id = 0;

//0以上の自然数かチェックする関数
const isNum = function(v) 
{
 return ((typeof v === 'number') && (isFinite(v)) && v >= 0);
};

//「いいね」ボタンがクリックされた時
$('.fav').on('click', function(event) {
    event.target.disabled = true; //処理が終了するまでボタンを無効化
    event.preventDefault();
    comment_id = event.target.dataset.fav;

    $.ajax({
      type: 'POST',
      url: '',
      dataType: 'text',
      data: {
        comment_id: comment_id,
      }
    }).done(function (data) { 
       //console.log(data); {"name":"far","cnt":13} エラー時は空文字
      let d = JSON.parse(data);
      if((d.name==='fas' || d.name==='far') && isNum(d.cnt))
      {
         event.target.innerHTML =  
          `<i class=\"${d.name} fa-heart\"></i> ${d.cnt}`;
      }
      event.target.disabled = false;  //ボタンを有効化
    });
});


「いいね」がPOSTされた際のアクション

$comment_id = $request->input('comment_id');
/*
    comment_id が自分自身のコメントでないことを確認
    データベース更新処理
    $name : 新たな Font Awesomeのクラス名(far, fas)
    $cnt : 「いいね」総数
*/
if(($name==='far' || $name==='fas') && is_int($cnt) && $cnt >= 0)
{
   return json_encode(["name" => $name, "cnt" => $cnt]);
}
//正常でない場合でもJSON形式で返す
return json_encode(['name' => 'error', 'cnt' => -1]);


CSRF対策

$.ajaxSetup({
    headers: {
        'X-CSRF-TOKEN': $('meta[name="csrf-token"]').attr('content')
    }
});

とりあえず動くように実装したので、雑なやり方だと思います。

質問をまとめると

  • 時々空文字を返してエラーになるのを防ぐにはどうすべきか。

  • セキュリティ的に問題はないか。
    (クラス名の動的生成、カスタムデータによるid判別、AjaxでPOSTする値の偽装対策等)

その他「そのやり方はよくない」等あればご指摘ください。

ご回答よろしくお願いします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

時々空文字を返してエラーになるのを防ぐにはどうすべきか。 

  • 空文字になる原因を特定する
  • エラーをただ防ぐ
if ( !data ) return;
let d = JSON.parse(data);

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/03/27 18:58

    ありがとうございました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.35%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る

  • PHP

    25496questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • jQuery

    8757questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

  • Ajax

    1407questions

    Ajaxとは、Webブラウザ内で搭載されているJavaScriptのHTTP通信機能を使って非同期通信を利用し、インターフェイスの構築などを行う技術の総称です。XMLドキュメントを指定したURLから読み込み、画面描画やユーザの操作などと並行してサーバと非同期に通信するWebアプリケーションを実現することができます。

  • Laravel

    1272questions

    LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

  • セキュリティー

    555questions

    このタグは、コンピューターシステムの安全性やデータの機密性に関連したトピックの為に使われます。