C ++类中带有指针的垃圾

Question

我正在使用Borland Builder C ++。 我有内存泄漏，我知道一定是因为我创建了这个类，但是我不确定如何解决它。 请查看我的代码-任何想法将不胜感激！

这是.h文件：

#ifndef HeaderH
#define HeaderH
#include <vcl.h>
#include <string>
using std::string;
class Header {

 public:
    //File Header
    char FileTitle[31];
    char OriginatorName[16];

    //Image Header
    char ImageDateTime[15];
    char ImageCordsRep[2];
    char ImageGeoLocation[61];

    NitfHeader(double latitude, double longitude, double altitude, double heading);
    ~NitfHeader();
    void SetHeader(char * date, char * time, double location[4][2]);  


 private:

    void ConvertToDegMinSec (double angle, AnsiString & s, bool IsLongitude);
    AnsiString ImageDate;
    AnsiString ImageTime;
    AnsiString Latitude_d;
    AnsiString Longitude_d;
    double Latitude;
    double Longitude;
    double Heading;
    double Altitude;

};

这是一些.cpp文件：

void Header::SetHeader(char * date, char * time, double location[4][2]){
    //File Header
strcpy(FileTitle,"Cannon Powershot A640");
strcpy(OperatorName,"Camera Operator");

   //Image Header
//Image Date and Time
   ImageDate = AnsiString(date);
   ImageTime = AnsiString(time);
   AnsiString secstr = AnsiString(ImageTime.SubString(7,2));
   AnsiString rounder = AnsiString(ImageDate.SubString(10,1));
   int seconds = secstr.ToInt();
    //Round off seconds  - will this be necessary with format hh:mm:ss in text file?
   if (rounder.ToInt() > 4) {
     seconds++;
    }
   AnsiString dateTime = ImageDate.SubString(7,4)+ ImageDate.SubString(4,2) + ImageDate.SubString(1,2) + ImageTime.SubString(1,2)
                    + ImageTime.SubString(4,2) + AnsiString(seconds);
   strcpy(ImageDateTime,dateTime.c_str());

   //Image Coordinates Representation
   strcpy(ImageCordsRep,"G");

   //Image Geographic Location
   AnsiString lat;
   AnsiString lon;
   AnsiString locationlat_d;
   AnsiString locationlon_d;
   AnsiString corner;

   for (int i = 0; i < 4; i++){

     ConvertToDegMinSec(location[i][0],lat,false);
     ConvertToDegMinSec(location[i][1],lon,true);

     if(location[i][0] < 0){
        locationlat_d = 'S';
        ConvertToDegMinSec(-location[i][0],lat,false);
      }else if(location[i][0] > 0){
        locationlat_d = 'N';
     }else locationlat_d = ' ';

     if(location[i][1] < 0){
        locationlon_d = 'W';
        ConvertToDegMinSec(-location[i][1],lon,true);
     }else if(location[i][1] > 0){
         locationlon_d = 'E';
     }else locationlon_d = ' ';

     corner += lat + locationlat_d + lon + locationlon_d;

   }
   strcpy(ImageGeoLocation,corner.c_str());

}  

现在，当我在main中使用该类时，基本上我只是创建一个指针：

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

其中corners-> location是另一个类的字符串，而t [5]和t [6]都是字符串。 问题在于imageLocation不包含预期的内容，通常只是垃圾。 我已经阅读了很多有关内存泄漏和指针的内容，但是对于编程我还是很陌生，其中有些令人困惑。 任何建议都很棒！

Answer 1

恐怕这里有很多问题。

对于初学者， char ImageCordsRep[1]; 不起作用...字符串始终以null终止，所以当您执行strcpy(ImageCordsRep,"G"); 您正在溢出缓冲区。

最好在构造函数中以null终止所有这些字符串缓冲区，因此它们始终是有效的字符串。

更好的方法是使用字符串类而不是char数组，或者至少使用“ strncpy”来防止缓冲区超载（如果传入的字符串大于您的预期）。

Answer 2

你的内存泄漏main ; 您使用new指针，但随后不调用delete指针。

如果你希望只创建类型的对象Header将被毁坏了，当main出口，只是宣称它是“ Header header; ”如果你想创建一个持久的指针，你应该使用new ，你做的，但一定要delete header; 在程序结束之前的某个时间点

Answer 3

您的问题是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更改到您不期望的地方。

Answer 4

我将strcpy()更改为strncpy() ，它解决了我的问题。

Answer 5

还有别的...

删除标题对象后，请不要使用imageLocation。 通常最好从对象中复制字符串，而不要获取指向它的指针。 在这种情况下，可以根据其余的代码确定。

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

Answer 6

谢谢Torlack和其他人这么快地答复。 基本上，除非我有其他代码，否则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