一直找不出问题是什么，提示是运行错误（段错误，检查是否有数组越界，指针...谢谢...

NYJYA

题目描述
在基于Internet的程序中，我们常常需要判断一个IP字符串的合法性。
合法的IP是这样的形式：
A.B.C.D
其中A、B、C、D均为位于[0, 255]中的整数。为了简单起见，我们规定这四个整数中不允许有前导零存在，如001这种情况。
现在，请你来完成这个判断程序吧^_^
输入
输入由多行组成，每行是一个字符串，输入由“End of file”结束。
字符串长度最大为30，且不含空格和不可见字符
输出
对于每一个输入，单独输出一行
如果该字符串是合法的IP，输出Y，否则，输出N

我的代码：

1. #include<iostream>
   
2. using namespace std;
   
3. #include<cstdio>
   
4. #include<cstring>
   
5. #include<cstdlib>
   
6. int main()
   
7. {
   
8.         char a[32],b[31] = "End of file",c[30];
   
9.         unsigned int num = 0,i = 0,k = 0,j,n;
   
10.         //bool t = false;
    
11.         while(strcmp(gets(a),b)!= 0)
    
12.         {
    
13.                 i = 0;k = 0;
    
14.                 n = strlen(a);
    
15.                 if(n > 15)
    
16. {printf("N\n");continue ;}
    
17.                 while(i < n)
    
18.                 {
    
19.                         if(a[i] < '0' || a[i] > '9')
    
20.                         {
    
21.                                 cout << "N" << endl;
    
22.                                 //t = true;
    
23.                                 break;
    
24.                         }
    
25.                         c[k++] = a[i++];
    
26.                         if(a[i] == '.')
    
27.                         {
    
28.                                 num = atoi(c);
    
29.                                 if(num < 0 || num > 255)
    
30.                                 {
    
31.                                         cout << "N" << endl;
    
32.                                         //t = true;
    
33.                                         break;
    
34.                                 }
    
35.                                 for(j = 0;j < k;j++)
    
36.                                 {
    
37.                                         c[j] = '\0';
    
38.                                 }
    
39.                                 i++;
    
40.                                 k = 0;
    
41.                         }
    
42.                        
    
43.                 }
    
44.                 if(a[i] == '\0')
    
45.                 cout << "Y" << endl;
    
46.         }
    
47.        
    
48.        
    
49.         return 0;
    
50. }

复制代码

superbe

这个代码是判断 . 前的数字属于[0,255]，但是最后一段没有判断，另外应该判断 . 的数量==3。
增加一个变量count(在wile(i<n)前面)，记录 . 的数量：
int count=0;   
。。。
if(a[ i ] == '.'){
    。。。
    count++;
}

最后的判断改成：
c[ i ]='\0';
if(a[ i ] == '\0'  && count==3 && (atoi(c)>=0 && atoi(c)<=255))
    cout << "Y" << endl;
else
    cout<< "N" << endl;

我电脑上测试没有提示运行错误。

NYJYA

superbe 发表于 2019-10-6 20:34
这个代码是判断 . 前的数字属于[0,255]，但是最后一段没有判断，另外应该判断 . 的数量==3。
增加一个变量 ...

最后一段我用字符长度判断的

superbe

你测试下原来的代码，输入192.168.1.256结果会打印Y
输入192.168也是打印Y，是有问题的。