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

    +160

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

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

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

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

    • трим лишний, а так, кажись, код имеет смысел
      Ответить
      • какой такой смысл? поубивать урл нафиг?
        Ответить
        • ну во первых не урл а только имя сервера,
          а во вторых, так красивее для вывода.

          а вот для вывода ссылок полученное значение использовать низзя
          Ответить
          • Не могу понять почему нельзя. Из-за отсутствия www. или причина более глубокая?
            Ответить
            • да, с www. и без оного это разные адреса. Большинство, конечно, делает алясы, но это не всегда
              Ответить
    • при чем тут вообще альясы-хуясы?
      1www.ru
      3www.ru

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

        preg_replace('|^www\\.|','',$_SERVER['HTTP_HOST']) вот правильное решение, но никак не удаление всех www. из строки
        Ответить
    • $_SERVER["HTTP_HOST"]?
      Ответить
      • И это тоже. Программист видимо не знал, что имя сервера может быть произвольным, а на одной директиве может сидеть несколько совершенно разных сайтов.
        Ответить

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