質問編集履歴
3
追記
title
CHANGED
|
File without changes
|
body
CHANGED
|
@@ -1,4 +1,9 @@
|
|
|
1
1
|
Laravelで画面制御を行なっており、middlewareで以下のように記述しました。
|
|
2
|
+
|
|
3
|
+
ログインユーザーの情報として、以下となっています。
|
|
4
|
+
user_id ユーザーID
|
|
5
|
+
role_list_flg ユーザー一覧画面権限フラグ 0or1 0=権限なし 1=権限あり
|
|
6
|
+
role_create_flg ユーザー登録画面権限フラグ 0or1 0=権限なし 1=権限あり
|
|
2
7
|
```
|
|
3
8
|
public function handle($request, Closure $next)
|
|
4
9
|
{
|
|
@@ -30,4 +35,5 @@
|
|
|
30
35
|
|
|
31
36
|
上記のように現在は一つ一つの処理を書いて、アクセスしたルート名と取得したルート名が一致→ユーザーが権限を持っていなければホーム画面にリダイレクトするとしてます。
|
|
32
37
|
ただ、投稿の場合、コメントの場合...と処理が続いていくので、似たような記述が増えてコードが冗長と思いました。
|
|
33
|
-
なので、上記のコードをどのようにリファクタリングしたらよろしいでしょうか？
|
|
38
|
+
なので、上記のコードをどのようにリファクタリングしたらよろしいでしょうか？
|
|
39
|
+
※上記の内容で動作することは確認済み。
|
2
追記
title
CHANGED
|
File without changes
|
body
CHANGED
|
@@ -21,7 +21,7 @@
|
|
|
21
21
|
Route::currentRouteName() == 'user.register.post'
|
|
22
22
|
) {
|
|
23
23
|
// ユーザーが登録権限を持っていない(role_create_flgが0)
|
|
24
|
-
if ($
|
|
24
|
+
if ($user->role_create_flg !== 1) {
|
|
25
25
|
// ユーザーが画面アクセスの権限を持っていなければホーム画面にリダイレクト
|
|
26
26
|
return redirect()->route('home');
|
|
27
27
|
}
|
1
追記
title
CHANGED
|
File without changes
|
body
CHANGED
|
@@ -29,4 +29,5 @@
|
|
|
29
29
|
```
|
|
30
30
|
|
|
31
31
|
上記のように現在は一つ一つの処理を書いて、アクセスしたルート名と取得したルート名が一致→ユーザーが権限を持っていなければホーム画面にリダイレクトするとしてます。
|
|
32
|
-
ただ、投稿の場合、
|
|
32
|
+
ただ、投稿の場合、コメントの場合...と処理が続いていくので、似たような記述が増えてコードが冗長と思いました。
|
|
33
|
+
なので、上記のコードをどのようにリファクタリングしたらよろしいでしょうか？
|