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

    +165

    1. 1
    $ext = substr($fileName, strrpos($fileName, '.') + 1);

    Про pathinfo никто и не слышал видимо. Код написан недавно кем-то из команды)

    Запостил: anycolor, 18 Января 2011

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

    • ava istem 18.01.2011 02:04 # +3

      ога... а ещё вдруг .htaccess захавает...
      Ответить

    • ava Lure Of Chaos 18.01.2011 12:24 # +1

      как это мне знакомо ))))
      Ответить

    • ava dwinner 18.01.2011 13:20 # −1

      Ну а где говно-то?!
      Ответить

      • ava Lure Of Chaos 18.01.2011 14:07 # +1

        есть же pathinfo, а это конструктор лего
        Ответить

      • ava Uchkuma 18.01.2011 14:12 # +1

        говнокод в том, что данный код не всегда вернет то, что хотелось бы. Если имя файла без расширения, то код вернет имя файла начиная со второго символа.
        Ответить

        • ava Lure Of Chaos 18.01.2011 14:26 # +2

          без расширения считать расширением?
          Ответить

        • ava dwinner 18.01.2011 14:51 # −2

          А pathinfo($fileName, PATHINFO_EXTENSION) вообще "ничего" (в кавычках, прошу заметить!) не вернет, если расширения нет. В любом случае нужна проверка:
          1) Либо strrpos($fileName, '.') !== false;
          2) Либо pathinfo($fileName, PATHINFO_EXTENSION) != false
          В чем же тогда разница, если тут одна строчка кода всего?!
          Ответить

          • ava anycolor 18.01.2011 16:55 # 0

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

            Еще бы через explode делали определение расширения или по отрезанию послених двух-трех символов)
            Ответить

            • ava dwinner 18.01.2011 20:07 # −3

              Суть в том, что этот код не является говном. Если я у картинки переименую расширение c .png на .txt файл тоже свой MIME type не потеряет, зато есть встроенная функция языка... bla bla bla
              Ответить

              • ava Анонимус 19.01.2011 04:15 # 0

                >>Если я у картинки переименую расширение c .png на .txt файл тоже свой MIME type не потеряет
                щито?))
                Ответить

              • ava anycolor 19.01.2011 04:31 # 0

                причем тут вообще MIME type вы тут приплели?
                Ответить

          • ava Анонимус 19.01.2011 04:17 # +1

            вернуть ничего -- это правильное поведение
            потому что расширения нет

            вернуть имя после второго символа (текущую дату, подключение к базе данных, телефон анджелы джоли) -- неправильное поведение для функции получения расширения.

            есть такое правило: поведение функции должно быть логичным.

            даже в пхп.
            Ответить

    • ava dwinner 19.01.2011 07:50 # 0

      См. 7-й коммент. Такое чувство, что я со стенкой в теннис играю... Вы походу оба на своей "волне"...
      Ответить

      • ava anycolor 19.01.2011 19:32 # +1

        ну или вы
        Ответить

      • ava Анонимус 19.01.2011 22:22 # 0

        очень трудно в одном посту объяснить почему писать красивый код лучше, чем писать говеный

        мы бессильны, увы
        Ответить

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