等效代码如何变得更紧凑？->PAT乙级1031 查验身份证 (15分)

98 阅读 0 评论 65 点赞

我是靠谱客的博主自觉招牌，这篇文章主要介绍等效代码如何变得更紧凑？->PAT乙级1031 查验身份证 (15分)，现在分享给大家，希望可以做个参考。

原题链接https://pintia.cn/problem-sets/994805260223102976/problems/994805290334011392
1031 查验身份证 (15分)
一个合法的身份证号码由17位地区、日期编号和顺序编号加1位校验码组成。校验码的计算规则如下：

首先对前17位数字加权求和，权重分配为：{7，9，10，5，8，4，2，1，6，3，7，9，10，5，8，4，2}；然后将计算的和对11取模得到值Z；最后按照以下关系对应Z值与校验码M的值：

Z：0 1 2 3 4 5 6 7 8 9 10
M：1 0 X 9 8 7 6 5 4 3 2

现在给定一些身份证号码，请你验证校验码的有效性，并输出有问题的号码。

输入格式：
输入第一行给出正整数N（≤100）是输入的身份证号码的个数。随后N行，每行给出1个18位身份证号码。

输出格式：
按照输入的顺序每行输出1个有问题的身份证号码。这里并不检验前17位是否合理，只检查前17位是否全为数字且最后1位校验码计算准确。如果所有号码都正常，则输出All passed。

输入样例1：
4
320124198808240056
12010X198901011234
110108196711301866
37070419881216001X

输出样例1：
12010X198901011234
110108196711301866
37070419881216001X

输入样例2：
2
320124198808240056
110108196711301862

输出样例2：
All passed
这是一道水题，就是整型数组和字符数组的应用，第18位单独拿出来处理就可以了，关键讲我如何优化代码，使等效的代码更紧凑。
这是我一开始的垃圾代码，写了40大几行。。。

复制代码

#include <iostream>
using namespace std;
int main()
{
    iostream::sync_with_stdio(0);
    int N;//变量其实可以和下面一起定义
    cin>>N;
    int i,j,k,ans=0;
    int b[19]={0,7,9,10,5,8,4,2,1,6,3,7,9,10,5,8,4,2};
    char d[13]={'1','0','X', '9', '8','7', '6', '5', '4', '3', '2'};
    while(N--)
    {
        string a1;
        int check=0,flag1=1,flag2=0,c[13]={0};//c数组其实并不需要
        char M;
        for(i=1;i<=17;i++){
            char x;//其实可以直接输入a1[i]的
            cin>>x;
            a1[i]=x;
            if(x>'9'||x<'0')flag1=0;
            x=x-'0';//这里可以和下一行
            check+=x*b[i];
        }
        cin>>M;
        int Z=check%11;//可以和下一行合并
        c[Z]++;
        for(i=0;i<11;i++){//其实不需要这个循环
            if(c[i]==1){
                if(d[i]==M){
                    flag2=1;
                    break;
                }
            }
        }
        if(flag2==0||flag1==0){
            for(i=1;i<=17;i++){
                cout<<a1[i];
            }
            cout<<M<<endl;
            ans++;
        }
    }
    if(ans==0)
    cout << "All passed" << endl;//其实不用分两行写的
    return 0;
}

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
#include <iostream>
using namespace std;
int main()
{
    iostream::sync_with_stdio(0);
    int N;//变量其实可以和下面一起定义
    cin>>N;
    int i,j,k,ans=0;
    int b[19]={0,7,9,10,5,8,4,2,1,6,3,7,9,10,5,8,4,2};
    char d[13]={'1','0','X', '9', '8','7', '6', '5', '4', '3', '2'};
    while(N--)
    {
        string a1;
        int check=0,flag1=1,flag2=0,c[13]={0};//c数组其实并不需要
        char M;
        for(i=1;i<=17;i++){
            char x;//其实可以直接输入a1[i]的
            cin>>x;
            a1[i]=x;
            if(x>'9'||x<'0')flag1=0;
            x=x-'0';//这里可以和下一行
            check+=x*b[i];
        }
        cin>>M;
        int Z=check%11;//可以和下一行合并
        c[Z]++;
        for(i=0;i<11;i++){//其实不需要这个循环
            if(c[i]==1){
                if(d[i]==M){
                    flag2=1;
                    break;
                }
            }
        }
        if(flag2==0||flag1==0){
            for(i=1;i<=17;i++){
                cout<<a1[i];
            }
            cout<<M<<endl;
            ans++;
        }
    }
    if(ans==0)
    cout << "All passed" << endl;//其实不用分两行写的
    return 0;
}

优化过后变成下面30行以内代码：

复制代码

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
#include <iostream>
using namespace std;
const int b[19]={0,7,9,10,5,8,4,2,1,6,3,7,9,10,5,8,4,2};
const char d[13]={'1','0','X', '9', '8','7', '6', '5', '4', '3', '2'};//加入const可以提高代码质量
int main()
{
    iostream::sync_with_stdio(0);//加快io速度
    int N,i,j,k,ans=0;//变量一起定义
    cin>>N;
    while(N--){
        string a1;
        int check=0,flag1=1,flag2=0;
        for(i=1;i<=17;i++){
            cin>>a1[i];
            if(a1[i]>'9'||a1[i]<'0')flag1=0;
            check+=(a1[i]-'0')*b[i];
        }
        char M;
        cin>>M;
        if(d[check%11]==M)flag2=1;//不需要数组加循环
        if(flag2==0||flag1==0){
            for(i=1;i<=17;i++)cout<<a1[i];
            cout<<M<<endl;
            ans++;
        }
    }
    if(ans==0)cout << "All passed" << endl;
    return 0;
}