回答編集履歴
1
解決の流れを修正
test
CHANGED
|
@@ -36,20 +36,6 @@
|
|
|
36
36
|
|
|
37
37
|
|
|
38
38
|
|
|
39
|
-
`Cannot read property 'forEach' of undefined`を翻訳すると、
|
|
40
|
-
|
|
41
|
-
`undefined`の`forEach`プロパティを参照しようとしたのでエラー。
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
転じて`tweets.forEach`の`tweets`は配列であるべきだったのに、
|
|
46
|
-
|
|
47
|
-
`undefined`が入っている事が諸悪の根源です。
|
|
48
|
-
|
|
49
|
-
なぜ`tweets`が配列でないのか？に着目してより上流の箇所を修正しましょう。
|
|
50
|
-
|
|
51
|
-
|
|
52
|
-
|
|
53
39
|
---
|
|
54
40
|
|
|
55
41
|
|
|
@@ -58,42 +44,72 @@
|
|
|
58
44
|
|
|
59
45
|
|
|
60
46
|
|
|
61
|
-
|
|
47
|
+
今回のエラー文を読み解きましょうか。
|
|
62
48
|
|
|
63
|
-
|
|
49
|
+
`Cannot read property 'forEach' of undefined`でしたっけ。
|
|
64
50
|
|
|
65
|
-
|
|
51
|
+
なので直訳すれば`undefined`の`forEach`プロパティを参照しようとしたのでエラー。
|
|
66
52
|
|
|
67
|
-
connection.query(
|
|
68
53
|
|
|
69
|
-
'UPDATE tweets SET good WHERE id=?',
|
|
70
54
|
|
|
71
|
-
|
|
55
|
+
`forEach`があるのは`tweets.forEach`の行ですね。
|
|
72
56
|
|
|
73
|
-
|
|
57
|
+
`tweets`は配列であるべきだったのに`undefined`が入っている事が問題です。
|
|
74
58
|
|
|
75
|
-
|
|
59
|
+
なぜ`tweets`が配列でないのか？に着目し、より上流の箇所を修正しましょう。
|
|
76
60
|
|
|
77
|
-
});
|
|
78
61
|
|
|
79
|
-
});
|
|
80
62
|
|
|
63
|
+
そういや「いいねボタン」押した時の挙動がおかしいんでしたっけ？
|
|
64
|
+
|
|
65
|
+
どれどれ？
|
|
66
|
+
|
|
67
|
+
|
|
68
|
+
|
|
69
|
+
> ```js
|
|
70
|
+
|
|
71
|
+
> //イイねボタンの更新
|
|
72
|
+
|
|
73
|
+
> app.post('/update_good', (req, res) => {
|
|
74
|
+
|
|
75
|
+
> connection.query(
|
|
76
|
+
|
|
77
|
+
> 'UPDATE tweets SET good WHERE id=?',
|
|
78
|
+
|
|
79
|
+
> [req.body.tweet_id],
|
|
80
|
+
|
|
81
|
+
> (error,results)=>{
|
|
82
|
+
|
|
83
|
+
> res.render('main.ejs',{user:username,tweets:results});
|
|
84
|
+
|
|
85
|
+
> });
|
|
86
|
+
|
|
87
|
+
> });
|
|
88
|
+
|
|
81
|
-
```
|
|
89
|
+
> ```
|
|
82
90
|
|
|
83
91
|
|
|
84
92
|
|
|
85
93
|
`'UPDATE tweets SET good WHERE id=?'`のSQL文を送った結果がどうなるのか確認してみましたか？
|
|
86
94
|
|
|
87
|
-
|
|
95
|
+
これは`UPDATE`の結果テーブルがどうなったかの結果が得られるはずです。
|
|
96
|
+
|
|
97
|
+
tweetsテーブルの値の配列が入っているはずがないんですよ。
|
|
88
98
|
|
|
89
99
|
|
|
90
100
|
|
|
91
|
-
|
|
101
|
+
`UPDATE`に成功したら改めてSQLの`SELECT`を発行し
|
|
92
102
|
|
|
93
|
-
|
|
103
|
+
`SELECT`の結果を`tweets`に束縛して使いましょう。
|
|
94
104
|
|
|
95
105
|
|
|
96
106
|
|
|
97
|
-
|
|
107
|
+
まぁこういったWebサーバを実装する上で、
|
|
98
108
|
|
|
109
|
+
Postの結果でHTMLを返す作りにするとF5キーでリロードする度にPostの処理が走るので微妙です。
|
|
110
|
+
|
|
111
|
+
リダイレクトさせるのが常套手段ですね。
|
|
112
|
+
|
|
113
|
+
|
|
114
|
+
|
|
99
|
-
[Expressのリダイレクト機能](https://expressjs.com/ja/api.html#res.redirect)を使って`/`パスのgetにリクエストを受け流し
|
|
115
|
+
[Expressのリダイレクト機能](https://expressjs.com/ja/api.html#res.redirect)を使って`/`パスのgetにリクエストを受け流す実装に変更しましょう。
|