回答編集履歴
4
文言修正
answer
CHANGED
|
@@ -2,7 +2,7 @@
|
|
|
2
2
|
|
|
3
3
|
補足として、
|
|
4
4
|
このソースの場合、idが入らなかった時(0)に`$data['article']`は定義されていないので`Notice: Undefined index`エラーが発生します。
|
|
5
|
-
そ
|
|
5
|
+
それを解決させるには下記のようにissetやemptyを挟む必要があります。
|
|
6
6
|
|
|
7
7
|
```
|
|
8
8
|
if(!isset($data['article']) || !$data['article']) {
|
3
修正
answer
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
基本的には最初に回答しているyng13さんの通りです。
|
|
2
2
|
|
|
3
3
|
補足として、
|
|
4
|
-
このソースの場合、idが入らなかった時に`$data['article']`は定義されていないので`Notice: Undefined index`エラーが発生します。
|
|
4
|
+
このソースの場合、idが入らなかった時(0)に`$data['article']`は定義されていないので`Notice: Undefined index`エラーが発生します。
|
|
5
5
|
そのためには下記のようにissetやemptyを挟む必要があります。
|
|
6
6
|
|
|
7
7
|
```
|
2
修正
answer
CHANGED
|
@@ -10,7 +10,7 @@
|
|
|
10
10
|
}
|
|
11
11
|
```
|
|
12
12
|
|
|
13
|
-
個人的にはソースの明瞭性に欠けると感じるので
|
|
13
|
+
個人的にはソースの明瞭性に欠けていたり、判定の内容が異なると感じるので以下のように区切ったほうが良いと思います。
|
|
14
14
|
あくまでも個人的な意見です。
|
|
15
15
|
```
|
|
16
16
|
if(!empty($id)) Response::redirect('articles');
|
1
修正
answer
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
基本的には最初に回答しているyng13さんの通りです。
|
|
2
|
-
idが指定されていない場合は、Model_Article::find($id)が通らないのではないでしょうか。(フレームワークによってはやってくれるのもある気がしますが)
|
|
3
2
|
|
|
3
|
+
補足として、
|
|
4
|
-
|
|
4
|
+
このソースの場合、idが入らなかった時に`$data['article']`は定義されていないので`Notice: Undefined index`エラーが発生します。
|
|
5
5
|
そのためには下記のようにissetやemptyを挟む必要があります。
|
|
6
6
|
|
|
7
7
|
```
|