php代码点火器在视图中生成数据库数据

Question

我正在生成一个 html 下拉列表，我目前有从数据库中选择数据并在视图中生成结果的代码？这可以吗，或者我应该以某种方式将它放在控制器中。如果是这样，我如何从视图中调用该数据？这是我的代码：

<select id="f_treeindex">
            <?php
                $query = $this->db->query('SELECT id, tree_name FROM trees');

                foreach ($query->result() as $row)
                {
                    echo '<option value="' . $row->id . '">' . $row->tree_name . '</option>';
                };
            ?>
</select>

我希望尽可能多地保留数据库数据。

修订：

型号：

function get_tree(){
    $query = $this->db->query('SELECT id, tree_name FROM trees');
    return $query->result_array();
}

控制器：

$data['trees'] = $this->Model_form->get_tree();

    if ($this->form_validation->run() == FALSE)
    {
        $this->load->view('view_form_group', $data);
    }

查看：

<?php
        foreach($trees as $tree){
            echo '<option value="' . $tree->id . '">' . $tree->tree_name . '</option>';
        }
        ?>

Answer 1

您的视图应该从包含数据的控制器接收一个数组，然后循环遍历该数组以填充选择。

Answer 2

为什么查询不包含在模型中？

所有数据库表都应该有一些形式的关联模型。在这种情况下，您可能需要Tree_Model 或类似的东西。控制器应负责将模型设置为正确状态（即，如果您的 select 中有 where 谓词，控制器将负责将该数据提供给模型）。

从那里开始，关于谁应该查询模型（控制器或视图）更像是一场宗教辩论。我通常会将模型查询放在视图中，因为模型已经处于正确的状态（由控制器设置）以保持我的控制器代码简洁。

归根结底，控制器或视图都不应该直接查询数据库。

Answer 3

查询进入树 model，在那里你有一个类似 get_trees() 的函数，你可以从 controller 调用它，比如 $trees = $this- >树->get_trees();

然后你将它传递给 view 并循环遍历它。这就是 MVC 方式。

对于您的 REVISED 代码，您还应该在模型中添加“return $query->result_array();”

Answer 4

查询应该进入控制器，然后传递给要显示的视图。