teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

2

xx

2018/10/15 10:01

投稿

退会済みユーザー
answer CHANGED
@@ -28,8 +28,10 @@
28
28
  var text ="";
29
29
  var target="";
30
30
  // Q1: なぜ.reverse()?
31
+ // A1: 削除や統合などの処理を行う場合、前から処理すると、
32
+ // 削除したときに後ろの要素が前に詰まりますよね?それが原因で
31
- // A1: 削除や統合などの処理を行う場合、前から処理すると、削除したときに後ろの要素が前に詰まりますよね?それが原因で配列の処理がおかしくなるケースがあったりします(処理の内容による)。習慣として、後ろから処理してるのかなと推察します。
33
+ // 配列の処理がおかしくなるケースがあったりします(処理の内容による)。
32
- //
34
+ // 習慣として、後ろから処理してるのかなと推察します。
33
35
  $($('#tokyo tr:first, .tokyo-td1').get().reverse()).each(function() {
34
36
  if ($(this).text() == text) {
35
37
  // Q2: ++って何?
@@ -37,7 +39,10 @@
37
39
  // ++自体はインクリメント演算子といい、詳細はぐぐってください。
38
40
  counter++;  
39
41
  // Q3: (target !="")ってどういうこと?というかこのtargetって具体的にどこを指してる?
42
+ // A3: targetが初期値でないことを確認しています。targetは
43
+ // ひとつ前のセルを表しますが、初回に限っては
40
- // A3: targetが初期値でないことを確認しています。targetはひとつ前のセルを表しますが、初回に限っては、var target = ""となっており、ひとつ前のセルではないです。初回はひとつ前のセルが存在しないので妥当ですよね。
44
+ // var target = ""とされており、ひとつ前のセルではないです。
45
+ // 初回はひとつ前のセルが存在しないので妥当な処理だと思います。
41
46
  if(target !="")
42
47
  target.remove();
43
48
  } else {
@@ -46,10 +51,16 @@
46
51
  // Q4:これは何?
47
52
  // A4: counterの値を1に初期化しないと、counterの値が増え続け、
48
53
  // 東京以外のセルが出てきた場合にそのセルまでくっつくことになります。
54
+ // なので、こうしてます。
49
55
  counter=1;
50
56
  }
57
+ // Q5: この記述は何のため?
51
- text = $(this).text();  ← Q5:この記述は何のため? A5:セルのテキストを保持しておきます。次の確認で、前のセルのテキストとして利用するためですね。
58
+ // A5: セルのテキストを保持しておきます。次の確認で、
59
+ // 前のセルのテキストとして利用するためですね。
60
+ text = $(this).text();
61
+ // Q6: この記述は何のため?
52
- target = $(this);  ← Q6:この記述は何のため? A6: セルを保持しておきます。次の確認で、前のセルとして利用するためです。
62
+ // A6: セルを保持しておきます。次の確認で、前のセルとして利用するためです。
63
+ target = $(this);
53
64
  });
54
65
  });
55
66
  ```

1

xxx

2018/10/15 10:01

投稿

退会済みユーザー
answer CHANGED
@@ -28,7 +28,7 @@
28
28
  var text ="";
29
29
  var target="";
30
30
  // Q1: なぜ.reverse()?
31
- // A1: 逆にしたほう、セルをっつけかどうか確認のに楽だから
31
+ // A1: 削除や統合などの処理を行う場合、前から処理すると、削除したときに後ろの要素前に詰まりますよね?それが原因で配列の処理がおかしケースがあったりしま(処理内容よる)。習慣として、後ろから処理してるのかなと推察しま
32
32
  //
33
33
  $($('#tokyo tr:first, .tokyo-td1').get().reverse()).each(function() {
34
34
  if ($(this).text() == text) {