[话题]屎山代码优化篇

Koarz

#include <stdio.h>

int main() {
  int a, b, c, d, e, f, g, h..., z; // 这里共有26个变量
  scanf("%d%d%d%d%d%d...", &a, &b, &c, &d, &e, &f, ... & z);
  bool flag{};
  switch (a) {
  case 0:
    b += a;
    flag = true;
    break;
  case 1:
    b += a;
    flag = true;
    break;
  case 2:
    b += a;
    flag = true;
    break;
  case 3:
    b += a;
    flag = true;
    break;
  case 20:
    b += a;
    flag = true;
    break;
  case 456:
    b += a;
    flag = true;
    break;
  }
  if (!flag) {
    b += c * d * e * f * ... * z; // c一直*到z
  }
  printf("%d", b);
  return 0;
}

你会怎么优化这段代码

If_you_love

Koarz

#include <stdio.h>

int main() {
  int a,b;
  scanf("%d%d＂,&a,&b);
  if(a==0||a==1||a==2||a==3||a==20||a==456)
{
printf(＂%d＂,a+b);
return 0;
}
else{
int if_res=1;
while(scanf(＂%d＂,&a)!=EOF)
{
if_res*=a;
}
if_res+=b;
printf(＂%d＂,if_res);
}
  return 0;
}

WWW

#include<stdio.h>
int main()
{  int a[26],c=1,b[]={0,1,2,3,20,456}
for(int i=0;i<25;i++)
{
scanf("%d",&a{i]);
}
bool flag{};
for(int i=0;i<25;i++)
{
if(a[i]==b[]){b+=a;
flag=true;}
}
if(!flag)
{
for(int i=2;i<25;i++)
c*=a[2];
}
printf("%d",b+c):
return 0;
}

感觉数组b用错了，求教

Koarz

WWW 代码有问题

yyyyye

#include <stdio.h>
#define MAX_ST_CAPACITY 1000
#define MAX_ARG_COUNT 24
#define MAX_ARRAY_CAPACITY 30
#define SPECIAL_VALUE_COUNT 6
typedef long long ll;

int base, arr[MAX_ARRAY_CAPACITY], val[] = {0, 1, 2, 3, 20, 456};
ll res;
bool st[MAX_ST_CAPACITY];

void init()
{
    for (int i = 0; i < SPECIAL_VALUE_COUNT; i++)
    {
        st[val[i]] = true;
    }
}

ll cal_res()
{
    ll ret = 1;
    for (int i = 1; i <= MAX_ARG_COUNT; i++)
    {
        ret *= arr[i];
    }
    return ret;
}

int main()
{
    init();
    scanf("%d %d", &base, &res);
    for (int i = 1; i <= MAX_ARG_COUNT; i++)
    {
        scanf("%d", &arr[i]);
    }
    res += st[base] ? base : cal_res();
    printf("%lld", res);
    return 0;
}

重构关键：

逻辑模块化
main函数内部尽可能简洁清晰
清楚的变量命名

xxingyu

#include <bits/stdc++.h>
using namespace std;
using i64 = long long;
void solve() {
  i64 a = 0, b = 0, res = 1;
  bool flag = false;
  cin >> a >> b;
  for (int i = 1; i <= 24; ++i) {
    int x;
    cin >> x;
    res *= x;
  }
  if ((a >= 0 && a <= 3) || a == 20 || a == 456) {
    b += a;
    flag = true;
  }
  if (!flag) b += res;
  cout << b;
}
int main() {
  solve();
  return 0;
}

xujingyuan

强烈怀疑优化失败

#include<stdio.h>
int main()
{
	int n = 26;
	int a[100];
	int t;
        int ans = 1;
	for (int i = 0; i < 26; i++)
	{
		scanf("%d", &a[i]);
	}
	if ((a[0] >= 0 && a[0] <= 3) || a[0] == 20 || a[0] == 456)
	{
		ans = a[0] + a[1];
	}
	else
	{
		for (int i = 2; i < n; i++)
		{
			ans *= a[i];
		}
		ans += a[1];
	}
	printf("%d", ans);
	return 0;
}

ygxdsh

提供一个不考虑浪费空间的方法

#include <stdio.h>

int main()
{
    int a, b, c = 1, d;
    scanf("%d%d", &a, &b);
    for (int i = 0; i < 24; i++)
    {
        scanf("%d", &d);
        c *= d;
    }
    bool t[457]{};
    t[0] = t[1] = t[2] = t[3] = t[20] = t[456] = 1;
    if (a >= 0 && a <= 456 && t[a])//前面两个判断是为了防止数组越界
        b += a;
    else
        b += c;
    printf("%d", b);
    return 0;
}

本来是打算让if判断简单一点的，写完了才发现还要判断数组越界，也不咋简单，或许用set会好一点