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

    +160

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    ob_start();
$goodsInterested = clone Core::$Controllers->Helpers->Good_Recomendation;
$goodsInterested->display(array(
	"modelId" => $this->model['id'],
	"type" => 'interested',
));
$interested = ob_get_clean();

    В системе виджеты - синглтоны.
    Чтобы как-то хоть работать, клонируем синглтон (!!!)
    Результат приходится оборачивать в ob_start, т.к не предусмотрено ничего типа echo $widget->render();

    Запостил: mitallast, 23 Ноября 2010

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

    • ava Анонимус 23.11.2010 13:18 # −1

      вот и протекла абстракция, прямо как у Спольски.

      с одной стороны мы пафосно городим контроллеры, с другой -- вносим знание о низкоуровневом ob_start.

      и опять хеши вместо объектов) ох уж пых
      Ответить

      • ava xXx_totalwar 23.11.2010 13:55 # 0

        >и опять хеши вместо объектов
        nani?
        Ответить

        • ava Анонимус 23.11.2010 13:57 # −1

          пытаюсь троллить
          Ответить

          • ava xXx_totalwar 23.11.2010 13:59 # 0

            я серьезно, причем тут объекты?
            Ответить

            • ava Анонимус 23.11.2010 14:02 # −2

              вот такую конструкцию
              array(
	"modelId" => $this->model['id'],
	"type" => 'interested',
)

              В сях бы описали структурой. в пыхе это логично сделать объектом.
              $goodsInterested->display(new RequestInfo($this->model->getId(), Types::Interested));


              что бы не опечататься в слове modelId, и не поймать потом null. Что бы легко переименовать его во время рефакторинга. Что бы внутри объекта можно было бы сделать какие-то преобразования.
              Итд.
              Ответить

              • ava xXx_totalwar 23.11.2010 14:08 # +2

                как все страшно в пыхомирке
                Ответить

                • ava mitallast 24.11.2010 08:23 # +2

                  Плохо программировать можно на любом языке. Тут дело то не в языке, а в людях, едва прочитавших о паттернах программирования - да и то, только первую главу о всемогущем синглтоне. Плюс ко всему понятия не имеют чем отличается статический класс от синглтона.

                  Вообще в практике программирования есть такое понятие, как прототип. Беда в том, что прототип стал продакшеном.
                  Ответить

                  • ava Анонимус 24.11.2010 12:32 # 0

                    люди, читавшие о паттернах обычно не пишут на пхп)
                    Ответить

                    • ava Vasiliy 24.11.2010 12:46 # 0

                      Пишут. Еще как пишут. В пыхе есть все для того чтобы отделять логику от представления даже я бы сказал логики друг от друга.
                      Ответить

                      • ava Анонимус 24.11.2010 14:06 # −1

                        малоли что там есть)
                        в пыхе есть PDO, но это не мешает всем писать mysql_query.
                        в пыхе есть объекты, но это не мешает всем писать ""type" => 'interested',"
                        Ответить

                        • ava Vasiliy 24.11.2010 14:27 # −2

                          PDO это обертка mysql_
                          Если разрабатывается небольшой проект и непредпологается использования других СУБД пишут mysql_ иначе используют PDO

                          ООП в пыхе напрочь кривое
                          в JS тоже есть ООП ну его же используют только в библиотеках
                          Ответить

                          • ava Анонимус 24.11.2010 14:33 # 0

                            >>PDO это обертка mysql_
                            и ADO.NET обертка и JDBC обертка

                            >>и непредпологается использования других СУБД
                            PDO это не только способ абстракции от БД (все равно диалекты SQL у всех разные), это еще и способ решить проблему SQL injection на корню.
                            Достаточно использовать statements, и ничего нигде не надо будет экскейпить.
                            Зачем работать с mysql_query, если есть нормальная, безопасная, универсальная обертка? Почему в perl всегда работают только через DBD, в джаве через JDBC, в нете через ADO.NET, и только в php я постоянно вижу mysql_query ?:)))

                            >>ООП в пыхе напрочь кривое
                            угу :(

                            >>в JS тоже есть ООП ну его же используют только в библиотеках
                            в JS ужасное ООП: все таки это прототипо-ориентированный язык, а не объектно-ориентированный.

                            Писать на JS тоже удовольствие не из больших (потому например и существует GWT)
                            Ответить

                          • ava bugmenot 24.11.2010 15:51 # 0

                            > в JS тоже есть ООП ну его же используют только в библиотеках
                            ORLY?
                            Ответить

                            • ava Vasiliy 24.11.2010 15:52 # 0

                              а кто еще в здравом уме будет его пользовать ??
                              Ответить

                              • ava bugmenot 24.11.2010 16:09 # 0

                                эх, школота

                                я нихуя не понял и не пользуюсь
                                я - медиана
                                -----------------------
                                никто не пользуется

                                вопрос на 3 балла - где ошибка в это силлогизме?
                                Ответить

                                • ava Vasiliy 24.11.2010 16:44 # 0

                                  я понял поэтому не пользуется, кто понял тот при составлении библиотек функций тоже возможно пользуется. Но любом случае это все край как криво.
                                  Ответить

                                  • ava bugmenot 24.11.2010 18:09 # 0

                                    чтобы не повторяться, о жс и библиотеках

                                    http://govnokod.ru/3512#comment34192
                                    Ответить

    • ava telnet 23.11.2010 13:19 # +6

      Если синглтон можно клонировать, это говна кусок, а не синглтон.
      Ответить

      • ava wmmorgun 23.11.2010 19:33 # +6

        угу, надо в синглтонах всегда писать магический метод __clone, тогда костыль не сработает:
        public function __clone() //для клоунов
{
  throw new Exception('Are you crazy?');
}
        Ответить

    • ava Lure Of Chaos 23.11.2010 16:52 # 0

      уже в "пыхомирке" принято, вместо смены архитектуры подставлять костыли.
      Зачем тут синглтон? "потому что это круто"
      Ответить

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