这个函数里面的if() 是不是写的太罗嗦了？

lylesong

下面这个代码是在wget的源码中看到的，
1、定义了两个指针p, cp 指向buffer[]数组，一个是static char 一个是 char，这样的好处是啥子？
2、感觉定义的static char *p = buffer;在这里定义有点多余，感觉用不上；
3、if语句中，为啥写成了p + size >= buffer + sizeof (buffer)，  直接用  size >= sizeof(buffer) 不是一样的么？
4、后面来了句 p += size;这个是搞啥子的啊？

发现完全不知道这个函数里面这么的意思了，请求解惑，谢谢。

1. /* Return a string that contains S with "no-" prepended.  The string
   
2.    is NUL-terminated and allocated off static storage at Wget
   
3.    startup.  */
   
4. 
   
5. static char *
   
6. no_prefix (const char *s)
   
7. {
   
8.   static char buffer[1024];
   
9.   static char *p = buffer;
   
10. 
    
11.   char *cp = p;
    
12.   int size = 3 + strlen (s) + 1;  /* "no-STRING\0" */
    
13.   if (p + size >= buffer + sizeof (buffer))
    
14.     abort ();
    
15. 
    
16.   cp[0] = 'n', cp[1] = 'o', cp[2] = '-';
    
17.   strcpy (cp + 3, s);
    
18.   p += size;
    
19.   return cp;
    
20. }
    

复制代码

sxcong

p比较的是指针，你改了之后是比较长度。
你考虑一下，no_prefix 这个函数多次调用会怎么样。

idi0t

p是静态的局部变量，这样写才可可重入，至于pm说这样的写法有问题，就不懂了

asdf2110

1.  static 变量只初始化一次，所以第2，3，。。。，n次进入函数时，p的指向就是 buffer + n 了
2.  参考1
3. p + size >= buffer + sizeof (buffer) 的含义是 buffer中剩余的空间是不是能存下字符串s，size >= sizeof(buffer)  这里sizeof(buffer) 是常量 1024， 所以需要根据上次的位置算一下
4. p+=size表示下次存储的起始位置

lin5161678

回复 6# pmerofc

越界是未定义行为 但是
要改的话 不能直接 size　>＝　sizeof(buffer)
size 这个数值应该是多次进入函数累计的

buffer + sizeof(buffer) - p > size
  

lin5161678

回复 8# pmerofc

我说得有点乱
代码第18行 p+=size;
还有就是5楼的回复

你都看看 应该能理解 size >　sizeof(buffer) 如果要写这句 那么这个size 应该是累加前面进入这个函数的每一个size