讨论数量:
- 本质来说 控制器不应具有 Validator 的业务逻辑, 完全可以放入 Request 层.
- 获取变量就写了 16 行, 还都是一次性变量,获取的意义在哪?
- 这段代码的意义在哪?
- 这段不应该放进中间件么?
- 这里完全可以使用依赖注入 不要直接new , 如果你到后面增加了一个 构造函数的 参数,那么就到处都要改,建议使用 依赖注入
- 这个方法本身不应该返回数组么?
- 如果你真的要在控制器里面验证, 验证规则应该是这样的格式,这样对于使用 phpstorm 的同事很友好.
'name'=>[ 'required', 'string', 'min:3', 'max:20' ]
- 这点为个人观点.我还是相对不提倡使用 input 获取参数的,认为参数是怎么过来的就要怎么获取. get 参数就用 get 获取, post 的就用post 获取.
个人的一点看法, 浓厚的个人色彩. 只是建议. 如果不妥 可以留言.
上面这十几行变量赋值的操作真的有意义么?
推荐文章: