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

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

ただいまの
回答率

91.00%

  • PHP

    17783questions

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

  • JavaScript

    13835questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • jQuery

    5824questions

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

JavaScriptのforループで回してるのにループが回っていない?

受付中

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 382

Yashichi

score 25

追記

console.log(response);の値を追記しておきます。

{rows: Array(10)}
rows:Array(10)
0:{0: "217", 1: "test", 2: "rest", 3: null, 4: null, id: "217", title: "test", content: "rest", complement: null, deadline: null}
1:{0: "216", 1: "", 2: "", 3: null, 4: null, id: "216", title: "", content: "", complement: null, deadline: null}
2:{0: "215", 1: "", 2: "", 3: null, 4: null, id: "215", title: "", content: "", complement: null, deadline: null}
3:{0: "214", 1: "sentence", 2: "root", 3: null, 4: null, id: "214", title: "sentence", content: "root", complement: null, deadline: null}
4:{0: "213", 1: "clear", 2: "content", 3: null, 4: null, id: "213", title: "clear", content: "content", complement: null, deadline: null}
5:{0: "212", 1: "", 2: "", 3: null, 4: null, id: "212", title: "", content: "", complement: null, deadline: null}
6:{0: "211", 1: "", 2: "", 3: null, 4: null, id: "211", title: "", content: "", complement: null, deadline: null}
7:{0: "210", 1: "", 2: "", 3: null, 4: null, id: "210", title: "", content: "", complement: null, deadline: null}
8:{0: "209", 1: "", 2: "", 3: null, 4: null, id: "209", title: "", content: "", complement: null, deadline: null}
9:{0: "208", 1: "", 2: "", 3: null, 4: null, id: "208", title: "", content: "", complement: null, deadline: null}
length:10
__proto__:Array(0)
__proto__:Object

前提・実現したいこと

PHPとjQueryを利用してAjaxでのToDoリストを作成しています。

現在、機能としてCRUDのUの部分(Update)を実装しようとしています。

実現したいこととしては、選んだ(ダブルクリックしたもの)タスクをHTMLのinput要素に突っ込むというものです。

発生している問題・エラーメッセージ

実現したいことを妨げていることとして、選んだタスクが正しく認識されていない、というのが挙げられます。
A,B,Cというタスクがあって、Bを選んでいるのにAが選ばれていると認識されているという状況です。

他にも、一度選んだタスクが最優先(?)になってしまって、別のタスクを選んでも最初に選んだタスクの内容から切り替わらないという現象が起きています。

該当のソースコード

表画面

            // 編集したいところをダブルクリックしたらイベント発動
            $(document).on('dblclick', ".labeling", function(){

                $('#editTitle').val("");
                $('#editContent').val("");

                // Ajax
                $.ajax({
                    url: 'php/read.php',
                }) .done(function(response){
                    console.log("成功してるよ");
                    var $everyResponse = response.rows;
                    var $updateTitle = '';
                    var $updateContent = '';

                    for(let i = 0; i < $everyResponse.length; i++) {
                        console.log($everyResponse[i].id);

                        $updateTitle += $everyResponse[i].title
                        $updateContent += $everyResponse[i].content

             //input要素へ値を突っ込む
                        $('#editTitle').val($updateTitle);
                        $('#editContent').val($updateContent);

                        // ループを中断
                        return false;

                    }

                }) .fail(function(){
                    console.log("失敗なのよね");
                });
            });


API側(read.php)

<?php
        // header関数より前にechoなどを使ってはいけない
        header('content-type: application/json; charset=utf-8');
        // データベースに接続
        $dsn = 'mysql:host=localhost;dbname=todolist;charset=utf8';
        $user = 'secret';
        $passwd = 'secret';

        try {
            $db = new PDO($dsn, $user, $passwd);
            $db->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);

            // プリペアドステートメント発行(最新の10件を取得)
            $statement = $db->prepare("
                SELECT * FROM list ORDER BY id DESC LIMIT 10;
                ");
// print_r($db->errorInfo());

            $statement->execute();
            $rows = $statement->fetchAll();
            // var_dump($rows);
            echo json_encode(compact('rows'));

        } catch (PDOException $e) {
            echo "Error :" . $e->getMessage();
        }

試したこと

SELECT文でデータベースの中身を取ってくる処理を書いたPHPへアクセスするAjaxの式を書き、
返ってきた値response.rowsの長さ分だけforでループを回します。

ループの中で変数にタスクのタイトルと内容を代入します。

inputに突っ込むときに値が連結されないようにするためにreturn falseをする、
という記述をしております。

補足情報(言語/FW/ツール等のバージョンなど)

PHP5.3
CentOS6.9

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 3

+4

$.ajax()がurlに対してパラメータを渡していませんが大丈夫ですか?
とりあえずはdoneのコールバックの中で以下を確認してみてください

console.log(response)

またajaxが受けるデータもdataTypeを指定していないのでtextでうけていませんか?
もしきちんとしたapiならjsonで吐き出してくれそうな気がするので

dataType:'json',


ただすでに console.log($everyResponse[i].id);が正しい値を返しているなら
不要な作業かもしれません

 追記

  $(document).on('dblclick', ".labeling", function(){
    $('#editTitle').val("");
    $('#editContent').val("");
    $.ajax({
    url: 'php/read.php',
    }) .done(function(response){
      console.log("成功してるよ");
      var everyResponse = response.rows;
      var updateTitle = '';
      var updateContent = '';
      for(var i = 0; i < $everyResponse.length; i++) {
        updateTitle += everyResponse[i].title;
        updateContent += everyResponse[i].content;
      }
      $('#editTitle').val($updateTitle);
      $('#editContent').val($updateContent);
    });
  });
});
<div class="labeling">dblclick!</div>
<textarea id="editTitle"></textarea>
<textarea id="editContent"></textarea>

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/15 17:29

    現状、"console.log($everyResponse[i].id);" の値は最初にダブルクリックした値とずっと同じ値を返してきます

    キャンセル

  • 2017/11/15 17:58

    jsonで受け取れているなら単純な話
    forの中でreturn falseしているので
    i=0だけしか参照していないでしょう

    キャンセル

  • 2017/11/15 18:59

    おっしゃる通りです..

    キャンセル

  • 2017/11/15 19:22

    どこでつまっているかわからないのでとりあえず追記しておきました

    キャンセル

+4

勘ですが

おそらくphp/read.phpでタスクを全件取得した後にforループで先頭の1件だけ取得してreturnしているからでしょう. 要するに, タスクの選択処理がまるまる抜けているからだと推測されます. なので, クライアントサイドなりサーバーでのデータ抽出部なりに検索処理を追加しましょう.


タスク選択の流れとしては次の何れかです.

一件取得のバターン

  1. UIにて何れかのタスクをタブルクリックする
  2. クリックイベントにおいてどのタスクがクリックされたかを判定
  3. 選択されたタスクに関するデータを取得するため, サーバー(PHPアプリ)に問い合わせる(Ajax)
  4. PHPアプリは渡されたタスクをSQLを使ってDBに問い合わせる
  5. PHPアプリは得られた結果をJSONとしてWEBブラウザに返す.
  6. WEBブラウザはPHPアプリから受け取ったタスク情報JSONを画面に表示する.

全件取得のパターン

  1. UIにて何れかのタスクをタブルクリックする
  2. クリックイベントにおいてどのタスクがクリックされたかを判定
  3. WEBブラウザはタスクに関するデータを全件取得するため, サーバー(PHPアプリ)に問い合わせる(Ajax)
  4. PHPアプリはタスクの全件をSQLを使ってDBに問い合わせる
  5. PHPアプリは得られた結果をJSONとしてWEBブラウザに返す.
  6. WEBブラウザはPHPアプリから受け取ったタスクの全件からダブルクリックされたタスクを見つけて画面に表示する.

で, 現在あなたのコードはタスク情報を10件取得し, forループで最初の一件だけを取り出しています.

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/15 17:30

    タスクの選択処理とは...なんでしょうか...?

    キャンセル

  • 2017/11/15 17:38

    defghi1977氏がおっしゃってるのは「サーバ側が選択されたものをクライアントで返す」一連の処理のことではないでしょうか
    現在その部分が実装されていないと仮定すると `var $everyResponse = response.rows;` ここのlengthが1になってしまっていませんか?

    キャンセル

  • 2017/11/15 17:42

    申し訳ございません、API側のコードを追記しました。

    キャンセル

  • 2017/11/15 18:09

    どうするかは貴方次第ですが, JavaScriptなりPHPなりSQLなり「必要とするデータだけ取り出す処理」を記述しなければ「選んだ(ダブルクリックしたもの)タスクをHTMLのinput要素に突っ込む」ことは出来ません.

    キャンセル

  • 2017/11/15 18:27

    for(let i = 0; i < $everyResponse.length; i++) {

    // 中略

    // ループを中断
    return false;
    }
    ↑このfor-loopで2周目に入るかどうかをよく考えて欲しい>yashichiさん

    キャンセル

  • 2017/11/15 18:58

    自分でコメント書いてる通り、return falseしちゃったらループが止まりますね...

    キャンセル

+3

サーバ側で SELECT * FROM list ORDER BY id DESC LIMIT 10; でレコードを取得し、クライアント側に返却。
クライアント側でもconsole.logの結果から値を受け取れているように見えます。

{rows: Array(10)}
rows:Array(10)
0:{0: "217", 1: "test", 2: "rest", 3: null, 4: null, id: "217", title: "test", content: "rest", complement: null, deadline: null}
1:{0: "216", 1: "", 2: "", 3: null, 4: null, id: "216", title: "", content: "", complement: null, deadline: null}
2:{0: "215", 1: "", 2: "", 3: null, 4: null, id: "215", title: "", content: "", complement: null, deadline: null}
3:{0: "214", 1: "sentence", 2: "root", 3: null, 4: null, id: "214", title: "sentence", content: "root", complement: null, deadline: null}
4:{0: "213", 1: "clear", 2: "content", 3: null, 4: null, id: "213", title: "clear", content: "content", complement: null, deadline: null}
5:{0: "212", 1: "", 2: "", 3: null, 4: null, id: "212", title: "", content: "", complement: null, deadline: null}
6:{0: "211", 1: "", 2: "", 3: null, 4: null, id: "211", title: "", content: "", complement: null, deadline: null}
7:{0: "210", 1: "", 2: "", 3: null, 4: null, id: "210", title: "", content: "", complement: null, deadline: null}
8:{0: "209", 1: "", 2: "", 3: null, 4: null, id: "209", title: "", content: "", complement: null, deadline: null}
9:{0: "208", 1: "", 2: "", 3: null, 4: null, id: "208", title: "", content: "", complement: null, deadline: null}
length:10
__proto__:Array(0)
__proto__:Object

返却値の長さがどうであろうと先頭の要素をinputにいれてそこでreturnしているのでforは2ループ以降は動かないと思います。
Yashichiさんはforループがどのように動くことを想定しているのでしょうか

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/15 18:51

    return falseを書いた意図としては、タスクを選んでinput要素の中に入れようとしたときに、
    選んだら選んだ分だけ後ろに連結してしまうので、それをなんとかしたいと思ったというのがあります。
    確かに自分でも書いてる通り、return falseを入れたらループはそこでストップしてしまいますね...

    ダブルクリックして選んだタスクだけがinput要素に入ってほしいというのがやりたいことなので、もしかしたらfor文自体が要らないのでしょうか...

    キャンセル

  • 2017/11/15 19:00

    ロジックではなく実装イメージの問題になってきますので、まずはYashichiさんがイメージするデザインを静的なHTMLで書いてみてそれを組むには受け取ったレコードからどのように当て込めばいいかを考え直してみるのはいかがでしょうか。
    完成系がHTMLでみえたら案外簡単でそれが近道かもしれません。

    キャンセル

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

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

関連した質問

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

  • PHP

    17783questions

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

  • JavaScript

    13835questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • jQuery

    5824questions

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