VPF::new && delete - Форум программистов

bel_nikita

Дата 11.3.2004, 11:21 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Эксперт
Сообщений: 2304
Регистрация: 12.10.2003
Где: Поезд №21/22 ( ст . Прага )

Репутация: 21
Всего: 47

Хелм ми, плиз
Проблемы с удалением

Код

struct sErrorString{
  std::string **ExtraString;
  std::string   String[5];
  sErrorString(){
    ExtraString=NULL;
  };
  sErrorString(char *aExtraString, char *aString1, char *aString2, char *aString3, char *aString4, char *aString5)
  {
    ExtraString = reinterpret_cast<std::string **>(aExtraString);
    String[0]   = aString1;
    String[1]   = aString2;
    String[2]   = aString3;
    String[3]   = aString4;
    String[4]   = aString5;
  }

 // ВОТ ЗДЕСЬ ПРОБЛЕМЫ НАЧИНАЮТСЯ
  int InsertString(const int &NumStr,char const *Str)
  {
    if ( ExtraString==NULL )
    {
      ExtraString=new std::string*[5];
      memset(&ExtraString[0],NULL,5*sizeof(std::string*));
      for ( unsigned i=0; i<5; ++i )
        ExtraString[i]=new std::string(String[i]);
    }

   // ПОСЛЕ ЭТОГО ПРОГРАММА ВЫЛЕТАЕТ:
    if ( ExtraString[NumStr] )
    {
      delete ExtraString[NumStr];
      ExtraString[NumStr]=NULL;
    }
    ExtraString[NumStr]=new std::string(Str);
    return 0;
  }
  ~sErrorString()
  {
    if ( ExtraString ){
      for ( unsigned i=0; i<5; ++i )
        delete ExtraString[i];
      delete []ExtraString;
    }
  }
};

Что не правильно в коде?

--------------------

— регистрация доменов от 150 руб.

Borisff2003

Дата 11.3.2004, 11:48 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 198
Регистрация: 26.2.2004
Где: г. Уфа

Репутация: 1
Всего: 1

Цитата

ExtraString = reinterpret_cast<std::string **>(aExtraString);

А вот это зачем?
И что ты туда передаешь?, может там ерунда, вот delete и ругается

т.е. если ты передаешь туда
char *s;
организованный так,
s=new std::string*[5];
memset(&s[0],NULL,5*sizeof(std::string*));
for ( unsigned i=0; i<5; ++i )
s[i]=new std::string(String[i]);
то нормально, а ведь можно передать и такое
char *s="gdfgdfgdfg";
и это прекрастно пройдет, но до удаления указателя, просто передавай первым параметром NULL
и не мучайся

Это сообщение отредактировал(а) Borisff2003 - 11.3.2004, 11:59

--------------------

Лень, двигатель прогресса

bel_nikita

Дата 11.3.2004, 11:58 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Эксперт
Сообщений: 2304
Регистрация: 12.10.2003
Где: Поезд №21/22 ( ст . Прага )

Репутация: 21
Всего: 47

Borisff2003 неа, ошибка не в этом. это мне для начальной инициализации требуется, для совместимости со старым методом.
Ошибка у меня в другом, а именно в отсуствии конструктора копий.
Вот вставил и все заработало

PS: кому интересно, конструктор копий:

Код

  sErrorString(const sErrorString &Copy)
  {
    String[0]   = Copy.String[0];
    String[1]   = Copy.String[1];
    String[2]   = Copy.String[2];
    String[3]   = Copy.String[3];
    String[4]   = Copy.String[4];
    if ( Copy.ExtraString )
    {
      ExtraString=new std::string*[5];
      memset(&ExtraString[0],NULL,5*sizeof(std::string*));
      for ( unsigned i=0; i<5; ++i )
        ExtraString[i]=new std::string(Copy.ExtraString[i]->c_str());
    }
    else ExtraString=NULL;
  }

--------------------

— регистрация доменов от 150 руб.

srd

Дата 12.3.2004, 16:08 (ссылка)

(нет голосов)

Загрузка ...

Нереварин

Профиль
Группа: Участник Клуба
Сообщений: 951
Регистрация: 30.8.2003
Где: Владивосток

Репутация: 27
Всего: 49

Цитата

std::string **ExtraString;

Может быть проще воспользоваться std::vector<std::string>? По крайней мере, не возникало бы подобных ошибок.

--------------------

Не смей читать мою подпись!!!

bel_nikita

Дата 12.3.2004, 16:38 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Эксперт
Сообщений: 2304
Регистрация: 12.10.2003
Где: Поезд №21/22 ( ст . Прага )

Репутация: 21
Всего: 47

srd Ты прав.
Я уже исправил на std::vector<std::string>.

--------------------

— регистрация доменов от 150 руб.

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Общие вопросы \| Следующая тема »