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

    +157

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    # http://habrahabr.ru/post/227277/
# открыл первый попавшийся скрипт на гитхабе по ссылке в посте, дальше не читал

function validate_email($str)
{
    return preg_match('/^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,3})$/',$str);
}

    Запостил: Sulik78, 25 Июня 2014

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

    • ava brutushafens 25.06.2014 23:53 # 0

      cleaned
      Ответить

    • ava kegdan 26.06.2014 08:15 # +1

      в принципе особого говна я не вижу. Ну разве что флаг i хотелось бы
      Ответить

      • ava bormand 26.06.2014 08:40 # +1

        > в принципе особого говна я не вижу.
        Не поддерживает новомодные говнодомены типа хуй.рф или hui.from.moskow :)
        Ответить

        • ava Fike 26.06.2014 09:00 # 0

          ну, скажем, хуйло.рф и я бы забанил.
          Ответить

          • ava bormand 26.06.2014 09:27 # +1

            Дык предварительную проверку e-mail'a делают не для бана по домену (ибо он не имеет никакого смысла) и не в целях безопасности (любая вменяемая smtp либа забреет кривой адрес, а без отправки письма не проверить ни наличия ящика, ни владения им), а всего лишь для удобства юзера...
            Ответить

            • ava kegdan 26.06.2014 09:40 # 0

              Да и не факт что данный имеил вообще существует, какая тут безопасность. Тупо, что бы юзер не ввел явного говна
              Ответить

              • ava bormand 26.06.2014 10:10 # 0

                > Да и не факт что данный имеил вообще существует
                На существование e-mail'а вообще всем насрать. Проверяется именно владение конкретного юзера этим мылом.

                > Тупо, что бы юзер не ввел явного говна
                Ага, поэтому регулярка из топика вполне прокатит. Хотя мне было бы лень ее писать/искать, и я бы поюзал готовую функцию ;)
                Ответить

                • ava kegdan 26.06.2014 10:24 # 0

                  >>Хотя мне было бы лень ее писать/искать, и я бы поюзал готовую функцию ;)

                  тоже верно. все говно уже написано до нас )
                  Ответить

        • ava eth0 26.06.2014 20:53 # 0

          Даже старые-добрые .info или .mobi
          Ответить

      • ava Qwertiy 26.06.2014 23:21 # 0

        Последний кусок - 2-3 символа ;)
        Ответить

    • ava Vasiliy 26.06.2014 13:41 # 0

      Этот кодярник не самая большая их проблема.
      Ответить

      • ava bormand 26.06.2014 13:44 # 0

        А какая самая большая?
        Ответить

        • ava Vasiliy 26.06.2014 15:03 # 0

          сама большая хз. Но это гораздо хуже.
          https://github.com/rustem-art/hd.rustem/blob/master/index.php#L27
          Ответить

          • ava bormand 26.06.2014 15:11 # 0

            mysql_real_escape_string() автор не забыл вызвать, так что терпимо.
            Ответить

            • ava myaut 26.06.2014 15:47 # 0

              Он это сделал уже после замечания на Хабре.
              Ответить

          • ava Fike 26.06.2014 16:14 # +1

            https://github.com/rustem-art/hd.rustem/blob/master/index.php#L69
            //setcookie('id', '', 0, "/");
    //setcookie('ps', '', 0, "/");
    // ТУТ УДАЛИТЬ КУКИ

            богато
            Ответить

            • ava bormand 26.06.2014 16:18 # 0

              В ps надеюсь не пароль лежит?
              Ответить

              • ava Fike 26.06.2014 17:03 # 0

                Гитхаб говорит, что нет такого больше в репе, а клонировать-грепать мне лень. Ну ему в принципе уже накидали хуев тачку (ладно, не тачку, всего четыре issue)

                https://github.com/rustem-art/hd.rustem/issues/3
                https://github.com/rustem-art/hd.rustem/issues/5
                Ответить

    • ava brutushafens 27.06.2014 01:19 # −2

      cleaned
      Ответить

      • ava 1024-- 27.06.2014 06:27 # +2

        nJKCNJ1
        Ответить

        • ava bormand 27.06.2014 07:02 # +1

          Юстас - Алексу.
          Ответить

          • ava kegdan 27.06.2014 08:25 # 0

            Расшифровка
            >>Господа, не соблаговолит ли кто заняться со мной анальным коитусом?
            >>Нет, сударь, ваш фаллос слишком велик
            Ответить

            • ava bormand 27.06.2014 09:31 # +2

              Опять тред свели к хуям и пидорасам ;( Живительного гоатсе на вас нет.
              Ответить

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