手机用户2502861877 发布于 2017-05-17 00:12

我在每个控制器的初始化方法中实例化了相应的Model,然后在各个方法里使用,不知道这样有什么不妥,感觉自己实现的不够优雅。 主要是网站有很多模块,都是差不多的功能,部分代码是替换了model名称实例化。

public function __construct()
    {
        parent::__construct();
        $this->model = new \Line();
    }


public function store()
{
    $input = \Input::all();
    $validator = \Validator::make($input,$this->model->getRules('create'),$this->model->getMessage());
    if ($validator->fails())
    {
        return \Redirect::back()->with('errorCode',1)->withErrors($validator)->withInput();
    }else{
        if($model = $this->model->create($input)){
            return \Redirect::back()->with('errorCode',0);
        }else{
            return \Redirect::back()->with('errorCode',2);
        }
    }

}
1 个回答
  • 如果是Model的话,不需要这样做。

    你是store可以这样写:

    public function store()
    {
        $input = \Input::all();
        $validator = \Validator::make($input, Line::getRules('create'), Line::getMessage());
        if ($validator->fails())
        {
            return \Redirect::back()->with('errorCode',1)->withErrors($validator)->withInput();
        }else{
            if($model = Line::create($input)){
                return \Redirect::back()->with('errorCode',0);
            }else{
                return \Redirect::back()->with('errorCode',2);
            }
        }
    
    }
    

    Lind的getRules和getMessage和具体的对象没有关系的话,就写成静态方法。
    下面的create直接用静态方法。

    2017-05-17 06:15 回答
撰写答案
今天,你开发时遇到什么问题呢?
立即提问
活跃用户
  • 1
    问道摩托
  • 2
    袁广龙976丶
  • 3
    逍遥子
  • 4
    亮仔
  • 5
    芙蓉花
  • 6
    作小die_syj
  • 7
    王丽珠景平怡伦_621
  • 8
    adfa3sd5f6a
  • 9
    欣欣大妮
  • 10
    sfasfs
PHP1.CN | 中国最专业的PHP中文社区 | json解析格式化 | PHP资讯 | PHP教程 | 数据库技术 | 服务器技术 | 前端开发技术 | PHP框架 | 开发工具 | 在线工具
Copyright © 1998 - 2020 PHP1.CN. All Rights Reserved PHP1.CN 第一PHP社区 版权所有