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

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

          З.Ы. если ты покушаешься на мою пиченьку, то я ее уже съел ;)
          Ответить
    • мне еще мерещится говно в архитектуре, всмысле:
      1. isExist() можно было бы просто exists()
      2. что оно требует передачи именно массива - хотя, может быть именно тут так и надо, а обычно достаточно проверки только на ключ
      Ответить
      • 1. Ну, правильно написания метода с точки зрения английского языка - это не говно в архитектуре, но неприятно..
        2. Не, тут же не на ключ, а на всю пару
        Ответить
        • 1. я понимаю, что isExist удовлетворяет соглашению по аксессорам, но логически тут же метод
          2. ну по аналогии с картами(хэшами) - что бы проверить, существует ли пара ключ-значение, достаточно проверить существование ключа. Конечно же, тут isExist может быть универсальнее, всмысле 'alias' => проверяет сущестование именно такого аляса, а может проверять еще чье-то.....
          Ответить
          • 1. Метод да, не столько важно логически, сколько это звучит по чурковски типа "есть существовать объекта насяйника"
            2. Вот-вот, скорей всего там эта универсальность, так как это ж не аляса похоже метод
            Ответить
            • видимо у автора аксессорское вероисповедование
              Ответить
              • Та, на самом деле, по крайней мере, для русскоговорящих не так страшно )
                Ответить
              • в жабе это может быть очень важно.
                Если метод не удовлетворяет биновыми требованиям (тоесть не get и не is), то до него может быть недобраца через expressions language в jsp и всяких других шаблонах, если не начинается с set -- его не засетит какой-нить там струтс итд..

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

                В этом плане даже проперти си шарпа выглядят на такими уж убогими
                Ответить
    • а пиченьку ужо сьели?
      я требую продолжения банкета! (ц)
      Ответить
      • пиченька-то - шоколадная
        Ответить
        • из того же материала?

          оффтоп: ой, а у вас кто на авке?
          Ответить
          • неужели это лицо никто не может узнать? :)
            Ответить
            • прикольный кулфейс - впервые такой вижу
              просто немногие с говнокода ходят на чаны.
              Ответить
              • упс, хотел плюсануть - промазал :-/

                да ну, старый мем, уже в интервебах должно быть узнаваемо :)
                Ответить
                • наверное, я слишком старый для этих мемов. хто это??? ссылку в студию, плиз...
                  Ответить
                  • http://encyclopediadramatica.com/Troolface#Origins
                    Ответить
                    • Точно!
                      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
                      Одно лицо! Не узнал, богатым будете! Новая причёска?
                      Ответить
                      • Но самый эффектный вариант здесь:
                        http://tinyurl.com/tro11face
                        (в URL две единицы)
                        Ответить

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