0

Linux（xubuntu 13.10）でバイナリファイルとの間で構造体を読み書きしています。最初：私はそれを開きます：

dtbFile = fopen(dtbLocation,"ab+");

次: 次の構造体を数回書き込みます (9)

typedef struct{
char flagRemoved; //This flag indicates if the struct is still used. If its not 0 (as in 00000000 or \0), the thing is removed
int value;
time_t timeStamp;
}__attribute__( ( packed ) ) DTBItem_HDD; //Prevent padding to save RAM and HDD

次に、すべてを印刷して、これが問題ないことを証明します。

flagRemoved = 0 アイテムの値: 1 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:46:23 2013 flagRemoved = 0 アイテムの値: 2 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:46:23 2013 flagRemoved = 0 アイテム値: 3 アイテム タイムスタンプ (現地時間として表現): 2013 年 10 月 21 日月曜日 11:46:23 flagRemoved = 0 アイテム値: 4 アイテム タイムスタンプ (現地時間として表現): 2013 年 10 月 21 日月曜日 11:46:23 flagRemoved = 0 アイテム値: 5 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:46:23 2013 flagRemoved = 0 アイテムの値: 6 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:46:23 2013 flagRemoved = 0 アイテムの値: 7 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:46:23 2013 flagRemoved = 0 アイテムの値: 8 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:46:23 2013 flagRemoved = 0 アイテムの値:9 アイテムのタイムスタンプ (現地時間で表現): 2013 年 10 月 21 日月曜日 11:46:23

出力する関数:

int printHDDContent(void){
//First: we are always at EOF so rewind.
rewind(dtbFile);
DTBItem_HDD hddItem;
DTBItem item; //Used to print the contents of the DTBItem_HDD
int iAmountRead; //Used to find eof
while (1){ //Cant put anything usefull here, since EOF is detected elswhere
    iAmountRead = fread(&hddItem,sizeof(hddItem),1,dtbFile);
    if (iAmountRead != 1){
        if (!feof(dtbFile)){ //Check if EOF, and if not its error time!
            return 11;
        }
        break; //EOF, break out of the while loop
    }
    //Convert it to a DTBItem and print it, but only if flagRemoved is still 0
    printf("flagRemoved = %d\n",hddItem.flagRemoved);
    if (!hddItem.flagRemoved){ //Check the flagRemoved
        item.value = hddItem.value;
        item.timeStamp = hddItem.timeStamp;
        printDTBItem(&item);
    }
}
return 0;
}

それで、これが私の質問に行き着くところです。次にやりたいことは、flagRemoved プロパティを 0 以外に設定して、アイテムを削除済みとしてマークすることにより、ファイルからアイテムを削除することです。このために、次のコードを書きました。

if (shouldBeRemoved){
            int itemSeek = -1 * (int)sizeof(hddItem);
            printf("Removing..(%d, %zu)\n",hddItem.value, hddItem.timeStamp);
            iErr = fseek(dtbFile,itemSeek,SEEK_CUR); 
            if (iErr) return 11; //Error while seeking (unknown file error)
            hddItem.flagRemoved = 1; //Set the removedFlag
            iErr = fwrite(&hddItem,sizeof(DTBItem_HDD),1,dtbFile); //Write the item back to the file
            if (iErr != 1) return 11; //Should be one, bacuse one item is written. If not: unknown error with file
        }

そして、ここで問題が発生します。ここで fwrite を使用すると、常にファイルの最後までシークし、カーソルの先頭からではなく、この新しいオブジェクトを書き込みます。これにより、古いオブジェクトが上書きされます。なぜこうなった？私がここで読んだことによると：CPPリファレンスでは、ストリーム内の現在の位置に書き込む必要がありますが、これはすべてのアイテムを削除しようとした後の出力です:

flagRemoved = 0 アイテムの値: 1 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:56:08 2013 flagRemoved = 0 アイテムの値: 2 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:56:08 2013 flagRemoved = 0 アイテムの値: 3 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:56:08 2013 flagRemoved = 0 アイテムの値: 4 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:56:08 2013 flagRemoved = 0 Item値: 5 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:56:08 2013 flagRemoved = 0 アイテムの値: 6 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:56:08 2013 flagRemoved = 0 アイテムの値: 7 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:56:08 2013 flagRemoved = 0 アイテムの値: 8 アイテムのタイムスタンプ (現地時間として表現): Mon Oct 21 11:56:08 2013 flagRemoved = 0 アイテムの値:9 アイテムのタイムスタンプ (現地時間で表現): 2013 年 10 月 21 日月曜日 11:56:08 flagRemoved = 1

末尾の追加の 1 は、削除されたもう 1 つの項目を示します。

4

1 に答える 1

2

まあ、全体的な間違いは、読み取り/書き込み/追加モードで開いていたことです。このモードは、何があっても常に書き込みを追加します。ファイルが存在することを保証できないため、ファイルを開くコードを次のように変更しました。

int initDatabase(char* dtbLocation){
int iErrCode = 0;
if (dtbFile == NULL){
    dtbFile = fopen(dtbLocation,"ab+"); //Open the file in read/write-append binary mode (do not overwrite current content, create if not exists) at the end of the file
    if (dtbFile == NULL){
        iErrCode = 2; //The file could not be opened
    } else if (fclose(dtbFile) == EOF){ //Close the file to re-open it in the correct way 
        iErrCode = 12;  //Something went wrong while closing file
    } else {
        dtbFile = fopen(dtbLocation,"rb+"); //Open in read/write, file must exist
        if (dtbFile == NULL){
            iErrCode = 2; //The file could not be opened
        }   
    }   
}
} else {
    iErrCode = 1; //The file was already opened
}
return iErrCode;

}

于 2013-10-21T10:35:36.090 に答える