回答編集履歴
5
追記2の2点目を具体的に
test
CHANGED
|
@@ -22,4 +22,4 @@
|
|
|
22
22
|
|
|
23
23
|
- Adress モデルのメソッド addresses は user が正しいかと思います。
|
|
24
24
|
|
|
25
|
-
- コントローラの test メソッドの
|
|
25
|
+
- コントローラの test メソッドの Address::find は User::find が正しいのではないでしょうか。
|
4
説明を追記
test
CHANGED
|
@@ -8,7 +8,13 @@
|
|
|
8
8
|
|
|
9
9
|
他の箇所では `Addresses` と複数形になってしまっています。
|
|
10
10
|
|
|
11
|
+
通常は、テーブル名は複数形（users, addresses など）にしますが
|
|
12
|
+
|
|
11
|
-
モデル名は
|
|
13
|
+
モデル名は単数形（User, Address など）にしますので、`Address` に統一した方が良さそうです。
|
|
14
|
+
|
|
15
|
+
|
|
16
|
+
|
|
17
|
+
ご参考: [Eloquentモデル規約](https://readouble.com/laravel/5.6/ja/eloquent.html#eloquent-model-conventions)
|
|
12
18
|
|
|
13
19
|
|
|
14
20
|
|
3
説明を追記
test
CHANGED
|
@@ -3,6 +3,10 @@
|
|
|
3
3
|
|
|
4
4
|
|
|
5
5
|
（追記）
|
|
6
|
+
|
|
7
|
+
モデル名は `class Address extends Model` と単数形になっているのに
|
|
8
|
+
|
|
9
|
+
他の箇所では `Addresses` と複数形になってしまっています。
|
|
6
10
|
|
|
7
11
|
モデル名は通常は単数形（User, Address など）にしますので、`Address` に統一した方が良さそうです。
|
|
8
12
|
|
2
他のおかしそうな点を追記
test
CHANGED
|
@@ -5,3 +5,11 @@
|
|
|
5
5
|
（追記）
|
|
6
6
|
|
|
7
7
|
モデル名は通常は単数形（User, Address など）にしますので、`Address` に統一した方が良さそうです。
|
|
8
|
+
|
|
9
|
+
|
|
10
|
+
|
|
11
|
+
（追記2）
|
|
12
|
+
|
|
13
|
+
- Adress モデルのメソッド addresses は user が正しいかと思います。
|
|
14
|
+
|
|
15
|
+
- コントローラの test メソッドの実装も色々とおかしそうです。
|
1
単数形に統一すべき理由を追記
test
CHANGED
|
@@ -1 +1,7 @@
|
|
|
1
1
|
Addresses ではなく、正しくは Address ではないでしょうか。
|
|
2
|
+
|
|
3
|
+
|
|
4
|
+
|
|
5
|
+
（追記）
|
|
6
|
+
|
|
7
|
+
モデル名は通常は単数形（User, Address など）にしますので、`Address` に統一した方が良さそうです。
|