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

    +133.7

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    if(no!=s.FrameCounter) 
            s.FramesGarbage++;  //выкидываем кадр
            else { s.FramesOK++; s.NakCount++; }
            curbufdata=packet-data;
            if(no==s.FrameCounter) 
            WriteAdminLog("Не готов принять отриц. квитанцию (не в ожидании подтверждения) ",no,0,TRUE);
            else if(no!=s.FrameCounter) {  
            WriteAdminLog("Получена отриц. квитанция на последний переданный кадр! Часть данных могла потеряться. Корректируется счётчик на:",no,PARAM_LONG,FALSE);
            s.FrameCounter=no;
            }

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

    Запостил: vyalcevanet, 22 Ноября 2009

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

    • не, ну а вдруг s.FrameCounter изменится))))
      Ответить
      • в многопоточной программе?

        может быть, только эту переменную меняет только один поток


        Хотя, вдруг кто-нибудь через дебагер её, сволочь, поменяет?
        Ответить
        • Сорри, про многопоточность не подумал. Дебаггер вроде при нормальном юзаньи обычно не используют)
          Ответить
    • У криворуких "особенно говнокод" получается именно на C и C++. Классический спагетти-код имени "всех обкурившихся грибов": что вижу - то пою.
      Ответить
    • Почему это в разделе С?
      Ответить
    • Ой сорри, протупил. показалось что методы вызываются.
      Ответить
    • Я это даже прочитать не могу )
      Вас бы прокляли за такой стиль )))
      Ответить
    • - Дождусь твоей выписки и перееду, а то вдруг тебя опять по голове ударят, и ты передумаешь.
      Ответить

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