1. ActionScript / Говнокод #4580

    −155

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    function requiredVersion(required) 
{ 
  version = getVersionNumber();
  if (version.nl < required.nl) return(false);
  if (version.nl > required.nl) return(true);
  if (version.n2 < required.n2) return (false);
  if (version.n2 > required.n2) return (true);
  if (version.n3 < required.n3) return (false); 
  if (version.n3 > required.n3) return (true);
  if (version.n4 < required.n4) return (false); 
  if (version.n4 >= required.n4) return (true);
}

    Запостил: psina-from-ua, 10 Ноября 2010

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

    • ava KirAmp 10.11.2010 18:58 # +4

      Что-то какой-то бред, ибо нехера не пойму применение этого гк
      Ответить

      • ava Oleg_quadro 10.11.2010 23:11 # 0

        Сравнение версий.
        Допустим, надо сравнить версию 1.2.3.4 с 1.3.4.5

        Версия 1.2.3.4 записывается так:
        version.n1 = 1;
        version.n2 = 2;
        version.n3 = 3;
        version.n4 = 4;

        Версия 1.3.4.5 записывается так:
        required.n1 = 1;
        required.n2 = 3;
        required.n3 = 4;
        required.n4 = 5;

        Ну и начинаем последовательно сравнивать числа (то есть поля n1, n2 ...).
        Если на определенном этапе числа не совпали, даем тру или фолс.
        Если совпали -- идём дальше.

        P.S автор не я))
        Ответить

        • ava KirAmp 10.11.2010 23:43 # +2

          А не такой уж и ГК получается)..

          Автор кода не знал про оператор || и не определил тип функции. Больше косяков не видю)
          Ответить

          • ava Oleg_quadro 10.11.2010 23:59 # +1

            Если по-хорошему, то версии нужно хранить в строках ('1.2.3.4')
            В функции проверки нарезать строки версий по точкам в массивчики.

            Ну и пробегаться по массивчикам, сравнивая попарно элементы.

            Если какой-то массивчик слишком рано закончился, то считать недостающие элементы нулями.

            Тогда спокойно можно будет сравнить версии с разным числом точек,
            и само собой будет поддержка новых разрядов.

            ===
            но в, принципе, если не желания делать универсально, автору можно оставить этот код.
            Ответить

          • ava Oleg_quadro 11.11.2010 00:05 # 0

            А как тут || помогло бы?
            Если не совпала очередная пара, то следующие пары мы вообще не смотрим.
            Ответить

            • ava KirAmp 11.11.2010 13:08 # 0

              Можно было-бы написать в этом духе:
              if (version.nl < required.nl || version.n2 < required.n2) return false;
              Ответить

              • ava ReallyBugMeNot 11.11.2010 13:38 # +2

                Бред!
                2.0 < 1.1?
                Кстати, если мне не изменяет память, из-за вот таких ламерских проверок когда-то в винде то ли 95 то ли 98 minor версию в defin'ах SDK сделали 95 (98). Ибо иначе она (4.0) по мнению многих программ оказывалась младше, чем древняя 3.11.
                Правильное условие
                if ((version.n1 < required.n1) || (((version.n1 == required.n1) && (version.n2 < required.n2)) return false;
                Но для 4 компонент условие получится слишком монструозное.

                По-моему, данные код вполне нормальный, он вполне соответствует своей задаче и весьма понятный. Например, хранение версии в строке убивает контроль типов ибо "this is a wrong version" тоже строка и типа тоже версия.
                Ответить

    • ava FIZZER 10.11.2010 20:17 # +3

      Все руки за такое надо повырывать.
      Ответить

    • ava psina-from-ua 10.11.2010 21:48 # 0

      Это, кстати, пример из книги "Macromedia Flash MX. Создание игр с помощью ActionScript"
      Ответить

    • ava j0kz 10.11.2010 23:17 # −4

      напишите ваш вариант в двух словах пжл
      Ответить

      • ava Lure Of Chaos 11.11.2010 00:25 # 0

        и сложить слово СЧАСТЬЕ из букв А О П Ж
        Ответить

        • ava j0kz 11.11.2010 00:28 # 0

          правда интересно. я бы точно также сравнивал, только по-другому алгоритм бы написал
          Ответить

          • ava Oleg_quadro 11.11.2010 00:32 # 0

            Я предложил своё решение.
            http://govnokod.ru/4580#comment52050
            Ответить

            • ava j0kz 11.11.2010 00:54 # 0

              ну так смысл тотже. все гавно только в написании?
              Ответить

              • ava Oleg_quadro 11.11.2010 00:54 # 0

                "Тогда спокойно можно будет сравнить версии с разным числом точек,
                и само собой будет поддержка новых разрядов."

                Здесь, что же будет если n3 есть null?
                Что надо делать если надо поддержать 5 разряд? заводить новое поле n5, и менять код функции?
                Ответить

                • ava j0kz 11.11.2010 01:19 # 0

                  ну какбе нет. достаточно версию писать в одном формате, где n3 будет 0, а не NULL.

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

    • ava bugmenot 11.11.2010 12:01 # 0

      А на хрена поле build в ActionScript`е?
      Ответить

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