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

Question

###追記
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

Answer 1

勘ですが

おそらく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ループで最初の一件だけを取り出しています.

Comments

[Yashichi]

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

[ms90]

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

[Yashichi]

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

[defghi1977]

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

[tkturbo]

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

// 中略

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

[Yashichi]

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

Answer 2

$.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>

Comments

[Yashichi]

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

[yambejp]

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

[Yashichi]

おっしゃる通りです..

[yambejp]

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

Answer 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ループがどのように動くことを想定しているのでしょうか

Comments

[Yashichi]

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

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

[ms90]

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