为什么指针在初始化后保持为nullptr [重复]

问题描述 投票:0回答:1

我写了二进制解析器,它读取和打印有关文件的各种信息。我希望直接使用E32ImageHeader数据来代替函数调用。

我传递nullptr初始化变量,函数返回变量后仍然是nullptr。在GetFileLayout()中,gdb打印iHdr的各种字段值,还有一些正确的其他需要修改。

代码看起来正确,但工具打印“Header not recognized !!!”,二进制代码测试kf__speedups_SDK.pyd取here

main.cpp中:

#include <iostream>
#include "e32info.h"
#include "e32parser.h"

using namespace std;

int main()
{
    E32ImageHeader *hdr = nullptr;
    E32Parser *parser = new E32Parser("AlternateReaderRecogE32.dll", hdr);
    parser->GetFileLayout(hdr);
    if(!hdr)
        cout << "Header not recognized!!!\n";
//    E32Info *f = new E32Info("h", "AlternateReaderRecogE32.dll");
//    f->Run();
    cout << "Hello world!" << endl;
    delete parser;
    return 0;
}

二进制解析器头文件e32parser.h:

#ifndef E32PRODUCER_H
#define E32PRODUCER_H

class E32ImageHeader;

class E32Parser
{
    public:
        E32Parser(const char* aFilename, E32ImageHeader *aHdr);
        ~E32Parser();
        void GetFileLayout(E32ImageHeader *result);
    private:
        void ReadFile();

    private:
        E32ImageHeader *iHdr = nullptr;

        const char *iFileName = nullptr;
        char *iBufferedFile = nullptr;
        size_t iE32Lenth = 0;
}

它是如何工作的e32parser.cpp:

#include <fstream>
#include <cstring>
#include <cstdlib>

#include "e32common.h"
#include "e32parser.h"

E32Parser::E32Parser(const char* aFilename, E32ImageHeader *aHdr):
    iHdr(aHdr), iFileName(aFilename)
{
}

E32Parser::~E32Parser()
{
    delete iBufferedFile;
}

将二进制文件读入内部缓冲区

void E32Parser::ReadFile()
{
    std::fstream fs(iFileName, fs.binary | fs.in);
    if(!fs)
        throw;
    fs.seekg(0, fs.end);
    int lenth = fs.tellg();
    fs.seekg(0, fs.beg);

    iBufferedFile = new char[lenth];
    fs.read(iBufferedFile, lenth);
    fs.close();;
}

void E32Parser::GetFileLayout(E32ImageHeader *result)
{
    ReadFile();

    iHdr = (E32ImageHeader*)iBufferedFile;
    result = (E32ImageHeader*)iBufferedFile;
}
c++ pointers binaryfiles
1个回答
0
投票

正如评论中指出的那样,您的问题是您正在修改传入的指针的COPY。请考虑以下小例子:

void modify(int* p) { p = new int{10}; }

int main()
{
    int* n = nullptr;
    modify(n);
    if (n)
        std::cout << *n;
}

这个程序的输出是什么(它泄漏了内存)。 modify不会改变原始值,它会更改n的副本。对此和您的程序的修复是通过引用传递指针。

void modify(int*& p) { p = new int{10}; }

void E32Parser::GetFileLayout(E32ImageHeader*& result)

但是,我认为更好的修复只是返回指针,以防止此问题:

E32ImageHeader* E32Parser::GetFileLayout()
{
    ...
    iHdr = (E32ImageHeader*)iBufferedFile;
    return iHdr;
}

但是自从iHdr传入E32Parser后,您可以将其更改为参考

© www.soinside.com 2019 - 2024. All rights reserved.