-1

Borland BuilderC++を使用しています。メモリリークがあり、作成したこのクラスが原因である必要があることはわかっていますが、修正方法がわかりません。私のコードを見てください-どんなアイデアでも大歓迎です！

.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に期待されるものが含まれておらず、多くの場合、ゴミだけが含まれていることです。私はメモリリークとポインタについてたくさん読んだことがありますが、私はまだプログラミングに非常に慣れておらず、そのいくつかは非常に混乱しています。どんな提案も素晴らしいでしょう！

4

6 に答える 6

5

ここには多くの問題があるのではないかと心配しています。

手始めchar ImageCordsRep[1];に機能しません...文字列は常にnullで終了するため、実行する strcpy(ImageCordsRep,"G");とバッファがオーバーフローします。

コンストラクターでこれらすべての文字列バッファーをnullで終了することもお勧めです。これにより、それらは常に有効な文字列になります。

char 配列の代わりに文字列クラスを使用するか、少なくとも 'strncpy' を使用して、着信文字列が予想よりも大きい場合にバッファ オーバーランを防ぐことをお勧めします。

于 2008-10-02T17:06:56.530 に答える
2

あなたのメモリリークはにmainあります; でポインタを作成していますがnew、その後は呼び出していませんdelete。

Header終了時に破棄されるタイプのオブジェクトを作成する場合は、次mainのように宣言します。 " Header header;"永続ポインタを作成する場合は、そのまま使用する必要がありますが、プログラムの前にnew必ず使用してください。delete header;エンディング。

于 2008-10-02T17:00:27.090 に答える
1

私はに変更strcpy()しstrncpy()、それは私の問題を解決しました。

于 2008-10-02T17:43:51.620 に答える
1

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 にデータが入力されていますか? そうである場合、後で ImageGeoLocation に損害を与える省略されたコードがない限り、imageLocation は有効なデータを指している必要があります。コードをステップ実行して、予期しない場所で ImageGeoLocation を実際に変更する行を確認できるため、ImageGeoLocation を表示しているウィンドウが役立つことを思い出してください。

于 2008-10-02T17:11:45.783 に答える
0

何か他の...

ヘッダー オブジェクトが削除された後は、imageLocation を使用しないように注意してください。オブジェクトへのポインターを取得するのではなく、オブジェクトから文字列をコピーする方がよい場合がよくあります。この場合、コードの残りの部分によっては問題ない可能性があります。

Header * header = new Header;
header->SetHeader(t[5],t[6],corners->location);
char * imageLocation = header->ImageGeoLocation;
于 2008-10-02T17:07:26.700 に答える
0

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
于 2008-10-02T17:27:00.380 に答える