質問編集履歴

3

追記

2021/12/27 06:07

投稿

laravel1221
laravel1221

スコア2

test CHANGED
File without changes
test CHANGED
@@ -1,4 +1,14 @@
1
1
  Laravelで画面制御を行なっており、middlewareで以下のように記述しました。
2
+
3
+
4
+
5
+ ログインユーザーの情報として、以下となっています。
6
+
7
+ user_id ユーザーID
8
+
9
+ role_list_flg ユーザー一覧画面権限フラグ 0or1 0=権限なし 1=権限あり
10
+
11
+ role_create_flg ユーザー登録画面権限フラグ 0or1 0=権限なし 1=権限あり
2
12
 
3
13
  ```
4
14
 
@@ -63,3 +73,5 @@
63
73
  ただ、投稿の場合、コメントの場合...と処理が続いていくので、似たような記述が増えてコードが冗長と思いました。
64
74
 
65
75
  なので、上記のコードをどのようにリファクタリングしたらよろしいでしょうか?
76
+
77
+ ※上記の内容で動作することは確認済み。

2

追記

2021/12/27 06:07

投稿

laravel1221
laravel1221

スコア2

test CHANGED
File without changes
test CHANGED
@@ -44,7 +44,7 @@
44
44
 
45
45
  // ユーザーが登録権限を持っていない(role_create_flgが0)
46
46
 
47
- if ($adminMasterUser->role_create_flg !== 1) {
47
+ if ($user->role_create_flg !== 1) {
48
48
 
49
49
  // ユーザーが画面アクセスの権限を持っていなければホーム画面にリダイレクト
50
50
 

1

追記

2021/12/27 05:32

投稿

laravel1221
laravel1221

スコア2

test CHANGED
File without changes
test CHANGED
@@ -60,4 +60,6 @@
60
60
 
61
61
  上記のように現在は一つ一つの処理を書いて、アクセスしたルート名と取得したルート名が一致→ユーザーが権限を持っていなければホーム画面にリダイレクトするとしてます。
62
62
 
63
- ただ、投稿の場合、こめんばあ
63
+ ただ、投稿の場合、コメントの場合...処理が続いていくので、似たような記述が増えてコードが冗長と思いました。
64
+
65
+ なので、上記のコードをどのようにリファクタリングしたらよろしいでしょうか?