"PHP" / Говнокод #10221 Ссылка на оригинал

0

  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 telnet, (Updated )

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

  • ava guest # Ссылка на оригинал 0

    подключение внешних шаблонов считается гавнокодом?
    Ответить

  • ava guest # Ссылка на оригинал 0

    странно только то, что оно в конце каждого контроллера, тогда это не MVC ...
    Ответить

  • ava Lure Of Chaos # Ссылка на оригинал 0

    вообще-то нужно проверять и наличие дефолтного шаблона...
    Ответить

    • ava telnet # Ссылка на оригинал 0 ⇈

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

      • ava guest # Ссылка на оригинал 0 ⇈

        > Ох уж это MVC ради MVC...
        Тогда уж "типичный контроллер", mvc само по себе не говорит, что нужно все пихать в контроллер.
        Ответить

        • ava telnet # Ссылка на оригинал 0 ⇈

          Принцип выбора view единый во всей CMSке - если нет файла из настроенного шаблона, берём из дефолтного. Повторяющийся аспект поведения просто просится быть вынесенным куда-то отдельно, не в каждый же метод его копипастить.
          Ответить

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

Из-за тебя ушел bormand, guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓