-2

我不确定我是否可以在这里发布这些类型的问题,请让我知道您的想法,如有必要,我可以删除帖子。

我正在尝试一些 C 风格的代码,但我无法找到我的错误。任何人都可以看到错误吗?

注意:我知道内存泄漏很少(我稍后会修复)

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

using namespace std;

size_t returnSize(const char* s)
{
       string string(s);
       return string.size();
};

size_t returnSize(const int& i)
{
       return sizeof(i);
};

size_t returnSize(const char& c)
{
    return sizeof(char);   
};

template<typename T>
string Serialize(const T& t)
{
    T* pt = new T(t);
    char CasttoChar[returnSize(t)];
    for (int i =0 ;i<returnSize(t);i++)
    {
        CasttoChar[i] = (reinterpret_cast<const char*>(pt)[i]);
    }

    char* pX = (char*)malloc(sizeof(char) * (returnSize(t) + 1));

    // I save size in byte 0 
    pX[0] = (char)returnSize(t);

    //I save value in subsequent bytes.
    for (int i = 1 ; i<=returnSize(t) ; i++)
    { 
        pX[i] = CasttoChar[i];                         
    }

    string returnString(pX);
    free(pX);

    return returnString;     
};
template<typename T>
T DeSerialize(const string& s)
{
     const char * cstr = s.c_str();

     int sizeofData = (int)cstr[0];

    char* cp = (char*)malloc(sizeof(char) * sizeofData);
    for (int i =0 ;i<sizeofData;i++)
    { 
        cp[i] = cstr[i];                          
    }

    T* result= reinterpret_cast<T*>(cp);

    return *result;

}
int main(int argc, char *argv[])
{
    int x = 10;
    string s = Serialize(x);
    cout << DeSerialize<int>(s);
  /*    
    I need to see: 
    10 as output
    now I see 4
  */    
    system("PAUSE");
    return EXIT_SUCCESS;
}

所以基本上我序列化数字 10,当我反序列化它时,我得到 4。

4

2 回答 2

1

您正在复制错误的字节:

for (int i =0 ;i<sizeofData;i++)
{ 
    cp[i] = cstr[i];                          
}

由于 cstr[0] 是长度,它应该是:

for (int i =0 ;i<sizeofData;i++)
{ 
    cp[i] = cstr[i+1];                          
}

[另外,您需要确保您的琴弦永远不会超过 128 的大小!]

哦,更进一步:将 a 传递给char*astd::string假定您的字符串是 C 风格的字符串。因此,如果您的字符串中有一个零字节,它将不起作用。拥有一个只有长度和动态分配的字符数组的数据结构可能会更好。

我也相信您会从类型中复制错误的字节std::string,因为您只是将字符串的地址转换为 char *,这是“object std::string”,而不是实际的字符串内容 - 您需要“c_str( )“ 为了那个原因。

于 2013-01-13T00:06:33.003 回答
0

错误来源之一:

string s = Serialize(x);

应该

string s = Serialize<int>(x);
于 2013-01-13T00:07:28.790 回答