1. C++ / Говнокод #4427

    +168

    1. 1
    2. 2
    3. 3
    4. 4
    #define loopv(v)    for(int i = 0; i<(v).length(); i++)
#define loopvj(v)   for(int j = 0; j<(v).length(); j++)
#define loopvk(v)   for(int k = 0; k<(v).length(); k++)
#define loopvrev(v) for(int i = (v).length()-1; i>=0; i--)

    Может я чего недопонимаю?
    Это из Cube 2: Sauerbraten

    Запостил: Werdn, 23 Октября 2010

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

    • ava Мистер Хэнки 23.10.2010 20:11 # 0

      Ъ-Си стайл
      Ответить

    • ava frp 23.10.2010 21:27 # 0

      ИМХО имеет смысл только первый и последний.
      Остальные, возможно, временный костыль, который забыли выпилить.
      Ответить

      • ava Werdn 23.10.2010 22:12 # +1

        чем сложно вписать человеческие циклы в коде и не плодить невесть что?
        Ответить

        • ava nil 24.10.2010 10:57 # 0

          Может, хотели абстрагироваться от способа хранения?
          Например, если реализовать в виде списка, то меняется одна строка на что-то вроде:
          for(list_elem* i=v; i; i=i->next)
          Ответить

          • ava xXx_totalwar 24.10.2010 11:05 # 0

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

            • ava nil 24.10.2010 11:12 # 0

              Ну, мало ли. Может, начальство не дает брать другой язык (как у нас). И вообще, чего мы хотим от игры с таким названием!
              Ответить

              • ava Werdn 24.10.2010 12:43 # 0

                Cube 2 — где там начальство?
                зато макросов там море, некоторые решают задачу через «жпоа»
                например генерируется уникальное имя для функции некоторой команды в скриптовом языке при помощи номера строки вызова макроса. Следовательно в разных файлах нельзя объявлять эти самые команды.
                Сам не шибко силен в с++, но там выстроена иерархия структур с функциями внутрях (отчего не классы — может так и лучше, не знаю).
                И еще много разного, от чего я думаю что я точно идиот.
                Ответить

                • ava nil 24.10.2010 12:45 # 0

                  Ну, инвесторы, или кто там кроме начальства:) Лиды.

                  Это надо у них спросить, чего они извращаются. Может, и объяснят!
                  Ответить

                  • ava danilissimus 06.11.2010 20:00 # 0

                    там никого нет. игру разрабатывают 4, может 5 человек.
                    Ответить

            • ava Govnoeb 24.10.2010 19:58 # 0

              например Haskell?
              Ответить

          • ava absolut 24.10.2010 12:11 # +2

            итераторы в массы
            Ответить

            • ava Govnoeb 24.10.2010 20:01 # 0

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

              • ava absolut 24.10.2010 20:47 # 0

                Значит надо сделать несколько команд. А то получается что стадо бежит со скоростью самого медленного.
                Ответить

      • ava Kornew 24.10.2010 00:05 # 0

        остальные по-ходу для Вложенных циклов...
        Ответить

    • ava absolut 24.10.2010 01:31 # +2

      Использование #define там где без него можно обойтись - злейшее зло.
      Ответить

      • ava Oops 24.10.2010 17:39 # −4

        Вообще, препроцессор Си не самая приятная вещь. :(
        Ответить

        • ava absolut 25.10.2010 16:21 # 0

          не всё то приятно, что полезно
          Ответить

          • ava Oops 26.10.2010 19:13 # 0

            Не спорю. Однако, использовать его надо аккуратно и осмотрительно, а не то может получиться не слишком красиво. Да и не о дефайнах я мысль вёл, а больше об инклудах.
            Ответить

    • ava Werdn 24.10.2010 01:40 # 0

      Во, вопрос назрел!
      Стоит ли это конкретно выправить, написать комментарий, что так делать нельзя и отправить кто там поддержкой всего этого занимается в виде патча?
      Ответить

      • ava Govnoeb 24.10.2010 20:10 # 0

        хз. еще неплохо было бы переименовать length() в size() ибо все должно быть похоже на STL
        Ответить

        • ava Werdn 24.10.2010 20:45 # 0

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

    • ava ctm 24.10.2010 20:47 # 0

      а может обфускация?
      Ответить

    • ava TheCalligrapher 25.10.2010 02:12 # 0

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

      Именно этого пывтались достичь авторы вышепроцитированных макросов. Идея - хорошая и похвальная. Реализация - говнокодовая. Есть такой стиль говнокодирования, когда самый верхний целочисленый цикл - всегда по `i`, вложенный в него - по `j`, а в него - по `k`. Причем независимо от того, по чему именно ведется итерирование. Вот под такой стиль организации циклов и заточены вышеприведенные макросы.
      Ответить

      • ava ctm 25.10.2010 07:33 # 0

        тогда уж
        #define loopiv(i,v) for(int i = 0; i<(v).length(); i++)
        Ответить

    • ava ngry 25.10.2010 15:43 # −7

      показать все, что скрытовы таки нихуя не шагите: есть такая рассово французкая либа CImg, таки матана вам тоже не понять, для чего она была создана, но в ней таки есть кодт и похуже, и не говно, а вы кушайте, кушайте:
      #define cimg_for_out1(boundi,i0,i1,i) \
 for (int i = (int)(i0)>0?0:(int)(i1)+1; i<(int)(boundi); ++i, i = i==(int)(i0)?(int)(i1)+1:i)

      и многа, многа такога... лососнули тунцов?
      Ответить

    • ava 3.14159265 29.10.2010 20:56 # +2

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

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