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

    • да, и вместо замены точек тире нужно городить регулярку с послаблением на регистр чисел
      Ответить
      • > вместо замены точек тире
        дык точка там вроде не экранирована и обозначает "любой символ"
        Ответить
        • ну а что там еще может быть в датах? это просто автор не учел
          Ответить
          • Там скорее не замена точек тире, а изменение порядка следования чисел на обратный. Локали не для нас.
            Ответить
            • Да, формат даты здесь изменяется регуляркой. :) Через GET передается d.m.Y, а потом оно перестраивается в Y-m-d для sql-запроса.
              Ответить
              • И как всегда strtotime() или strptime() если формат точно известен идут лесом...
                Ответить
    • Это такой корявый плейсхолдер в поле ввода. Т.е. по дефолту написано "Поиск", и потому эта строка игнорируется. Правда не уверен насчет пробелов.
      Ответить
    • О trim() как я понимаю не слышали?
      Ответить
      • Ага. Я уж молчу как это будет локализовываться...
        Ответить
        • Просто не будет. И это радует.... мне хватило футбола
          Ответить
    • xD
      Если я пойду в магазин или пойду в магазин то я куплю колу.
      Ответить
    • trim()? не, не слышал.
      Ответить
      • >Lure_Of_Chaos2

        И зачем люди ставят цифры в конце ника?
        Ответить
        • Тупые патаму што. _Подчеркивание_ гораздо менее палевное.
          Ответить
        • И не говори-ка, странные.
          Ответить
        • Потому, что какой-то уёбок, занял мой ник.
          Ответить
          • Как?
            Ответить
            • да у него сначала по науке закос под меня был - и без цифры, и аватарка, только вот айдишник не тот да и подчеркивания.
              я Страйкеру сказал, он циферку этому клону и поставил
              Ответить
              • >>по науке закос под меня был - и без цифры, и аватарка, только вот айдишник не тот да и подчеркивания.

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

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