面试题求教

starwing83

回复 9# zehn_w


    1.    while (head->next){
2.        if (strcmp(head->word, word))
3.    strcpy(node->word, word, 20);
4.    node = head->next;
      node->next = NULL;

为嘛有错自己想去……

zehn_w

回复 11# starwing83
谢谢~~ 错误太白痴了，哈哈哈。


   

kelvin_fly

回复 1# zehn_w
兄弟，看来你和我水准差不多哇。 哈哈哈，是刚毕业在找工作吗？

   

ymonster

我看到的是改完的代码

其实吧，说隐患主要也就是个strncpy这类东西，实际工作的时候如果你能确认这个问题基本就ok，别的无伤大雅，只要你自己能确认就行

其它的
1、char xxx[21]; //如果你自己能确认字符串长度最大值，不需要用char *来做动态分配，并且没有什么特别的空间限制，尽量不要用21这种字节不对其的分配数值
2、malloc出来的东西，你无法保证分配内存里面全都是0，也就是说不是所有编译器都会帮你干这个事情，这样你strncpy过去20个字符，那么xxx[20]不是0，就有在其后进行字符串操作发生访问越界的可能。我觉得这才是更重要的隐患: strncpy这东西，如果你能确认没问题，不如用strcpy，不能确认，别忘了结尾的'\0'

folklore

回复 14# ymonster

说说我的观点吧：

    1、char xxx[21]; //如果你自己能确认字符串长度最大值，不需要用char *来做动态分配，并且没有什么特别的空间限制，尽量不要用21这种字节不对其的分配数值

和我一样的恶习。Malloc还要做薄记功能，所以，会分配比指定大小多一点的内存的（当然，不是所以实现都这样），这个没办法，如果能够在运行时知道所要的内在大小，
malloc一个正好的大小是比较好的，

2、malloc出来的东西，你无法保证分配内存里面全都是0，也就是说不是所有编译器都会帮你干这个事情，这样你strncpy过去20个字符，那么xxx[20]不是0，就有在其后进行字符串操作发生访问越界的可能。我觉得这才是更重要的隐患: strncpy这东西，如果你能确认没问题，不如用strcpy，不能确认，别忘了结尾的'\0'

1. 
   
2. strncpy(ss,s,strlen(s)+1);
   

复制代码

吧

ymonster

folklore 发表于 2012-08-15 12:12
回复 14# ymonster

说说我的观点吧：

使用strlen就要保证strlen后+1的数值不会超出开的buffer的问题，不过一般用strncpy的时候都是有特别的需求或者心里有数的

starwing83

回复 15# folklore


    strncpy(ss,s,strlen(s)+1);

这句就是strcpy……………………