Найти максимальную оценку студента и вывести его ID потом фамилию и максимальную оценку

@AlisherIITU · Регистрация: 10.09.2013

Здравствуйте!
Мне нужно найти максимальную оценку студента и вывести его ID потом фамилию и макс оценку
Вот образец

INPUT:
3
1 Ivano 51
2 Zuckerberg 99.1
3 Mendelson 99.0

OUTPUT:
2 Zuckerberg 99.1

Вот мой код происходит сбой при вводе Помогите

C++

#include <iostream>
using namespace std;
struct Student
{
    int ID;
    char *surname;
    double grade;
};
 
int main ()
{
    Student a[100];
    double max=0;
    int n;
    cin>>n;
    for(int i=0;i<n;i++)
    {
        cin>>a[i].ID;
        cin>>a[i].surname;
        cin>>a[i].grade;
        if(a[i].grade>max)
        {
            max=a[i].grade;
        }
        cout<<a[i].ID<<" "<<a[i].surname<<" "<<max;
    }
    return 0;
 
}

@taras atavin · 17.11.2013, 12:52

Сообщение от AlisherIITU

C++

[for(int i=0;i<n;i++)
  {
  cin>>a[i].ID;
  cin>>a[i].surname;
  cin>>a[i].grade;
  if(a[i].grade>max)
  {
      max=a[i].grade;
  }
  cout<<a[i].ID<<" "<<a[i].surname<<" "<<max;

А какого ответ запихнут в цикл ввода?

@AlisherIITU · 17.11.2013, 12:54 **[ТС]**

Сообщение от taras atavin

А какого ответ запихнут в цикл ввода?

ну он в цикле должен быть же, тогда a[i] не будет работать же

@taras atavin · 17.11.2013, 13:01

Сообщение от AlisherIITU

ну он в цикле должен быть же, тогда a[i] не будет работать же

А зачем там a[i]? По смыслу это не текущий, а искомый, значит [m], а не [i]. Кроме того, даже если надо вывести в цикле, ввод к этому моменту должен быть завершён, если только это не вывод подсказки ко вводу, поэтому остальной вывод в цикле делается в отдельном цикле. Но здесь надо вывести один раз.

@AlisherIITU · 17.11.2013, 13:07 **[ТС]**

Сообщение от taras atavin

А зачем там a[i]? По смыслу это не текущий, а искомый, значит [m], а не [i]. Кроме того, даже если надо вывести в цикле, ввод к этому моменту должен быть завершён, если только это не вывод подсказки ко вводу, поэтому остальной вывод в цикле делается в отдельном цикле. Но здесь надо вывести один раз.

К сожалению я вас не понел, и что значит [m], а не [i]?

@taras atavin · 17.11.2013, 13:12

То и значит.

C++

a[m].ID

.

Добавлено через 1 минуту

C++

Student a[100];
    double max=0;
    int m;
    int n;
    cin>>n;
    for(int i=0;i<n;i++)
    {
        cin>>a[i].ID;
        cin>>a[i].surname;
        cin>>a[i].grade;
        if(a[i].grade>max)
        {
            max=a[i].grade;
            m=i;
        }
      }
    cout<<a[m].ID<<" "<<a[m].surname<<" "<<max;
    return 0;

@AlisherIITU · 17.11.2013, 13:18 **[ТС]**

Спасибо теперь все понятно!

@taras atavin · 17.11.2013, 13:49

А можно даже не хранить сам максимум:

C++

Student a[100];
    int m;
    int n;
    cin>>n;
    for(int i=0, m=0; i<n; ++i)
    {
        cin>>a[i].ID;
        cin>>a[i].surname;
        cin>>a[i].grade;
        if(a[i].grade>a[m].grade)
        {
            m=i;
        }
      }
    cout<<a[m].ID<<" "<<a[m].surname<<" "<<a[m].grade;
    return 0;

Добавлено через 1 минуту
Да, кстати, здесь дыра переполнения массива, память надо или выделять динамически, или работать только с буфером, а не массивом, или n должно быть константой.

Добавлено через 51 секунду

Сообщение от AlisherIITU

struct Student
{
* * int ID;
* * char *surname;
* * double grade;
};

И это вводится?

C++

1	std::string surname;

.

@AlisherIITU · 17.11.2013, 14:02 **[ТС]**

Да я с этим разобрался))

@AlisherIITU 2 / 2 / 3 Регистрация: 10.09.2013 Сообщений: 126
	17.11.2013, 12:54 [ТС]	3
	Сообщение от taras atavin А какого ответ запихнут в цикл ввода? ну он в цикле должен быть же, тогда a[i] не будет работать же 0

@taras atavin 4197 / 1790 / 211 Регистрация: 24.11.2009 Сообщений: 27,563
	17.11.2013, 13:01	4
	Сообщение от AlisherIITU ну он в цикле должен быть же, тогда a[i] не будет работать же А зачем там a[i]? По смыслу это не текущий, а искомый, значит [m], а не [i]. Кроме того, даже если надо вывести в цикле, ввод к этому моменту должен быть завершён, если только это не вывод подсказки ко вводу, поэтому остальной вывод в цикле делается в отдельном цикле. Но здесь надо вывести один раз. 0

@AlisherIITU 2 / 2 / 3 Регистрация: 10.09.2013 Сообщений: 126
	17.11.2013, 13:07 [ТС]	5
	Сообщение от taras atavin А зачем там a[i]? По смыслу это не текущий, а искомый, значит [m], а не [i]. Кроме того, даже если надо вывести в цикле, ввод к этому моменту должен быть завершён, если только это не вывод подсказки ко вводу, поэтому остальной вывод в цикле делается в отдельном цикле. Но здесь надо вывести один раз. К сожалению я вас не понел, и что значит [m], а не [i]? 0

@AlisherIITU 2 / 2 / 3 Регистрация: 10.09.2013 Сообщений: 126
	17.11.2013, 13:18 [ТС]	7
	Спасибо теперь все понятно! 0

@AlisherIITU 2 / 2 / 3 Регистрация: 10.09.2013 Сообщений: 126
	17.11.2013, 14:02 [ТС]	9
	Да я с этим разобрался)) 0

Опции темы