C++ dll返回char*时new和delete的使用是否正确？

现有写法可行性判断 #

你当前的写法修复低级错误后可以跑通，但不属于合理的C++实践，存在多个风险点：

• 代码存在低级拼写错误：nulptr应为nullptr，cunk.size应为chunk.size
• 全局response未显式初始化，虽然C++全局变量默认零初始化，但多编译单元场景下存在初始化顺序风险
• 缺少字符串终止符：你申请的内存长度和响应长度完全一致，没有预留1字节补'\0'，如果Clarion将返回值当做C字符串处理会出现读越界
• 线程不安全：全局变量会导致多线程同时调用函数时出现内存竞争、内容覆盖、野指针崩溃等问题
• 调用方使用限制多：前一次调用返回的指针会在下次调用时被释放/覆盖，必须保证Clarion在下次调用前就完成内容拷贝，否则会访问无效内存
• 内存泄漏风险：全局内存没有释放逻辑，DLL多次加载卸载或频繁返回大响应时会产生不必要的内存占用

更优的实现方案 #

因为你需要兼容Clarion调用，无法使用C++标准库的智能指针和std::string，可以选择以下两种改造方案：

方案一：最小改动适配现有逻辑 #

如果你要保留「返回全局内存、调用方不需要主动释放」的约定，只需做少量修改即可规避大部分风险：

// 显式初始化全局指针，多线程场景下加thread_local修饰避免竞争
char *response = nullptr;

struct MemoryStruct {
    char *memory;
    size_t size;
};

char *function() {
    // ... 省略libcurl请求逻辑，已拿到chunk ...

    // delete[]空指针是合法操作，无需冗余判断，直接释放旧内存即可
    delete[] response;
    // 多申请1字节存储字符串终止符
    response = new char[chunk.size + 1];
    memcpy(response, chunk.memory, chunk.size);
    // 手动补终止符，避免字符串越界
    response[chunk.size] = '\0';

    // ... 省略libcurl清理逻辑 ...

    return response;
}

// 额外导出释放函数，供DLL卸载前或不需要返回值时主动调用释放内存
__declspec(dllexport) void free_global_response() {
    delete[] response;
    response = nullptr;
}

方案二：更安全的跨语言调用约定（推荐） #

彻底去掉全局变量，约定调用方用完返回值后调用你提供的释放函数回收内存，没有线程安全问题，多次调用返回的内存互不干扰：

struct MemoryStruct {
    char *memory;
    size_t size;
};

__declspec(dllexport) char *function() {
    // ... 省略libcurl请求逻辑，已拿到chunk ...
    char *res = new char[chunk.size + 1];
    memcpy(res, chunk.memory, chunk.size);
    res[chunk.size] = '\0';
    // ... 省略libcurl清理逻辑 ...
    return res;
}

// 导出通用释放函数，Clarion拿到返回值用完后必须调用此函数释放内存
__declspec(dllexport) void free_curl_response(char *ptr) {
    delete[] ptr;
}

内容的提问来源于stack exchange，提问作者look_up