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

er

原帖由 lenovo 于 2006-6-15 13:44 发表

为什么？

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

isjfk

确实无敌...

njmpop

while (src != "\0")

这句话也是错的。

njmpop

void *memmove (void *dest, const void *src, size_t n);
{
        assert ((dest != NULL) && (src != NULL));

        dest = (void *) malloc (n);  /*能返回到上一级？*/
        if (dest = NULL)                  /*能执行进去？*/
        {
                return NULL;
        }


        void *address = src;     /*返回源的值？奇怪*/
        while (src != "\0")        /*字符串和字符是不一样的；n怎么没用上？*/
        {
                *dest = *src;      /*void的值是？大概有默认值*/
                dest++;
                src++;
        }

        return address;
        free (dest);            /*其实不算错误*/
        dest = NULL;
}

ssssss24

...........

flw

原帖由 converse 于 2006-6-15 13:07 发表
还有if (dest = NULL)

我一般都是把常量放在左边的

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

[ 本帖最后由 flw 于 2006-6-15 18:17 编辑 ]

lvmart

写成这样也是需要一定的水平的

assiss

同意大家的意见。错成这样，需要很多水平。呵呵。

我习惯把常量放在右面。还是很多年前犯过一个=的低级错误，之后再也没犯过，所以有信心。

assiss

1. 
   
2. void *memmove (void *dest, const void *src, size_t n);
   
3. {
   
4.         assert ((dest != NULL) && (src != NULL));//如果要在函数内部分配内存，则dest为NULL也没关系。
   
5. 
   
6.         dest = (void *) malloc (n);//这里不算错误。要求不同，风格不同。
   
7.         if (dest = NULL)//看到这里笑了，==
   
8.         {
   
9.                 return NULL;
   
10.         }
    
11. 
    
12. 
    
13.         void *address = src;//通常需要把void*具体化，一般用char *。
    
14.         while (src != "\0")//看到这里我笑了，*src!='\0'，但既然是mem移动了，还比较'\0'干什么呢？应该是while(n--)
    
15.         {
    
16.                 *dest = *src;//风格问题，一般我喜欢把下面两行提前，*dest++ = *src++;
    
17.                 dest++;
    
18.                 src++;
    
19.         }
    
20. 
    
21.         return address;//从这里开始，佩服得五体投地……
    
22.         free (dest);
    
23.         dest = NULL;
    
24. }
    

复制代码