1. Си / Говнокод #11265

    +137

    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
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    #define FMT_2_LEN  16
#define FMT_4_LEN  24
// и т.д. всего около десятка форматов

void calc_check_code(const unsigned char * from, unsigned fmt, unsigned * code)
{
    switch (fmt) {
    case 2:
        //...
        memset(data, 0, sizeof(FMT_2_LEN));
        // выборочное наполнение data из from
        make_code(data, FMT_2_LEN, code);
        break;
    case 4:
        //...
        memset(data, 0, sizeof(FMT_4_LEN));
        // выборочное наполнение data из from
        make_code(data, FMT_2_LEN, code);
        break;
    // для всех остальных аналогично
}

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

    Запостил: defecate-plusplus, 20 Июня 2012

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

    • ava Steve_Brown 20.06.2012 17:16 # +4

      Если не следить за чистотой кода, могут появиться насекомые. В данном случае это, по-видимому, FMT_2_LEN вместо 4 и memset(...sizeof(...))?

      (да, помню как-то, нужно было выделить size байт, тоже по инерции написал p = malloc(sizeof(size)); . Потом долго не мог понять, почему все глючит.)
      Ответить

    • ava sayidandrtfm 20.06.2012 18:09 # +2

      sizeof() не нужен и копипастерство?
      Ответить

      • ava defecate-plusplus 20.06.2012 19:15 # +3

        да, не нужен
        коду 100500 лет
        писал его тогдашний тимлид
        Ответить

        • ava vistefan 20.06.2012 20:27 # +5

          Я извиняюсь за оффтоп, но
          Уважаемый Обосраться++, ты зачем усы сбр аву сменил?
          Ну правда, на старую любовался, хотя бы скажите, что за красавец это был.
          Ответить

          • ava defecate-plusplus 21.06.2012 07:32 # +1

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

            • ava TarasB 21.06.2012 10:08 # +3

              а издали на борю м. похож
              Ответить

              • ava defecate-plusplus 21.06.2012 10:33 # +5

                Тарас, может мы зря грешим на орлежку, что это он тебя регистрирует на разных сайтах?
                Ответить

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