pat乙级1077 互评成绩计算，很简单的一道题，求帮忙看看，一个点都没过。

[没错就是阿罗]

https://pintia.cn/problem-sets/994805260223102976/problems/994805262303477760

/*1077 互评成绩计算（20 分）
在浙大的计算机专业课中，经常有互评分组报告这个环节。一个组上台介绍自己的工作，其他组在台下为其表现评分。
最后这个组的互评成绩是这样计算的：所有其他组的评分中，去掉一个最高分和一个最低分，剩下的分数取平均分记
为 G​1；老师给这个组的评分记为 G​2 。该组得分为 (G​1+G​2​​ )/2，最后结果四舍五入后保留整数分。本题就要求你写
个程序帮助老师计算每个组的互评成绩。

输入格式：
输入第一行给出两个正整数 N（> 3）和 M，分别是分组数和满分，均不超过 100。随后 N 行，每行给出该组得到的
 N 个分数（均保证为整型范围内的整数），其中第 1 个是老师给出的评分，后面 N−1 个是其他组给的评分。合法的
输入应该是 [0,M] 区间内的整数，若不在合法区间内，则该分数须被忽略。题目保证老师的评分都是合法的，并且每
个组至少会有 3 个来自同学的合法评分。

输出格式：
为每个组输出其最终得分。每个得分占一行。

输入样例：
6 50
42 49 49 35 38 41
36 51 50 28 -1 30
40 36 41 33 47 49
30 250 -25 27 45 31
48 0 0 50 50 1234
43 41 36 29 42 29
输出样例：
42
33
41
31
37
39*/
#include <iostream>
#include <algorithm>

using namespace std;

int main()
{
	int n,M,f[101],t=0;
	double a=0;
	cin>>n>>M;
	for(int i=0;i<n;i++)
	{
		t=0;a=0;
		for(int j=0;j<n;j++)
		{
			cin>>f[j];
			if(f[j]<0||f[j]>M)
			{f[j]=-1;t++;}
		}
		sort(f+1,f+n); 
		for(int j=2+t;j<n-1;j++)
		a=a+(double)f[j];
		a=a/(double)(n-3-t);
		a=(a+(double)f[0])/2.0;
		printf("%.0f",a); 
		if(i!=n-1)
		cout<<endl;
	}
	return 0;
}

好崩溃。我觉得这题还挺简单的啊

[lang_dye]

你的编程习惯好差，我用的C语言，第一也是有一个点出错，但循环用的是i<n*n,后来改成嵌套循环就对了。代码如下

#include <stdio.h>

int main()
{
	int n, m;
	scanf("%d%d", &n, &m);
	
	float g1, g2;
	int g[n];

	for(int i = 0; i < n; i++){
		int tot = 0, cnt = 0;
		int max = -1, min = m+1;
		for(int j = 0; j < n; j++){
			int grade;
			scanf("%d", &grade);
			
			if(j == 0)	g1 = grade;
			else
				if(grade >= 0 && grade <= m){
					if(grade > max)	max = grade;
					if(grade < min) min = grade;
					tot += grade;
					cnt++;
				}
		}
		tot = tot - min - max;
		cnt -= 2;
		g2 = 1.0f * tot/cnt;
		g[i] = (int)((g1+g2)/2+0.5);
	}

	for(int i = 0; i < n; i++)
		printf("%d\n", g[i]);
	
	return 0;
 }

[lang_dye]

引用 3 楼 Slzde_sub 的回复:

多说一句，第一位道友说的还是很中肯的，代码习惯很重要（虽然他也不咋地。。半斤八两），但是。。像你这样写c++了c系的强制转换就不要出现了

小到大的强制转换是没问题，最近也是在刷题，强迫自己用C语言写.....。小程序用复杂变量一是写的慢，还有其实别人看的也很吃力，不简洁，因为一般程序太小不会存在看不懂的情况，况且题目给力n,m,g，可以直接用来做变量名称。

[lang_dye]

说一下自己认为楼主代码习惯不好的地方吧！
1.不要从头写到尾，要留空行！！！空行在不同的内容，逻辑之间，这样代码层次更清楚。
2.int n,M,f[101],t=0;不要一个类型的全在一起，定义和声明是根据变量表达的内容来分的；
可以int n, M； int f[101], t;
3.这个不是一定，不要浮点数就用double，虽然现在内存大，但注意一点还是好的。
4.float的数值最好一般后面加个f，因为小数是double类型的，比如float = 2.5 * float; 2.5这里是double，计算前要转换。

[日立奔腾浪潮微软松下联想]

#include <stdio.h>

int main()
{
  int n, m, c1, c2, gn, g1, g2, gx, gi, x;
  scanf("%d%d", &n, &m);
  for (c1 = n, gn = g1 = gx = 0, gi = m; c1; c1--, gn = g1 = gx = 0, gi = m)
  {
    scanf("%d", &g2);
    for (c2 = n - 1; c2; c2--)
    {
      scanf("%d", &x);
      if (x >= 0 && x <= m) {g1 += x, ++gn; if (x > gx) gx = x; if (x < gi) gi = x;}
    }
    g1 -= gx + gi, ----gn;
    printf("%0.0f\n", (g2 + (int)(g1 + 0.5) / gn + 0.5) / 2);
  }

  return 0;
}

[没错就是阿罗]

引用 5 楼 lang_dye 的回复:

说一下自己认为楼主代码习惯不好的地方吧！
1.不要从头写到尾，要留空行！！！空行在不同的内容，逻辑之间，这样代码层次更清楚。
2.int n,M,f[101],t=0;不要一个类型的全在一起，定义和声明是根据变量表达的内容来分的；
可以int n, M； int f[101], t;
3.这个不是一定，不要浮点数就用double，虽然现在内存大，但注意一点还是好的。
4.float的数值最好一般后面加个f，因为小数是double类型的，比如float = 2.5 * float; 2.5这里是double，计算前要转换。

谢谢啦，都用double是因为有的题不用double过不了，所以都改成了double试试，还有第四条是后面加个f是什么意思我不太懂。

[没错就是阿罗]

引用 3 楼 Slzde_sub 的回复:

多说一句，第一位道友说的还是很中肯的，代码习惯很重要（虽然他也不咋地。。半斤八两），但是。。像你这样写c++了c系的强制转换就不要出现了

绿的心发慌

这样用会有什么问题吗，我学的不太好分不太清c和c++的一些用法。。。代码习惯这种东西应该怎么办啊

[没错就是阿罗]

真的很谢谢各位，这道题把printf("%.0f",a); 改成了 int x=a+0.5；cout<<x<<endl;就过了。
这是为啥啊，第一种也可以四舍五入啊。

[lang_dye]

小数后面加f是声明该小数为float类型，因为一般的小数是double类型，而double转换成float是大到小，虽然一般不会有问题，但这个习惯是不好的，小到大是没问题的，但大到小会出问题。例如float = 2.5 * float。 这个2.5会从double强制转换成float，这样不好。所以不是避免不避免的问题。

[Halo124444]

感恩楼主，我也是跟你一样的错误找了半天~233