memmove代码（）

Question

以下代码段显示了memmove()的实现。

void my_memmove(void* dest, const void* src, size_t size)
{
    unsigned int i;

    char* d = (char*)dest;
    char* s = (char*)src;

    if( s > d )
    {
            for( i = 0; s[i] && i < size; ++i )
                    d[i] = s[i];
    }
    else
            for( i = size-1; d[i] && i >= 0; --i )
                    d[i] = s[i];
}

int main()
{
    char my_str[] = "abcdefgh";

    char str[] = "abcdefgh";

    my_memmove(my_str+1, my_str, 4);

    memmove(str+1, str, 4);

    printf("%s %s\n", my_str, str);

    return 0;
}

我得到的是output：

 aabcdfgh  

为什么my_memmove（）无法正常工作（它会更改my_str，使其输出空字符串）？

Answer 1

对于无符号整数，条件i >= 0始终为true。用＆＃34; -1＆＃34;写出惯用循环会更好。逆转偏见：

for (i = 0; i != size; ++i)
    d[size - i - 1] = s[size - i - 1];

此外，附加条件s[i] &&看起来很简单。

Answer 2

您假设src和dest指向 指向同一个内存块。

...
char* d = (char*)dest;
char* s = (char*)src;

if( s > d )
...

这意味着如果有人用src调用你的函数并且dest指向内存中的2个不同位置，你将得到未定义的行为。

这非常糟糕，只需使用内置的memmove（）。