1

在这个特定问题上，我一直在努力解决问题，并希望得到一些建议。我有以下结构：

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

this 被作为来自外部应用程序的指针传递给函数，如下所示：

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（复制的最后一个值）

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

4

3 回答 3

0

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

于 2013-07-16T09:01:44.797 回答
0

您需要以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 > this.maxLength （预分配的长度与字符串大小不同）时才在 assign 方法中重新分配内存。复制构造函数和赋值运算符当前被禁用，因为如果在管理堆上的内部内存时未正确实施，它们可能会导致问题。

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

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

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

于 2013-07-16T09:39:57.250 回答
0 
RequestInfo[1] = "1";

没有按照你的想法去做。要么是

RequestInfo[1].string = "1";

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

RequestInfo->string = "1";

ifRequestInfo是指向单个MqlStr对象的指针。

于 2013-07-16T08:42:32.480 回答