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

    +67

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    if ($_GET['search'] == ' Поиск' || $_GET['search'] == 'Поиск') {
	$_GET['search'] = '';
}
if ($_GET['date'] == ' Дата' || $_GET['date'] == 'Дата') {
	$_GET['date'] = '';
} else {
	$_GET['date'] = preg_replace('/(\d+).(\d+).(\d+)/i','$3-$2-$1', $_GET['date']);
}

    Уж не знаю, что и сказать... А вдруг перед текстом попадутся 2 пробела? Или 3?

    Запостил: domaster, 02 Июля 2012

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

    • ava Lure Of Chaos 02.07.2012 10:42 # 0

      да, и вместо замены точек тире нужно городить регулярку с послаблением на регистр чисел
      Ответить

      • ava roman-kashitsyn 02.07.2012 10:48 # 0

        > вместо замены точек тире
        дык точка там вроде не экранирована и обозначает "любой символ"
        Ответить

        • ava Lure Of Chaos 02.07.2012 11:02 # 0

          ну а что там еще может быть в датах? это просто автор не учел
          Ответить

          • ava roman-kashitsyn 02.07.2012 11:03 # 0

            Там скорее не замена точек тире, а изменение порядка следования чисел на обратный. Локали не для нас.
            Ответить

            • ava domaster 02.07.2012 11:08 # 0

              Да, формат даты здесь изменяется регуляркой. :) Через GET передается d.m.Y, а потом оно перестраивается в Y-m-d для sql-запроса.
              Ответить

              • ava Lowezar 02.07.2012 13:08 # 0

                И как всегда strtotime() или strptime() если формат точно известен идут лесом...
                Ответить

    • ava Vindicar 02.07.2012 12:36 # +4

      Это такой корявый плейсхолдер в поле ввода. Т.е. по дефолту написано "Поиск", и потому эта строка игнорируется. Правда не уверен насчет пробелов.
      Ответить

    • ava Julike 02.07.2012 13:06 # +2

      О trim() как я понимаю не слышали?
      Ответить

      • ava Vindicar 02.07.2012 16:33 # +1

        Ага. Я уж молчу как это будет локализовываться...
        Ответить

        • ava Vasiliy 02.07.2012 21:05 # +2

          Просто не будет. И это радует.... мне хватило футбола
          Ответить

    • ava guest 06.07.2012 11:29 # 0

      xD
      Если я пойду в магазин или пойду в магазин то я куплю колу.
      Ответить

    • ava Lure_Of_Chaos2 13.07.2012 10:05 # 0

      trim()? не, не слышал.
      Ответить

      • ava rat4 13.07.2012 11:19 # 0

        >Lure_Of_Chaos2

        И зачем люди ставят цифры в конце ника?
        Ответить

        • ava guest 13.07.2012 11:29 # 0

          Тупые патаму што. _Подчеркивание_ гораздо менее палевное.
          Ответить

        • ava eth0 13.07.2012 15:26 # 0

          И не говори-ка, странные.
          Ответить

        • ava Lure_Of_Chaos2 13.07.2012 20:08 # +1

          Потому, что какой-то уёбок, занял мой ник.
          Ответить

          • ava TarasB 13.07.2012 20:20 # 0

            Как?
            Ответить

            • ava Lure Of Chaos 14.07.2012 10:53 # +1

              да у него сначала по науке закос под меня был - и без цифры, и аватарка, только вот айдишник не тот да и подчеркивания.
              я Страйкеру сказал, он циферку этому клону и поставил
              Ответить

              • ava guest 14.07.2012 15:31 # +2

                >>по науке закос под меня был - и без цифры, и аватарка, только вот айдишник не тот да и подчеркивания.

                А. Ну тогда всё правильно сделал, да. Хвалю.
                Ответить

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