写的一个查找字符串的小程序，请老大们拍砖

jemy.zhang

学习C程序中，不知道有什么地方写的不好，不符合规范，请老大们指教

1. 
   
2. 
   
3. int instr(char *string, char *t_str, int start_pos)
   
4. {
   
5.     int scnt,dcnt,ret = -1;
   
6.    
   
7.     if( (string == NULL) || (t_str == NULL) ) return -1;
   
8. 
   
9.     for(scnt = 0, dcnt = 0;
   
10.         (*(string + scnt) != '\0' && *(t_str + dcnt) != '\0') || (ret < start_pos);
    
11.         scnt ++)
    
12.     {
    
13.         if(dcnt == 0) ret = scnt;
    
14.         if(*(t_str + dcnt) == *(string + scnt))
    
15.         {
    
16.             dcnt ++;
    
17.         }else
    
18.         {
    
19.             dcnt = 0;
    
20.         }
    
21.     }
    
22.     if( (ret == 0 && dcnt == 0) || (*(t_str + dcnt) != '\0') ) return -1;
    
23.     return ret;
    
24. }
    

复制代码

思一克

是在string 的第start_pos处开始查找t_str吗？

快乐的土豆

原帖由 思一克 于 2007-2-28 15:43 发表
是在string 的第start_pos处开始查找t_str吗？

第一个问题：没有注释或者方法签名不直观。

飞灰橙

俺喜欢函数名是动词开头，
而不是形容词或副词或名词

boxpei

乱说几句，别拍转。

1 函数，变量名命名不直观
2 注释没有
3 作为输入参数的字符串推荐用const char*
4 scnt,dcnt这两个变量用char*就没有那么多的t_str + dcnt，string + scnt吧
5 for循环里面的逻辑和函数返回值得处理有更简明的方式。

it_0508

怎么不用函数strstr来做，即简单又明了。

jemy.zhang

感谢感谢~~
没有写注释是我写程序的一个很大的缺点，以后一定注意！

之所以用instr作为名称，是因为以前用vb很多，vb里面有这样一个函数，所以就照样写了一个

不用strstr是因为做嵌入式开发，不能过分依赖库，呵呵

const char*，呵呵，非常有道理，受教~ 不用char*定义scnt是因为我要获取位置，否则还要倒过来计算偏移

思一克

LZ,
做嵌入式开发更不能用你编的这样的函数。反而要用strstr, strcpy等。因为你这样的函数，可靠性，错误，尺寸等和库函数比都不行。

jemy.zhang

原帖由 思一克 于 2007-2-28 17:07 发表
LZ,
做嵌入式开发更不能用你编的这样的函数。反而要用strstr, strcpy等。因为你这样的函数，可靠性，错误，尺寸等和库函数比都不行。

但是未必会有这样的库函数存在啊，如果不是标准的C函数库

思一克

LZ,

用strstr等来写。而不要自己从头写更可靠更节省。

还有，连接库函数未必连接整个库。
还有一个方法，将strstr的原程序直接你自己用。