两个版本的函数any，哪一个比较好呢？

yuprxk

/*首先要说的是，这两个函数any，不管哪一个都有缺陷。我想问一下，哪一个较好呢?
*/

#include <stdio.h>

int any(char s[],const char d[],const int i)

{

int res;

int si ;

res = -1;

for( si = 0; s[si] != '\0' ; si++)

{

  if(s[si] == d[i])


   res = si;

}

return res;

}



int main()

{

int i;

char tests[] = "What I can see";

char testd[] = "tue";

i = any(tests,testd,0);

  

if(i == -1)

  printf("Not find\n");

else

  printf("Find\n");

return 0;

}

/*

*另一个版本的any，与上一个主要 的区别是return语句。

*/

#include <stdio.h>

int any(char s[],const char d[],const int i)

{

int si ;

for( si = 0; s[si] != '\0' ; si++)

{

  if(s[si] == d[i])//此处的d已经修改为d[i]
   return si;

}

return -1;

}



int main()

{

int i;

char tests[] = "What I can see";

char testd[] = "tue";

i = any(tests,testd,0);


if(i == -1)

printf("Not find\n");

else

printf("Find\n");

return 0;

}

[i]
我想请问一下，这两个版本的any哪一个较好 呢？[/i]

excessstone

原則上講第二個好，因为提前结束遍历。
但是这两个程序能运行吗？

yuprxk

第二个有较多的分支，可能不易控制。
第一个有点小问题，我本来想返回第一次出现的位置，这倒好，返回的是最后一次出现的位置了。

excessstone

我现在没有编译器，不过看起来似乎都不会有正确结果的样子。等我试试以后再说。

excessstone

if(s[si] == d) 这一行有错

        1        IntelliSense: operand types are incompatible ("char" and "const char *")        c:\users\windows\documents\visual studio 2010\projects\anyc\anyc\any.c        17        12        anyc