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

    +160

    1. 1
    $VAR['server'] = trim(str_replace("www.","",strtolower($_SERVER['SERVER_NAME'])));

    Угадайте что этот код должен делать, и что он делает на самом деле. Да, это ужас. Это один из многих ужасов, доставшихся мне от предыдущего программиста. До этого самым большим был explode имени файла через точку и подставление второго элемента как расширение.

    Запостил: BiSe_Trojanov, 02 Марта 2011

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

    • ava Lure Of Chaos 02.03.2011 18:44 # 0

      трим лишний, а так, кажись, код имеет смысел
      Ответить

      • ava Crazyzubr 02.03.2011 19:47 # 0

        какой такой смысл? поубивать урл нафиг?
        Ответить

        • ava Lure Of Chaos 02.03.2011 20:40 # 0

          ну во первых не урл а только имя сервера,
          а во вторых, так красивее для вывода.

          а вот для вывода ссылок полученное значение использовать низзя
          Ответить

          • ava Int 02.03.2011 21:34 # 0

            Не могу понять почему нельзя. Из-за отсутствия www. или причина более глубокая?
            Ответить

            • ava Lure Of Chaos 02.03.2011 21:51 # 0

              да, с www. и без оного это разные адреса. Большинство, конечно, делает алясы, но это не всегда
              Ответить

    • ava Uchkuma 03.03.2011 00:43 # +3

      при чем тут вообще альясы-хуясы?
      1www.ru
      3www.ru
      и подобных адресов немало.
      Ответить

      • ava BiSe_Trojanov 03.03.2011 19:57 # 0

        Именно. Дело не в алиасах, а в том как режется поддомен.

        preg_replace('|^www\\.|','',$_SERVER['HTTP_HOST']) вот правильное решение, но никак не удаление всех www. из строки
        Ответить

    • ava babaika 03.03.2011 03:44 # +1

      $_SERVER["HTTP_HOST"]?
      Ответить

      • ava BiSe_Trojanov 03.03.2011 19:59 # 0

        И это тоже. Программист видимо не знал, что имя сервера может быть произвольным, а на одной директиве может сидеть несколько совершенно разных сайтов.
        Ответить

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