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

    +68.4

    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
    char rc;
...
bool ResourceManager()
{
	static TResourceRC r;
	...
	delete r;
	...
	r=new TResourceRC(rc);
	...
	return true;
	assert(false);
};
...

    От предков проект достался. Я думал, чего он переодически необьяснимо глючит...
    ResourceManager() - вызываеться регулярно в цикле.

    Запостил: Говногость, 11 Ноября 2009

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

    • ava guest 11.11.2009 10:57 # 0

      Знатно, знатно... Только судя по коду должно быть static TResourceRC *r ?
      Ответить

      • ava guest 11.11.2009 14:31 # 0

        Да. Удалил случайно.
        Ответить

      • ava Говногость 11.11.2009 14:35 # 0

        Собственно, это из игры казуалки. Вызываеться регулярно, каждый кадр. Получаеться первый раз delete r; срабатывает не верно.
        Ответить

    • ava guest 11.11.2009 17:24 # 0

      assert(false); - после return.
      Ответить

    • ava guest 11.11.2009 21:55 # 0

      static TResourceRC *r;
      статические переменные инициализируются "0" если не указано.
      Область видимости при этом не важна. delete 0 выполняется без проблем.
      Где говнокод? ассерт за return???
      Ответить

      • ava guest 12.11.2009 02:42 # 0

        Да? Значит там у меня периодически бывает не нуль. Порча памяти?
        Ответить

        • ava guest 12.11.2009 12:12 # 0

          Нуль там может быть только один раз -- после старта программы.
          Потом _никогда_.
          Если там периодически нуль возникает то налицо порча.
          У тебя судя по всему наведённая бага, этот код ни при чём.
          Ответить

      • ava guest 12.11.2009 02:43 # 0

        Да. Есть ассерт за ретурном.
        Ответить

    • ava guest 12.11.2009 02:46 # 0

      Тут ещё в bool возращаеться из функции всегда true, хотя никогда не проверяеться.
      Использвали бы void, раз такое дело...
      Ответить

    • ava pushkoff 12.11.2009 16:41 # 0

      в дебаге r будет 0xCDCDCDCD
      Ответить

      • ava guest 12.11.2009 18:28 # 0

        Куча может при этом портиться, если запускать не из отладчика?
        Ответить

        • ava pushkoff 12.11.2009 18:31 # 0

          вообще-то это невалидный адрес, при обращении к нему будет исключение (должно быть)...
          Ответить

        • ava pushkoff 12.11.2009 18:33 # 0

          черт, вопрос не сразу понял... если не из отладчика, там может быть все что угодно, но в 99% случаев будет 0...
          Ответить

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