Question

我试图在c ++中实现自己的复制字符串函数

void pcstrdup(const char* szStr, char* szStrCpy)
{
    int nLen = strlen(szStr);
    if (!nLen)
        throw "Error : attempt copying an empty string";
    ++nLen;
    szStrCpy = static_cast<char*>(malloc(sizeof(char) * nLen));
    if (!szStrCpy)
        throw "Error : memory allocation failed";
    for (int i = 0; i < nLen; i++)
    {
        szStrCpy[i] = szStr[i];
    }
}

我已经调试并检查了字符是否被复制并且除了\ 0字符之外它们都复制了它们，当它到达那一点时我得到了一个异常

Unhandled exception at 0x011A5BA1 in assignment2.exe: 0xC0000005: Access violation reading location 0x00000000.

这让我有了这个功能：

static size_t __CLRCALL_OR_CDECL length(const _Elem *_First)
    {   // find length of null-terminated string
    return (*_First == 0 ? 0
        : _CSTD strlen(_First));
    }

请注意，istrlen（）函数是我写的函数。

int istrlen(const char* szStr)
{
    int count = 0;
    for (int i = 0; szStr[i] != NULL; i++)
    {
        ++count;
    }
    return count;
}

Answer 1

问题出在你的函数声明中：

void pcstrdup(const char* szStr, char* szStrCpy)

当您为内存分配szStrCpy时，调用者看不到更改，因为指针是按值传递的。当您的pcstrdup返回时，分配给szStrCpy的内存将丢失，并且调用者会看到旧值（在这种情况下，它看起来像是NULL）。

您可以通过引用传递szStrCpy来解决此问题：

void pcstrdup(const char* szStr, char *&szStrCpy)

更好的是，您应该返回szStrCpy而不是将其作为第二个参数：

char *pcstrdup(const char* szStr)

复制一个字符串函数

1 个答案: