1. PHP / Говнокод #10221

    +70

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (file_exists(DIR_TEMPLATE . $this->config->get('config_template') . '/template/product/compare.tpl')) {
    	$this->template = $this->config->get('config_template') . '/template/product/compare.tpl';
    } else {
    	$this->template = 'default/template/product/compare.tpl';
    }

    Спешите видеть в конце каждого контроллера OpenCart. Ох уж это MVC ради MVC...

    Запостил: telnet, 08 Мая 2012

    Комментарии (7) RSS

    • подключение внешних шаблонов считается гавнокодом?
      Ответить
    • странно только то, что оно в конце каждого контроллера, тогда это не MVC ...
      Ответить
    • вообще-то нужно проверять и наличие дефолтного шаблона...
      Ответить
      • Только почему о том, какой файл шаблона подключить, голова болит у контроллера? Скормил ответственной за это части кода (шаблонизатору либо некой обёртке над ним) имя шаблона и всё, до свидания. Как именно там шаблоны реализованы (на файлах ли, на чём-то ещё) - не его дело.
        Ответить
        • > Ох уж это MVC ради MVC...
          Тогда уж "типичный контроллер", mvc само по себе не говорит, что нужно все пихать в контроллер.
          Ответить
        • какой файл шаблона подключить какой view показать
          Ответить
          • Принцип выбора view единый во всей CMSке - если нет файла из настроенного шаблона, берём из дефолтного. Повторяющийся аспект поведения просто просится быть вынесенным куда-то отдельно, не в каждый же метод его копипастить.
            Ответить

    Добавить комментарий