回答編集履歴

3

蛇足

2019/06/14 10:07

投稿

m.ts10806
m.ts10806

スコア80861

test CHANGED
@@ -41,3 +41,17 @@
41
41
  ----
42
42
 
43
43
  あと、さすがにあのスペルミスは恥かくので直したほうがいいです。
44
+
45
+
46
+
47
+ ----
48
+
49
+
50
+
51
+ 蛇足:オブジェクト指向観点
52
+
53
+ 直接呼び出されないメソッドがpublicなのもおかしい。
54
+
55
+ むしろデータ取得などの処理であればコントローラーに直接書くのは微妙。
56
+
57
+ Carbon関係の処理をまとめた別クラス、例えばServiceに入れておいてDIしたほうが良いと思います。

2

修正

2019/06/14 10:07

投稿

m.ts10806
m.ts10806

スコア80861

test CHANGED
@@ -34,7 +34,7 @@
34
34
 
35
35
 
36
36
 
37
- Laravelの基本というよりPHPの基本。
37
+ Laravelの基本というよりPHP(というよりプログラミング言語関数の使い方)の基本。
38
38
 
39
39
 
40
40
 

1

修正

2019/06/14 10:02

投稿

m.ts10806
m.ts10806

スコア80861

test CHANGED
@@ -35,3 +35,9 @@
35
35
 
36
36
 
37
37
  Laravelの基本というよりPHPの基本。
38
+
39
+
40
+
41
+ ----
42
+
43
+ あと、さすがにあのスペルミスは恥かくので直したほうがいいです。