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

    • ava guest 08.05.2012 14:33 # −1

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

    • ava guest 08.05.2012 14:36 # 0

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

    • ava Lure Of Chaos 08.05.2012 14:56 # +1

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

      • ava telnet 08.05.2012 15:17 # 0

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

        • ava guest 08.05.2012 15:28 # 0

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

        • ava Lure Of Chaos 09.05.2012 00:43 # 0

          какой файл шаблона подключить какой view показать
          Ответить

          • ava telnet 09.05.2012 11:07 # +2

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

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