Question

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

我正在尝试一些C风格的代码，但我找不到我的bugg。任何人都可以看到错误吗？

注意：我知道内存泄漏很少（我稍后会修复）

#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。

Answer 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！]

哦，更进一步：将char*传递给std::string假设您的字符串是C风格的字符串。因此，如果字符串中有一个零字节，它将无法工作。拥有一个只是一个长度的数据结构和一个动态分配的char数组可能会更好。

我也相信你会从std::string类型复制错误的字节，因为你只是将字符串的地址转换为char *，那就是“object std :: string”，而不是实际的字符串内容 - 你需要“c_str（）”。

Answer 2

一个漏洞来源：

string s = Serialize(x);

应该是

string s = Serialize<int>(x);

我的代码内存操作中的错误

2 个答案: