Мусор с указателями в классе, C++

У вас утечка памяти в main; вы создаете указатель с new, но не вызываете впоследствии delete.

Если вы хотите просто создать объект типа Header, который будет уничтожен при выходе из main, просто объявите его как «Header header;». Если вы хотите создать постоянный указатель, вы должны использовать new, как и вы, но обязательно delete header; и некоторые точка перед окончанием программы.

Боюсь, здесь есть ряд проблем.

Для начала, char ImageCordsRep[1]; не работает ... строка всегда заканчивается нулем, поэтому, когда вы выполняете strcpy(ImageCordsRep,"G");, вы переполняете буфер.

Также было бы неплохо завершить все эти строковые буферы нулевым значением в вашем конструкторе, чтобы они всегда были допустимыми строками.

Еще лучше было бы использовать класс строк вместо массивов символов или, по крайней мере, использовать strncpy для предотвращения переполнения буфера, если входящие строки больше, чем вы ожидаете.

Что-то другое...

Будьте осторожны, чтобы не использовать imageLocation после удаления объекта заголовка. Часто лучше скопировать строку из объекта, чем получать на него указатель. В этом случае все могло быть нормально, в зависимости от остальной части кода.

Header * header = new Header;
header->SetHeader(t[5],t[6],corners->location);
char * imageLocation = header->ImageGeoLocation;

Ваша проблема в том, что ImageGeoLocation - это мусор или у вас утечка памяти?

Если ваш код написан как таковой:

Header * header = new Header;
header->SetHeader(t[5],t[6],corners->location);
char * imageLocation = header->ImageGeoLocation;
delete header;
printf("ImageLocation is %s", imageLocation);

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

Если это не так, отладьте свой метод SetHeader. Заполняется ли ImageGeoLocation данными, как вы ожидаете? Если это так, то imageLocation должен указывать на действительные данные, если не будет пропущенного кода, который впоследствии повредит ImageGeoLocation. Память о том, какое окно смотрит на ImageGeoLocation, может помочь, поскольку вы сможете пошагово просмотреть свой код и увидеть, какая строка на самом деле изменяет ImageGeoLocation там, где вы этого не ожидаете.

Спасибо, Торлак и другие за столь быстрый ответ. По сути, imageLocation заполняется нормально, если перед ним нет другого кода. Например, у меня есть этот список строк, который в основном содержит имена файлов.

    AnsiString fileType ("*.jpg");
    AnsiString path = f + fileType;
    WIN32_FIND_DATA fd;
    HANDLE hFindJpg = FindFirstFile(path.c_str(),&fd);

   //Find all images in folder
    TStringList * imageNames = new TStringList;

    if (hFindJpg != INVALID_HANDLE_VALUE) {
        do{

            if (!(fd.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY)){
                image = AnsiString(fd.cFileName);
                imageNames->Add(image);

                jpgFileCount++;
            }

        }while(FindNextFile(hFindJpg,&fd));
    }else ShowMessage ("Cannot find images.");

    FindClose(hFindJpg);

Теперь, когда я пытаюсь сослаться на изображение из списка непосредственно перед ним, я получаю имя изображения, помещенного в imageLocation.

 //char * imageLocation = header->ImageGeoLocation; //as expected
Image1->Picture->LoadFromFile(imageNames->Strings[j]);
char * imageLocation = header->ImageGeoLocation; //puts name of jpg file in imageLocation

Я изменил strcpy() на strncpy(), и это решило мою проблему.