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

Garbage with pointers in a class, C++

本文关键字：C++ 指针      更新时间：2023-10-16

我使用的是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不包含预期的内容，而且通常只是垃圾。我读过很多关于内存泄漏和指针的文章，但我对编程还是很陌生，其中有些内容很令人困惑。任何建议都会很棒！！

恐怕这里有很多问题。

首先，char ImageCordsRep[1];不起作用。。。字符串总是以null结尾，所以当执行strcpy(ImageCordsRep,"G");时，缓冲区溢出。

在构造函数中用null终止所有这些字符串缓冲区也是一种很好的做法，这样它们总是有效的字符串。

更好的方法是使用字符串类而不是char数组，或者如果传入的字符串比您预期的要大，至少使用"strncpy"来防止缓冲区溢出。

您的内存泄漏在main中；您正在使用new创建指针，但随后不会调用delete。

如果您只想创建一个类型为Header的对象，该对象将在main退出时被销毁，只需将其声明为"Header header;"。如果您想创建持久指针，您应该像往常一样使用new，但请确保使用delete header;和程序结束前的某个点。

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。

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

其他。。。

删除标头对象后，请注意不要使用imageLocation。通常最好是从对象中复制字符串，而不是获取指向它的指针。在这种情况下，这可能是可以的，这取决于代码的其余部分。

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

感谢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
相关文章：