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

本文主要是介绍等效代码如何变得更紧凑？-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;
}

优化过后变成下面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;
}