Question

我一直在讨论这个问题，并希望得到一些建议。我有以下结构：

struct MqlStr // MQL String Array
{
int               len;
char              *string;
};

这是作为来自外部应用程序的指针传递给函数的：

MT4_EXPFUNC double __stdcall CheckExecutionRequests(MqlStr* RequestInfo)

在函数中我生成了许多字符串值，我需要分配给MqlStr数组的各种元素。以下工作正常：

RequestInfo[1].string = "1";
RequestInfo[2].string = "2";

但是当我使用strcpy将生成的字符串值放入数组时，它会用我复制的值覆盖整个数组。例如：

string field1 = value.substr(Demark + 1, Demark2 - Demark - 1);
strncpy(RequestInfo[1].string, field1.c_str(), field1.size());

string field2 = value.substr(Demark + 1, Demark2 - Demark - 1);
strncpy(RequestInfo[2].string, field2.c_str(), field2.size());

如果field1 = 1且field2 = 2则整个RequestInfo []数组将等于2（复制的最后一个值）

有人能指出我正确的方向吗？

Answer 1

RequestInfo[1] = "1";

没有按你的想法行事。它要么是

RequestInfo[1].string = "1";

如果RequestInfo是包含至少2个元素的MqlStr个对象的向量，或

RequestInfo->string = "1";

如果RequestInfo是指向单个MqlStr对象的指针。

Answer 2

您是否为RequestInfo元素中的.string指针分配了足够的空间？ strncpy没有为你分配空间，为此使用strdup。

Answer 3

您需要以安全的方式管理MqlStr内存，这可以通过使用标准容器（如std::string）或编写方法来分配和释放内部内存来实现。

以下是管理内部内存的简单类的示例：

#include <cstdlib>
#include <iostream>
#include <string.h>
#include <sstream>

struct MqlStr
{
public:
    int    len;
    char  *string;

    MqlStr() { init   (); }
   ~MqlStr() { dealloc(); }
    void assign(std::string& str) {
        dealloc();
        string = new char[str.length() + 1];
        strncpy(string, str.c_str(), str.length());
        string[str.length()] = 0;
        len = str.length();
    }
    void dealloc() {
        if(string != 0) delete [] string; init();
    }

private:
    void init() { string = 0; len = 0; }
         MqlStr(const MqlStr &);
    void operator= (const MqlStr &);
};

double CheckExecutionRequests(MqlStr* RequestInfo)
{
    static int callCount = 0;
    std::ostringstream stringstream; stringstream<<"callCount: "<<callCount++;
    std::string field1 = stringstream.str();
    RequestInfo->assign(field1);
    return 1.0;
}

int main(int argc, char** argv)
{
    MqlStr s[5];

    std::cout<<"First call"<<std::endl;
    for(unsigned i = 0; i < sizeof(s)/sizeof(s[0]); ++i)
        CheckExecutionRequests(s + i);

    for(unsigned i = 0; i < sizeof(s)/sizeof(s[0]); ++i)
        std::cout<<"MqlStr["<<i<<"].string = "<<s[i].string<<std::endl;

    std::cout<<"Second call"<<std::endl;
    for(unsigned i = 0; i < sizeof(s)/sizeof(s[0]); ++i)
        CheckExecutionRequests(s + i);

    for(unsigned i = 0; i < sizeof(s)/sizeof(s[0]); ++i)
        std::cout<<"MqlStr["<<i<<"].string = "<<s[i].string<<std::endl;

    return EXIT_SUCCESS;
}

使用相同的MqlStr实例执行第二次CheckExecutionRequests不会破坏内存。代码的扩展可以是字符串大小的预分配，并且只有在新的str.length＆gt;中才重新分配assign方法中的内存。 this.maxLength（预分配长度不同于字符串大小）。复制构造函数和赋值运算符当前已禁用，因为如果在管理堆上的内部内存时未正确实现它们，则会导致问题。

更简单的解决方案是使用标准容器编写结构，如下所示：

struct MqlStr
{
public:
    std::string string;
}

然后只需将字段获得的字符串分配给MqlStr字符串。

C ++字符串到char *数组结构

3 个答案: