Question

我正在生成一个包含单个模型中所有行的选择菜单，我已经提出了3种不同的解决方案 - 但我很难决定哪一行最有意义并遵循MVC / Laravel最佳实践。

模型状态

states table:
id    abbr    name
1     AL      Alabama
2     AK      Alaska
3     AZ      Arizona
4     AR      Arkansas
etc.

解决方案＃1：从控制器中的模型中检索所有状态，执行逻辑以在控制器中生成状态数组，将其传递给查看并使用laravels表单生成选择菜单，并传入数组。

*控制器：my_controller.php *

public function get_index()
{
    $states = State::all();
    foreach ($states as $state)
    {
        $states_array[$state->id] = $state->name;   
    }
        return View::make('my_view')->with('states_array',$states_array);
}

*查看：my_view.php *

{{ Form::select('state_id',$states_array); }}

解决方案＃2：从控制器中的模型中检索所有状态，将检索到的状态模型传递给视图，执行逻辑以在视图中生成状态数组，并使用laravels form class生成选择菜单。 / p>

*控制器：my_controller.php *

public function get_index()
{
    $states = State::all();
    return View::make('my_view')->with('states',$states);
}

*查看：my_view.php *

@foreach ($states as $state)
    $states_array[$state->id] = $state->name;
@endforeach
{{ Form::select('state_id',$states_array); }}

解决方案＃3：将all_array（）（或可选地扩展all）方法添加到状态模型中，该模型提取所有状态记录，执行逻辑以创建数组并返回结果数组。在控制器中调用所述方法并将其传递给视图。使用带有传入数组的类的laravels表单生成选择菜单。

*型号：my_model.php *

public static function all_array()
{
    $states = self::all();
    foreach ($states as $state)
    {
        $states_array[$state->id] = $state->name;   
    }
    return $states_array;
}

*控制器：my_controller.php *

public function get_index()
{
    $states = State::all_array();
    return View::make('my_view')->with('states',$states);
}

*查看：my_view.php *

{{ Form::select('state_id',$states); }}

那么这3个解决方案中哪一个最有意义并且更接近MVC / Laravel最佳实践？可选 - 是否有比上述3更好的解决方案？

非常感谢任何和所有回复！

Answer 1

我会使用选项1，但使用模型中的lists（）方法进行简化。

$states_array = State::lists( 'name', 'id' );

这会生成您正在寻找的数组。

Answer 2

我会选择解决方案＃3。模型没有理由只需要以ORM方式运行。如果检索完整状态列表是您需要执行的常见活动，那么请务必将其添加到模型类中。这样，如果您需要做任何事情，例如排除某些状态，您可以在一个地方进行更改。

Answer 3

我不会使用任何这些解决方案。您应该实现presenter模式以包装任何表示逻辑并使其远离视图。保持模型可能是下一个最佳方式，但理想情况下，您将域逻辑与表示逻辑分开。

有一些presenter bundles和Laravel 4甚至有一些presenter packages可用。

养成分离逻辑的习惯是很好的，特别是当你意识到你在视图中做了大量的表示逻辑时。

但是要明确，最好的方法是使用lists()方法，如上所述。仍然让主持人看一看。

将逻辑放置在控制器与视图与模型之间

3 个答案: