回答編集履歴

2

xx

2018/10/15 10:01

投稿

退会済みユーザー
test CHANGED
@@ -58,9 +58,13 @@
58
58
 
59
59
  // Q1: なぜ.reverse()?
60
60
 
61
- // A1: 削除や統合などの処理を行う場合、前から処理すると、削除したときに後ろの要素が前に詰まりますよね?それが原因で配列の処理がおかしくなるケースがあったりします(処理の内容による)。習慣として、後ろから処理してるのかなと推察します。
61
+ // A1: 削除や統合などの処理を行う場合、前から処理すると、
62
62
 
63
+ // 削除したときに後ろの要素が前に詰まりますよね?それが原因で
64
+
65
+ // 配列の処理がおかしくなるケースがあったりします(処理の内容による)。
66
+
63
- //
67
+ // 習慣として、後ろから処理してるのかなと推察します。
64
68
 
65
69
  $($('#tokyo tr:first, .tokyo-td1').get().reverse()).each(function() {
66
70
 
@@ -76,7 +80,13 @@
76
80
 
77
81
  // Q3: (target !="")ってどういうこと?というかこのtargetって具体的にどこを指してる?
78
82
 
83
+ // A3: targetが初期値でないことを確認しています。targetは
84
+
85
+ // ひとつ前のセルを表しますが、初回に限っては
86
+
79
- // A3: targetが初期値でないことを確認しています。targetはひとつ前のセルを表しますが、初回に限っては、var target = ""となっており、ひとつ前のセルではないです。初回はひとつ前のセルが存在しないので妥当ですよね。
87
+ // var target = ""とされており、ひとつ前のセルではないです。
88
+
89
+ // 初回はひとつ前のセルが存在しないので妥当な処理だと思います。
80
90
 
81
91
  if(target !="")
82
92
 
@@ -94,13 +104,25 @@
94
104
 
95
105
  // 東京以外のセルが出てきた場合にそのセルまでくっつくことになります。
96
106
 
107
+ // なので、こうしてます。
108
+
97
109
  counter=1;
98
110
 
99
111
  }
100
112
 
101
- text = $(this).text();  ← Q5この記述は何のため? A5:セルのテキストを保持しておきます。次の確認で、前のセルのテキストとして利用するためですね。
113
+ // Q5: この記述は何のため?
102
114
 
115
+ // A5: セルのテキストを保持しておきます。次の確認で、
116
+
117
+ // 前のセルのテキストとして利用するためですね。
118
+
119
+ text = $(this).text();
120
+
121
+ // Q6: この記述は何のため?
122
+
103
- target = $(this);  ← Q6:この記述は何のため? A6: セルを保持しておきます。次の確認で、前のセルとして利用するためです。
123
+ // A6: セルを保持しておきます。次の確認で、前のセルとして利用するためです。
124
+
125
+ target = $(this);
104
126
 
105
127
  });
106
128
 

1

xxx

2018/10/15 10:01

投稿

退会済みユーザー
test CHANGED
@@ -58,7 +58,7 @@
58
58
 
59
59
  // Q1: なぜ.reverse()?
60
60
 
61
- // A1: 逆にしたほう、セルをくっつけるかどうか確認するのに楽だらで
61
+ // A1: 削除や統合などの処理を行う場合、前から処理すると、削除したときに後ろの要素前に詰まりますよね?それが原因で配列の処理がおかしなるケースがあたりします(処理の内容によ)。習慣として、後ろら処理してるのかなと推察しま
62
62
 
63
63
  //
64
64