请大家看看这个函数究竟能找出几个错误

飞灰橙

原帖由 tan1 于 2006-6-15 17:16 发表
我写一个，大家看看有没有错误：
void *memmove(void *dst,const void *src,size_t len)
{
        size_t i=len;
        char *pdst=dst;
        char *psrc=(char *)src;
        for(i--;i>0;i--)
...

写的很不错了，不过还是有一个错误，楼上当成memcpy了

飞灰橙

原帖由 flw 于 2006-6-15 18:13 发表
真是匪夷所思啊。
能说说这么做的理由吗？

类似的有不少，做不做，其实仁者见仁：
1. 定义指针是不忘赋空值
   char *p = NULL;
2. free以后指针置空
   free (p);
    p = NULL;
3. 即时一句话的if也加上花括号
   if ( a == b )
    {
        c = 3;
    }

飞灰橙

原帖由 er 于 2006-6-15 15:53 发表


别人也是动脑子了,虽然错了,没必要拿出来大家评论啊,当然如果当事人同意例外

兄弟说得对，当时我把答题贴出来，确实是出于自己的一点愤懑和嘲讽。
这是我做的不对。

converse

原帖由 flw 于 2006-6-15 18:13 发表

没想到你也有这么烂的习惯。
真是匪夷所思啊。
能说说这么做的理由吗？

这样防止赋值，当然了，当你想到这么写的时候也许就不会犯赋值的错误了，只是习惯罢了。

converse

原帖由 飞灰橙 于 2006-6-15 21:09 发表


类似的有不少，做不做，其实仁者见仁：
1. 定义指针是不忘赋空值
   char *p = NULL;
2. free以后指针置空
   free (p);
    p = NULL;
3. 即时一句话的if也加上花括号
   if ( a == b )
    {
     ...

我一般用一个宏来释放内存，

1. #define DELETE_PTR(p) \
   
2.               do{                  \
   
3.                        if (NULL != ptr)\
   
4.                        {                     \
   
5.                              free(p);    \
   
6.                              p = NULL;\
   
7.                        }                     \
   
8.               }while(0)                \

复制代码

mik

LZ 的函数呢？ 删了？

飞灰橙

原帖由 converse 于 2006-6-15 22:41 发表


我一般用一个宏来释放内存，

[code]#define DELETE_PTR(p) \
              do{                  \
                       if (NULL != ptr)\
                       {                     \
     ...

用p而不用(p)，写宏的一大禁忌!

lenovo

原帖由 converse 于 2006-6-15 22:39 发表


这样防止赋值，当然了，当你想到这么写的时候也许就不会犯赋值的错误了，只是习惯罢了。

以前也这样写，
可感觉看起来不爽，又回去了。

njmpop

void *memmove(void *dst,const void *src,size_t len)
{
        size_t i=len;
        char *pdst=dst;
        char *psrc=(char *)src;
        for(i--;i>0;i--)  /*说实话，这个算法俺不喜欢，看着难受*/
        {
                *(pdst+i)=*(psrc+i);/*前面减后面加，不是麻烦？*/

        }
        *pdst=*psrc;
        return pdst;
}

hellioncu

人家只是说做过，具体水平怎样也不一定呀。
我面试过的比这差的有很多，语法错误都一大堆。
很多人都不知道memcpy和memmove的差别。

[ 本帖最后由 hellioncu 于 2006-6-16 08:39 编辑 ]