初莫M 发表于 2023-10-27 21:18:46

错在哪了啊

题目描述
一年一度的江苏省小学生程序设计比赛开始了,组委会公布了所有学生的成绩,成绩按分数从高到底排名,成绩相同按年级从低到高排。现在主办单位想知道每一个排名的学生前,有几位学生的年级低于他。

输入描述
第 1 行只有一个正整数 n
(1≤n≤200
),表示参赛的学生人数。

第 2∼n+1
行,每行有两个正整数 s
(0≤s≤400
)和 g
(1≤g≤6
),之间用一个空格隔开,其中第 i+1
行的第一个数 s
表示第 i
个学生的成绩,第 i+1
行的第二个数 g
表示第 i
个学生的年级。

输出描述
输出 n
行,每行只有一个正整数,其中第 i
行的数 k
表示排第 i
名的学生前面有 k
个学生排名比他高,且年级比他低。
#include<bits/stdc++.h>
using namespace std;
int main()
{
        int n,a,i,j,t;
    cin>>n;
    for(i=0;i<n;i++)
    {
      cin>>a>>a;
    }
    for(i=0;i<n-1;i++)
    {
      for(j=0;j<n-i-1;j++)
      {
            if(a<a)
            {
                t=a;a=a;a=t;
            }
      }
    }
    for(i=0;i<n-1;i++)
    {
      for(j=0;j<n-i-1;j++)
      {
            if((a==a)&&(a>a))
                t=a;a=a;a=t;
      }
    }
    for(i=0;i<n;i++)
    {
      int s=0;
      for(j=0;j<i;j++)
      {
            if(a>a) s++;
      }
      cout<<s<<endl;
    }
}

isdkz 发表于 2023-10-27 21:19:23

这段代码的问题在于第二个排序的循环中,缺少了一对大括号,导致if语句只作用于t=a这一行,而不包括后面的两行交换操作。

正确的代码应该是这样的:

#include<bits/stdc++.h>
using namespace std;
int main()
{
    int n,a,i,j,t;
    cin>>n;
    for(i=0;i<n;i++)
    {
      cin>>a[ i]>>a[ i];
    }
    for(i=0;i<n-1;i++)
    {
      for(j=0;j<n-i-1;j++)
      {
            if(a<a)
            {
                t=a;a=a;a=t;
            }
      }
    }
    for(i=0;i<n-1;i++)
    {
      for(j=0;j<n-i-1;j++)
      {
            if((a==a)&&(a>a))
            {
                t=a;a=a;a=t;
            }
      }
    }
    for(i=0;i<n;i++)
    {
      int s=0;
      for(j=0;j<i;j++)
      {
            if(a[ i]>a) s++;
      }
      cout<<s<<endl;
    }
}


这样修改后,代码就能正确地按照题目要求输出每个学生前面有几个学生排名比他高且年级比他低了。

liuhongrun2022 发表于 2023-10-27 21:19:53

这个程序的问题在于它的排序算法。在这个程序中,两次冒泡排序分别用来按照分数和年级排序,但是这样做会导致第一个排序的结果被第二个排序破坏。

具体来说,第一次冒泡排序是按照分数从大到小排序的,但是在第二次冒泡排序中,如果两个学生的分数相同,年级较低的会被放在前面。这破坏了第一次排序的结果,因为可能会把分数高但年级也高的学生放到分数低但年级低的学生后面,这与题目要求的“成绩按分数从高到底排名,成绩相同按年级从低到高排”是矛盾的。

为了解决这个问题,你可以在一个排序过程中同时考虑分数和年级,即如果两个学生的分数不同,就按照分数排序;如果分数相同,就按照年级排序。这样可以保证排序的结果满足题目的要求。

修改后的代码如下:

#include<bits/stdc++.h>
using namespace std;

struct Student {
    int score;
    int grade;
};

bool compare(const Student &a, const Student &b) {
    if (a.score != b.score) {
      return a.score > b.score;
    }
    return a.grade < b.grade;
}

int main()
{
    int n;
    cin>>n;
    vector<Student> a(n);
    for(int i=0;i<n;i++)
    {
      cin>>a.score>>a.grade;
    }

    sort(a.begin(), a.end(), compare);

    for(int i=0;i<n;i++)
    {
      int s=0;
      for(int j=0;j<i;j++)
      {
            if(a.grade>a.grade) s++;
      }
      cout<<s<<endl;
    }
}


在这个修改后的代码中,我们定义了一个结构体 Student 用来存储每个学生的分数和年级,然后使用 std::sort 函数进行排序,排序规则是先按照分数从大到小排序,如果分数相同,再按照年级从小到大排序。这样就能保证排序的结果满足题目的要求。

Mike_python小 发表于 2023-10-27 21:19:54

这段代码存在以下问题:

1. 缺少头文件引用:需要包含 <iostream> 和 <algorithm> 头文件。

2. 数组越界访问:声明 a,但实际上数组的长度应为 a。

3. 排序算法错误:目前使用的是冒泡排序,但题目要求按分数从高到低排名,成绩相同时按年级从低到高排。冒泡排序只考虑成绩,没有对年级进行排序比较。

4. 年级判断错误:在计算每个学生前面有多少学生年级低于他时,使用了 a>a 的条件判断,实际上应该是 a<a。

修正后的代码如下所示:


#include <iostream>
#include <algorithm>

using namespace std;

int main() {
    int n;
    cin >> n;
   
    int a;
    for (int i = 0; i < n; i++) {
      cin >> a >> a;
    }

    // 按照题目要求排序
    sort(a, a + n, [](const int* x, const int* y) {
      if (x != y)
            return x > y;
      else
            return x < y;
    });

    for (int i = 0; i < n; i++) {
      int count = 0;
      for (int j = 0; j < i; j++) {
            if (a < a)
                count++;
      }
      cout << count << endl;
    }

    return 0;
}


注意,在修改代码后,还需要检查其他部分的排版和标点符号是否正确。

球一个最佳答案谢谢啦!这对我非常重要!{:10_254:}{:10_254:}{:10_254:}
页: [1]
查看完整版本: 错在哪了啊