今天看到一家公司的源代码，我疯了！

tpli

哪里要死啊。。

我怎么感觉可以啊。
看来我是垃圾程序员了

bruceteen

看完，我想死的心都有了。
------ 你应该说 “看完，我想掐死它的心都有了。”

坚持用 memset(query, sizeof(query),  0) 的人都是真正的高手，他们无论在数字还是字母中都能拍到第二位。

tajial

多余的初始化也就罢了，还初始化两次{:3_196:}

狗气球

难道是写php出身的……php出身也写不出来这个吧……

chinesedragon

char query[129] = {0};
memset(query, sizeof(query),  0);
sprintf(query, "%s", "select xxx,xxx,xxx form xxx");
sql_query(query);

只不过初始化两次，有死的必要吗？

cookis

你们全错了，
1. 他的memset用法有问题  应该是

1. 
   
2. void *memset(void *s, int c, size_t n);
   

复制代码

2. 如果是常量字符串 直接 static char query[] = "fuck fuck";  就OK

3. 尽量使用snprintf

4. 对于保存字符串的空间大小  尽量使用 127 + 1  而不是 128 + 1

chinesedragon

你们全错了，
1. 他的memset用法有问题  应该是2. 如果是常量字符串 直接 static char query[] = "fuck fu ...
cookis 发表于 2010-11-13 09:50

    确实是这样的，但两句合到一起却没有问题。估计是故意做的吧，呵呵

lsupper

神码.....

flw

不错了，最起码人家没按照 free(p) 之后再 p=NULL 的精神，
把 query 再 memset 一遍。

狗气球

其实我想说，
我猜本来是这个意思

sql_query("select xxx,xxx,xxx form xxx");
原来的代码里sql命令中连个变量都没出现