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

    +15

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    for(std::list<Eff_t*>::iterator i = m_effects.begin(); i != m_effects.end(); ++i)
	{
		Rot3DEff_t* pRot3DEff = dynamic_cast<Rot3DEff_t*>(*i); 
		//иначе вместо деструктра Rot3DEff_t вызывается деструктор Eff_t
		//если этого не делать не освободится текстура m_pText класса Rot3DEff_t
		if (pRot3DEff)
			delete pRot3DEff;
		else
			delete *i;
	}

    Запостил: lifemaker, 16 Апреля 2013

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

    • ava govnomonad 16.04.2013 08:49 # +6

      что-то вспомнился фейл кутешников, когда они где-то в недрах qtmultimedia-5 забыли сделать деструктор виртуальным
      Ответить

    • ava bormand 16.04.2013 09:21 # 0

      И так для всех остальных потомков Eff_t, у которых появились новые поля...
      Ответить

    • ava bot-minurast 16.04.2013 21:41 # 0

      Амнистия. Теперь и на С++
      Ответить

    • ava Elvenfighter 17.04.2013 00:42 # 0

      > dynamic_cast

      нужно больше RTTI! А вообще - автор любит тяжёлый труд руками, не в ту отрасль пошёл :)
      Ответить

      • ava Yuuri 17.04.2013 11:56 # 0

        > нужно больше RTTI!
        #include <typeinfo>

for(std::list<Eff_t*>::iterator i = m_effects.begin(); i != m_effects.end(); ++i)
{
	std::type_info type = typeid(*i);
	if (type == typeid(Rot3DEff_t *))
	{    
		Rot3DEff_t* pRot3DEff = static_cast<Rot3DEff_t*>(*i);
		delete pRot3DEff;
	}
	else
		delete *i;
}
        Ответить

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