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

回答編集履歴

5

追記2の2点目を具体的に

2018/04/16 01:51

投稿

Lulucom
Lulucom

スコア1904

answer CHANGED
@@ -10,4 +10,4 @@
10
10
 
11
11
  (追記2)
12
12
  - Adress モデルのメソッド addresses は user が正しいかと思います。
13
- - コントローラの test メソッドの実装も色々とおかそう
13
+ - コントローラの test メソッドの Address::find は User::find が正いのはないでしょうか

4

説明を追記

2018/04/16 01:51

投稿

Lulucom
Lulucom

スコア1904

answer CHANGED
@@ -3,8 +3,11 @@
3
3
  (追記)
4
4
  モデル名は `class Address extends Model` と単数形になっているのに
5
5
  他の箇所では `Addresses` と複数形になってしまっています。
6
+ 通常は、テーブル名は複数形(users, addresses など)にしますが
6
- モデル名は通常は単数形(User, Address など)にしますので、`Address` に統一した方が良さそうです。
7
+ モデル名は単数形(User, Address など)にしますので、`Address` に統一した方が良さそうです。
7
8
 
9
+ ご参考: [Eloquentモデル規約](https://readouble.com/laravel/5.6/ja/eloquent.html#eloquent-model-conventions)
10
+
8
11
  (追記2)
9
12
  - Adress モデルのメソッド addresses は user が正しいかと思います。
10
13
  - コントローラの test メソッドの実装も色々とおかしそうです。

3

説明を追記

2018/04/15 08:35

投稿

Lulucom
Lulucom

スコア1904

answer CHANGED
@@ -1,6 +1,8 @@
1
1
  Addresses ではなく、正しくは Address ではないでしょうか。
2
2
 
3
3
  (追記)
4
+ モデル名は `class Address extends Model` と単数形になっているのに
5
+ 他の箇所では `Addresses` と複数形になってしまっています。
4
6
  モデル名は通常は単数形(User, Address など)にしますので、`Address` に統一した方が良さそうです。
5
7
 
6
8
  (追記2)

2

他のおかしそうな点を追記

2018/04/15 08:29

投稿

Lulucom
Lulucom

スコア1904

answer CHANGED
@@ -1,4 +1,8 @@
1
1
  Addresses ではなく、正しくは Address ではないでしょうか。
2
2
 
3
3
  (追記)
4
- モデル名は通常は単数形(User, Address など)にしますので、`Address` に統一した方が良さそうです。
4
+ モデル名は通常は単数形(User, Address など)にしますので、`Address` に統一した方が良さそうです。
5
+
6
+ (追記2)
7
+ - Adress モデルのメソッド addresses は user が正しいかと思います。
8
+ - コントローラの test メソッドの実装も色々とおかしそうです。

1

単数形に統一すべき理由を追記

2018/04/15 08:20

投稿

Lulucom
Lulucom

スコア1904

answer CHANGED
@@ -1,1 +1,4 @@
1
- Addresses ではなく、正しくは Address ではないでしょうか。
1
+ Addresses ではなく、正しくは Address ではないでしょうか。
2
+
3
+ (追記)
4
+ モデル名は通常は単数形(User, Address など)にしますので、`Address` に統一した方が良さそうです。