php - 在我看来，代码是否太多了？ CakePHP/MVC设计模式

Question

我正在使用google maps cakePHP helper通过 dereuromark 在我的 View 中显示 map 。 map 还具有标记，每个标记都有其自己的关联监听器，用于进行 ajax 调用。

map 正在完美渲染，一切都按预期进行。但是，我不确定所有这些代码是否都与我的 View 分开？我是 MVC 架构的新手...

<h1>Render map</h1>
<?php

// returns true if user logged in
if (!$session->check('Auth.User.id')){
    echo $this->Html->link('Log in',array('controller'=>'users', 'action' => 'login'));
} else 
{   
    echo "Hello " . $session->read('Auth.User.username') . "... ";
    echo $this->Html->link('Log out',array('controller'=>'users', 'action' => 'logout'));
}
echo $this->GoogleMapV3->map(array('map' => array('lat' => '44.230065', 'lng' => '-76.5000', 'zoom' => 14), 'div'=>array('id'=>'my_map3', 'height' => '400px', 'width' => '786px')));

foreach ($posts as $post) {
    $options = array(
        'lat' => $post['Post']['lat'],
        'lng' => $post['Post']['lng'],
    );

    $marker = $this->GoogleMapV3->addMarker($options);
    $script = "$.ajax({
                          url: \"show_post.php?q=\"+{$post['Post']['id']},
                          success: function(html){
                            $(\"#results\").html(html);
                          }
                        });";
    $this->GoogleMapV3->addCustomEvent($marker,$script);

}

echo $this->GoogleMapV3->script();

?>

如果您还可以指出任何设计缺陷，那就太好了。谢谢

Answer 1

看起来不错，只要您不在 View 中进行任何模型调用或太多业务逻辑，就可以了。您可以将问候语粘贴在元素中，然后调用 $this->element('greeting');在顶部而不是 if