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

    +993

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    if(layerHeader.mFlags&AE_ANIM_BEZIER_EFFECT)
{
		btlv=(Point*)calloc(layer->frames.size(),sizeof(Point));
		btlt=(Point*)calloc(layer->frames.size(),sizeof(Point));
		btrt=(Point*)calloc(layer->frames.size(),sizeof(Point));
		brtv=(Point*)calloc(layer->frames.size(),sizeof(Point));
		brtt=(Point*)calloc(layer->frames.size(),sizeof(Point));
		brbt=(Point*)calloc(layer->frames.size(),sizeof(Point));
		bbrv=(Point*)calloc(layer->frames.size(),sizeof(Point));
		bbrt=(Point*)calloc(layer->frames.size(),sizeof(Point));
		bblt=(Point*)calloc(layer->frames.size(),sizeof(Point));
		blbv=(Point*)calloc(layer->frames.size(),sizeof(Point));
		blbt=(Point*)calloc(layer->frames.size(),sizeof(Point));
		bltt=(Point*)calloc(layer->frames.size(),sizeof(Point));
}

    Запостил: lifemaker, 16 Марта 2012

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

    • ava niXman 16.03.2012 13:16 # −4

      так а в чем говнокод-то?
      Ответить

    • ava roman-kashitsyn 16.03.2012 13:24 # 0

      Им бы копипаст взять и отменить
      Ответить

    • ava Steve_Brown 16.03.2012 13:40 # −4 

      btlv=btlt=btrt=brtv=brtt=brbt=bbrv=bbrt=bblt=blbv=blbt=bltt=
  (Point*)calloc(layer->frames.size(),sizeof(Point));

      Fixed.
      Ответить

    • ava aquahawk 16.03.2012 13:52 # 0

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

      • ava Steve_Brown 16.03.2012 13:57 # +2

        Хм...
        union
{
  struct { PointPtr btlv, btlt, btrt, brtv, brtt, brbt, bbrv, bbrt, bblt, blbv, blbt, bltt; }
  PointPtr b[12];
};

        интересно, имеет ли смысл?
        Ответить

        • ava wecanstoptrain 16.03.2012 14:55 # +2

          главное, не забыть про танцы с #pragma pack (благо, его большая часть компиляторов понимает).

          #pragma pack(push) /* push current alignment to stack */
          #pragma pack(1) /* set alignment to 1 byte boundary */

          ...

          #pragma pack(pop) /* restore original alignment from stack */
          Ответить

        • ava aquahawk 16.03.2012 20:37 # 0

          Тут может и да, но я это писал на ActionScript 3.0 который так не умеет.
          Ответить

        • ava TheCalligrapher 16.03.2012 21:29 # +1

          Работать может, но это тем не мнее - класска говнохакинга. Автору исходного кода надо было изначально сделать енум и массив, т.е. (на правах эскиза)

          enum B { btlv, btlt, btrt, brtv, brtt, brbt, bbrv, bbrt, bblt, blbv, blbt, bltt, b_count_ };
Point* b[b_count_];
          Ответить

      • ava 3.14159265 16.03.2012 20:57 # +1

        >Это реально удобно, и нельзя их просто в массив загонять т.к. они имеют каждая своё значение.
        А что обычно люди создают массивы, в которых переменные НЕ имеют каждая своё значение.
        Ответить

        • ava aquahawk 16.03.2012 21:09 # 0

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

        • ava absolut 16.03.2012 21:12 # 0

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

          • ava 3.14159265 21.03.2012 20:25 # 0

            2 absolut, aquahawk
            Именно для этих целей придумали ассоциативные массивы.
            Ответить

    • ava absolut 16.03.2012 21:15 # +3

      Удивляет, что код заявлен как c++, а написан в c-стиле
      Ответить

      • ava 3.14159265 21.03.2012 20:23 # +5

        Такой постановки вопроса "Где здесь С++?" я еще не видел.
        Ответить

      • ava inkanus-gray 27.07.2015 16:21 # +1

        Сишник бы написал отдельный оператор разыменования и оператор взятия поля: (*layer).frames.

        Стрелочку (->) пишут крестовики.
        Ответить

        • ava kegdan 27.07.2015 16:41 # 0

          новый символ нужно ввести (->) - "крестоблядская стрелка"
          Ответить

    • ava gooseim 16.03.2012 23:14 # 0 

      Point** pnts[] = {&btlv, &btlt, &btrt, ...}

for(size_t i = 0; i < sizeof(pnts) / sizeof(pnts[0]); i++)
   *pnts[i] =(Point*)calloc(layer->frames.size(),sizeof(Point));
      Ответить

    • ava Antervis 27.07.2015 09:15 # +1

      либо это не плюсы, либо можно через new
      Ответить

    • ava guest8 09.04.2019 12:50 # −999

      показать все, что скрытоvanished
      Ответить

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