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

    +148

    1. 1
    2. 2
    3. 3
    4. 4
    if( !$this->dirsmodel->isExist(array('alias' => $_POST['alias']))) {
    $this->messages[] = 'Такой алиас уже существует';
    return $this->redirect("admin/dirs/#" . $cat_id);
}

    Говнокод тонкий особый. Кто найдёт получит пиченьку

    Запостил: DrFreez, 11 Сентября 2010

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

    • ava u1tr0n 11.09.2010 13:55 # +2

      Ну во первых "!" в сочетании с isExist() какбэ говорит нам что мессага должна быть 'Такого алиаса еще не существует', вовторых хз зачем передавать массив, ну и наконец еще '#" в редиректе наверняка лишняя, не думаю что он редиректит на якорь.
      Ответить

      • ava DrFreez 11.09.2010 14:27 # 0

        пиченька твоя
        Ответить

      • ava Wave 11.09.2010 16:18 # 0

        А по-моему, как раз на якорь и редиректит. Т.е. не просто на страницу, а к конкретному #" . $cat_id на странице.
        Ответить

        • ava u1tr0n 11.09.2010 17:51 # +2

          хз, по данному фрагменту кода непонятно, логически, если алиас уже существует то в случае с якорем надо кинуть на него, а тогда переменная $cat_id (опять таки по моей логике) должна получать свое значение уже после if или каким-нибудь явным видов, или хотя бы с коментом, куда это пойдет, а то пойду но не знаю куда.

          З.Ы. если ты покушаешься на мою пиченьку, то я ее уже съел ;)
          Ответить

    • ava Lure Of Chaos 12.09.2010 01:24 # 0

      мне еще мерещится говно в архитектуре, всмысле:
      1. isExist() можно было бы просто exists()
      2. что оно требует передачи именно массива - хотя, может быть именно тут так и надо, а обычно достаточно проверки только на ключ
      Ответить

      • ava bildja 12.09.2010 01:31 # 0

        1. Ну, правильно написания метода с точки зрения английского языка - это не говно в архитектуре, но неприятно..
        2. Не, тут же не на ключ, а на всю пару
        Ответить

        • ava Lure Of Chaos 12.09.2010 01:36 # 0

          1. я понимаю, что isExist удовлетворяет соглашению по аксессорам, но логически тут же метод
          2. ну по аналогии с картами(хэшами) - что бы проверить, существует ли пара ключ-значение, достаточно проверить существование ключа. Конечно же, тут isExist может быть универсальнее, всмысле 'alias' => проверяет сущестование именно такого аляса, а может проверять еще чье-то.....
          Ответить

          • ava bildja 12.09.2010 01:39 # 0

            1. Метод да, не столько важно логически, сколько это звучит по чурковски типа "есть существовать объекта насяйника"
            2. Вот-вот, скорей всего там эта универсальность, так как это ж не аляса похоже метод
            Ответить

            • ava Lure Of Chaos 12.09.2010 01:40 # 0

              видимо у автора аксессорское вероисповедование
              Ответить

              • ava bildja 12.09.2010 01:42 # 0

                Та, на самом деле, по крайней мере, для русскоговорящих не так страшно )
                Ответить

              • ava Анонимус 12.09.2010 03:26 # 0

                в жабе это может быть очень важно.
                Если метод не удовлетворяет биновыми требованиям (тоесть не get и не is), то до него может быть недобраца через expressions language в jsp и всяких других шаблонах, если не начинается с set -- его не засетит какой-нить там струтс итд..

                Короче соглашения лучше соблюдать, даже если получается кошмар типа isExists.

                В этом плане даже проперти си шарпа выглядят на такими уж убогими
                Ответить

    • ava Lure Of Chaos 12.09.2010 01:49 # 0

      а пиченьку ужо сьели?
      я требую продолжения банкета! (ц)
      Ответить

      • ava bugmenot 12.09.2010 15:46 # −1

        пиченька-то - шоколадная
        Ответить

        • ava Lure Of Chaos 12.09.2010 17:13 # +2

          из того же материала?

          оффтоп: ой, а у вас кто на авке?
          Ответить

          • ava bugmenot 12.09.2010 19:06 # −1

            неужели это лицо никто не может узнать? :)
            Ответить

            • ava 3.14159265 12.09.2010 20:37 # +1

              прикольный кулфейс - впервые такой вижу
              просто немногие с говнокода ходят на чаны.
              Ответить

              • ava bugmenot 12.09.2010 21:21 # −1

                упс, хотел плюсануть - промазал :-/

                да ну, старый мем, уже в интервебах должно быть узнаваемо :)
                Ответить

                • ava ZX_Spectrum 12.09.2010 23:36 # +2

                  наверное, я слишком старый для этих мемов. хто это??? ссылку в студию, плиз...
                  Ответить

                  • ava bugmenot 12.09.2010 23:59 # −1

                    http://encyclopediadramatica.com/Troolface#Origins
                    Ответить

                    • ava inkanus-gray 13.09.2010 00:13 # +2

                      Точно!
                      http://images.encyclopediadramatica.com/images/thumb/f/f2/Trollface_More_HD.png/657px-Trollface_More_HD.png
                      http://gravatar.com/avatar/b94d9cb39fdb23c4d78ba35b35ba9c4e?size=51 2
                      Одно лицо! Не узнал, богатым будете! Новая причёска?
                      Ответить

                      • ava inkanus-gray 13.09.2010 01:55 # +1

                        Но самый эффектный вариант здесь:
                        http://tinyurl.com/tro11face
                        (в URL две единицы)
                        Ответить

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