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

    +146

    1. 1
    2. 2
    //Функция some_function возвращает данные в формате var1=val1&var2=val2
parse_str(some_function());

    Мой код примерно полуторагодовой давности.
    Разбирал старые проекты встретил и подумал - говно или имеет право на жизнь?
    Классическое

    //Функция some_function возвращает данные в формате val1:val2
    
list ($var1, $var2) = explode(':', some_function())
    ведь будет работать медленнее, не так ли? :)

    Запостил: 7ion, 02 Февраля 2011

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

    • ava gegMOPO4 02.02.2011 22:45 # 0

      А возвратить массив неспортивно?
      Ответить

      • ava 7ion 03.02.2011 00:41 # 0

        Массив из двух элементов будет затратнее хранения двух переменных.
        Ответить

        • ava bober_maniac 03.02.2011 01:01 # +13

          У вас с головой хорошо? Вы говорите о гребаных двух переменных, а рассуждаете о скорости.

          У похапистов вообще на тему скорости с головой непорядок - любой идиотский алгоритм они выдают за оптимизацию по скорости.
          Ответить

        • ava guest 03.02.2011 01:58 # +5

          1)Нужно делать не микро оптимизации, а оптимизации алгоритма.
          2)Нужно делать оптимизацию, только при наличии весомых оснований, если это требует каких-то затрат, по сравнению с более простым способом.
          Ответить

        • ava gegMOPO4 03.02.2011 09:33 # +1

          На 10 байт? Да и то не факт.
          Ответить

        • ava sectus 03.02.2011 10:06 # +5

          А разбор строки у Вас "бесплатный"?
          Ответить

        • ava ___ 03.02.2011 10:08 # 0

          >Массив из двух элементов будет затратнее хранения двух переменных.

          Как будто explode не создает массив, предварительно изнасиловав строку с данными. Оптимизация, да.
          Ответить

          • ava Lure Of Chaos 03.02.2011 12:50 # 0

            рассмотрим запись:
            <?php
$pair=explode('=',$line);
list($key,$value)=explode('=',$line);
            внимание, вопрос:

            какая запись понятнее,
            $pair[0], $pair[1], или $key,$value?
            Ответить

            • ava gegMOPO4 03.02.2011 13:16 # −1

              Что мешает сделать list($key,$value)=$pair?
              Ответить

              • ava 7ion 03.02.2011 13:18 # 0

                Что мешает сделать это сразу, а не создавать лишний массив?
                Ответить

                • ava gegMOPO4 03.02.2011 13:47 # 0

                  Я не знаю, что мешает вам сделать это сразу, а не упаковывать данные в строку, а потом парсить её обратно.

                  Вы наверное вместо $x=expr... всегда пишете parse_str("x=".urlencode(expr...))?
                  Ответить

                  • ava 7ion 03.02.2011 13:51 # 0

                    А кто вам сказал, что я так делаю?
                    Мне вообще массив не нужен, поэтому функция возвращает строку, которая парсится parse_str.
                    Это намного быстрее, чем explode и занимает меньше памяти, чем массив.
                    Ответить

                    • ava gegMOPO4 03.02.2011 14:02 # 0

                      Вы ошибаетесь, это может быть намного медленнее и требовать намного больше памяти (и скорее всего почти всегда). Не говоря уж о побочных эффектах.
                      Ответить

                      • ava 7ion 03.02.2011 14:04 # −2

                        Каких побочных эффектах?
                        Ответить

                        • ava gegMOPO4 03.02.2011 14:14 # 0

                          Потеря типа. А если не используете urlencode -- то и возможные глюки вплоть до эксплойта.

                          Вот не пойму, что вам мешает прямо в функции присвоить результат данным глобальным переменным? Большей части недостатков избежали бы, с так волнующими вас скоростью и памятью тоже было бы лучше.
                          Ответить

                          • ava 7ion 03.02.2011 14:16 # −2

                            >А если не используете urlencode -- то и возможные глюки вплоть до эксплойта.
                            Функция возвращает строго определенные значения, в которых не может быть знака равно и амперсанда.

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

              • ava Lure Of Chaos 03.02.2011 13:19 # 0

                опять?
                Ответить

          • ava 7ion 03.02.2011 12:56 # 0

            В том-то и дело, что parse_str массив не создает, а explode - создает, вне зависимости от применения list массив в памяти будет создан.
            Ответить

    • ava Lure Of Chaos 02.02.2011 23:51 # −1

      извращаетесь
      Ответить

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