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

    +50.1

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    ...
    double total, free;
    GetCensoredServer()->GetCensoredMemory(total, free);
    if (free<0.) // <-- чудесная проверка
    {
    	MessageBox(AfxGetMainWnd()->m_hWnd, censored("Please check censored for enough memory!")), _T(""), MB_OK|MB_ICONWARNING);
    	return false;
    }
    ...

    Найдено в рабочем коде. Некоторые идентификаторы изменены.

    Запостил: Xentrax, 01 Сентября 2009

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

    • И в чём проблема?
      Ответить
      • Свободная память меньше 0. Как себе представляешь?
        Ответить
        • овердрафт
          Ответить
        • // первый guest

          Ну мало ли что GetCensoredMemory может возвращать, надо смотреть её поведение. Странно ещё, что она объём доступной памяти возвращает в double...
          Ответить
          • В этом и говнокод. Она не должна возращать такую не очевидную тупость...
            Ответить
    • Из говнокодерского замечу ещё стрелку -> в "GetCensoredServer()->GetCensoredMemory"
      Ответить

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