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

yuprxk

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

1. #include <stdio.h>
   
2. 
   
3. int any(char s[],const char d[],const int i)
   
4. 
   
5. {
   
6. 
   
7. int res;
   
8. 
   
9. int si ;
   
10. 
    
11. res = -1;
    
12. 
    
13. for( si = 0; s[si] != '\0' ; si++)
    
14. 
    
15. {
    
16. 
    
17.   if(s[si] == d[i])
    
18. 
    
19. 
    
20.    res = si;
    
21. 
    
22. }
    
23. 
    
24. return res;
    
25. 
    
26. }
    
27. 
    
28. 
    
29. 
    
30. int main()
    
31. 
    
32. {
    
33. 
    
34. int i;
    
35. 
    
36. char tests[] = "What I can see";
    
37. 
    
38. char testd[] = "tue";
    
39. 
    
40. i = any(tests,testd,0);
    
41. 
    
42.   
    
43. 
    
44. if(i == -1)
    
45. 
    
46.   printf("Not find\n");
    
47. 
    
48. else
    
49. 
    
50.   printf("Find\n");
    
51. 
    
52. return 0;
    
53. 
    
54. }
    
55. 
    
56. /*
    
57. 
    
58. *另一个版本的any，与上一个主要 的区别是return语句。
    
59. 
    
60. */
    
61. 
    
62. #include <stdio.h>
    
63. 
    
64. int any(char s[],const char d[],const int i)
    
65. 
    
66. {
    
67. 
    
68. int si ;
    
69. 
    
70. for( si = 0; s[si] != '\0' ; si++)
    
71. 
    
72. {
    
73. 
    
74.   if(s[si] == d[i])//此处的d已经修改为d[i]
    
75.    return si;
    
76. 
    
77. }
    
78. 
    
79. return -1;
    
80. 
    
81. }
    
82. 
    
83. 
    
84. 
    
85. int main()
    
86. 
    
87. {
    
88. 
    
89. int i;
    
90. 
    
91. char tests[] = "What I can see";
    
92. 
    
93. char testd[] = "tue";
    
94. 
    
95. i = any(tests,testd,0);
    
96. 
    
97. 
    
98. if(i == -1)
    
99. 
    
100. printf("Not find\n");
     
101. 
     
102. else
     
103. 
     
104. printf("Find\n");
     
105. 
     
106. return 0;
     
107. 
     
108. }
     

复制代码

[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