C中处理字符串函数，请帮忙看看哪里出了问题？感谢！【已解决】

Mr_Yao

1. #include <stdio.h>
   
2. #include <stdlib.h>
   
3. 
   
4. int main()
   
5. {
   
6.     char testline[56] = "sln:\\127.0.0.1:80sln";
   
7.     char *ip;
   
8.     char *port;
   
9.     char *str;
   
10.     int iport;
    
11.     ip = (char*)malloc(32);
    
12.     port = (char*)malloc(10);
    
13.     str = (char*)malloc(56);
    
14.     if(ip == NULL || port == NULL ||str == NULL)
    
15.     {
    
16.            printf("Error!malloc!\n");
    
17.            return -1;
    
18.     }
    
19.     strncpy(str,testline,56-1);
    
20.     while(*str<=48 || *str>= 57)
    
21.     {
    
22.         str ++;
    
23.         printf("str is [%s]\n",str);
    
24.     }
    
25.     while(*str == '.' || (*str<=57 && *str>=48))
    
26.     {
    
27.         *(ip++) = *(str++);
    
28.     }
    
29.     printf("ip is [%s]\n",ip);
    
30. 
    
31.     str++;
    
32.     while(*str >=48 && *str <= 57)
    
33.     {
    
34.            *port++ = *str++;
    
35.     }
    
36.     iport = atoi(port);
    
37.     printf("iport is [%d]\n",port);
    
38.     getch();
    
39.    
    
40.     return 0;
    
41. }
    
42. 
    

复制代码

大家帮忙看下，为什么ip和port输出的结果很奇怪呢？谢谢！

kamingli

申请完的空间， 最好先清空， 奇怪的结果就是由于没有遇到结束符所致。

hellioncu

写得太啰嗦，直接找到\和:就可以了，也用不着拷贝

rubylc_unix

回复 1# Mr_Yao


    ip指针变了，不是指的开头，发现没有？

wb112200

建议使用 strtook函数解析 这样太罗嗦了吧 另外新分配的堆内存需要做初始化 还有用完了要释放的
个人感觉在这里完全没有必要使用malloc 直接申请栈空间就满足了要求了...

rubylc_unix

回复 5# wb112200


    亦非，完全同意你的说法。

wb112200

回复 6# rubylc_unix


    {:3_196:} ... 今天压火箭比赛了 可惜不能看啊 杯具...

Mr_Yao

回楼上几位，我也觉得罗嗦的很，现在代码找到错误了，各位看看能不能优化一下？用比较精简的代码实现？

1. 
   
2. #include <stdio.h>
   
3. #include <stdlib.h>
   
4. 
   
5. int main()
   
6. {
   
7.     char *str = "sln:\\168.0.0.1:80sln";
   
8.         char *iptmp;
   
9.         char *ip;
   
10.         char *porttmp;
    
11.         char *port;
    
12.         int iport = 0;
    
13.         iptmp = (char*)malloc(32);
    
14.         memset(iptmp,0,32);
    
15.         porttmp = (char*)malloc(10);
    
16.         memset(iptmp,0,10);
    
17. 
    
18.         if(iptmp == NULL || porttmp == NULL)
    
19.         {
    
20.            printf("Error!malloc!\n");
    
21.            getch();
    
22.            return -1;
    
23.     }
    
24.         ip = iptmp;
    
25.         port = porttmp;
    
26. 
    
27.         while(*str<=48 || *str>= 57)
    
28.         {
    
29.                 str ++;
    
30.         }
    
31.         while(*str == '.' || (*str<=57 && *str>=48))
    
32.         {
    
33.         *iptmp++ = *str++;
    
34.     }
    
35.         printf("IP is [%s]\n",ip);
    
36.         while(*str<=48 || *str>= 57)
    
37.         {
    
38.                 str ++;
    
39.         }
    
40.         while(*str>=48 && *str<= 57)
    
41.         {
    
42.         *porttmp++ = *str++;
    
43.         }
    
44.         printf("Port is [%s]\n",port);
    
45.         iport = atoi(port);
    
46.         printf("iport is [%d]\n",iport);
    
47.         free(iptmp);
    
48.         free(porttmp);
    
49.         getch();
    
50.    
    
51.     return 0;
    
52. }
    
53. 
    

复制代码

Mr_Yao

回复 4# rubylc_unix


    恩，发现了。这个错误不应该犯。。。

rubylc_unix

回复 9# Mr_Yao


    然后修改下，基本上可以了，可以参考5楼说得。我调了下