Загадочное поведение кода (MFC / CArray / vector)

[Алик]

Добрый день!

Есть класс:

Код: (C++)

class CBytes : public std::vector<BYTE>
  {
  public:
    CBytes()
      {}

    CBytes(const CBytes& str)
      :std::vector<BYTE>(str.begin(), str.end())
      {
      }
    CBytes& operator= (const CBytes& str)
      {
      if (this != &str)
        {
        clear();
        insert(end(), str.begin(), str.end());
        }
      return *this;
      }
  };

И есть вот этот код:

Код: (C++)

CArray<CBytes , CBytes> testArray;
CBytes killme;
testArray.Add (CBytes ());
killme = testArray[0];
testArray.Add (CBytes ());
killme = testArray[0];

   

если запустить этот код в дебаговой конфигурации – программа падает на последней строчке этого кода killme = testArray[0]

Если конкретнее - падает внутри operator= на строке insert(end(), str.begin(), str.end());


кто может объяснить данный феномен?

У меня Visual Studio 2010, Windows 7 Pro 64

Пробовал в проекте MFC Dialog-based, Unicode

[Алексей++]

а где CArray то определён ?

[Алик]

это - стандартный МФЦшный класс

[DneprSMV]

Алик,
поставь отладочные распечатки на что указывает str
в operator, проверку на NULL или посмотри в отладчике что там в памяти.
Результат insert.
CString cs1; cs1.Format("p=%p", str); AfxMessageBox(cs1);
где-то так.

[Алексей++]

Алик, а кто заставляет им пользоваться ? Может там глюк. К примеру, в CFile есть глюк с поиском - самый первый файл не находится (первый - это такой же по счёту первый, как если бы пробовать найти файлы ::FindFirstFile)

Добавлено через 39 секунд:
Вообще, зачем смешивать контейнеры, пользуйся std::

[Алик]

Алексей1153++, это - старый код. И у нас половина огромного проекта использует этот класс. Хотел разобраться в проблеме.

мне в общих чертах проблему объяснил коллега:
дело в том, что, когда добавляется второй элемент в CArray - происходит переаллокация памяти. при этом первый элемент переносится на новый адрес не через оператор=, а простым копированием памяти.
Майкрософтовская реализация вектора в качестве отладочной информации зачем-то хранит указатели на какие-то свои члены. И после копирования по новому адресу эти указатели становятся недействительными/сбойными.
В общем, CArray - зло.

Думаю, тему можно закрывать. Спасибо всем за ответы.

[Алексей++]

Алик, переопредели класс CArray, если не хочется много проекта переписывать

[Алик]

Хмм... не, слишком глобально. Сеньоры не одобрят
Да и вообще, как можно стандартный класс МФЦ переопределить?! компилер же разноется

[DneprSMV]

Алик,
("Пробовал в проекте MFC Dialog-based, Unicode ")
Отсади и попробуй в консольном запусти в main().
printf("\n\np=%p\n", &killme); итд
По крайней мере я твой код на VC6 пустил - не вешает, не вываливается.
Опции компилятора-линкера-проекта тоже стоит посмотреть-покрутить.

[Алик]

DneprSMV, дык и у нас раньше подобного косяка не было (а проекту уже лет 20). В консольном приложении - та же история. Сейчас у нас ВижуалСтудия 2010. вероятно, что-то в МФЦ донаворотили в новой версии.

уже с этим ковыряться не будем. похоже, косяк вылазит только в дебаг-версии. решили в данном конкретном месте заменить CArray на vector и не париться

[Алексей++]

Хмм... не, слишком глобально. Сеньоры не одобрят
Да и вообще, как можно стандартный класс МФЦ переопределить?! компилер же разноется

так я предлагаю не класс менять, а по всему проекту заменить имя CArray на, скажем, CArrayWithBrain. Последний надо реализовать (как вариант - скопировать туда все исходники из CArray и исправить). Для совместимости с функциями MFC сделай оператор
operator const CArray&()



Добавлено через 1 минуту и 5 секунд:

решили в данном конкретном месте заменить CArray на vector и не париться

всё равно потом опять взорвётся где-нибудь