Найти ошибку в программе

@TopLoader · Регистрация: 10.12.2011

Студворк — интернет-сервис помощи студентам

Решаю задачу:

В метании молота состязается n спортcменов. Каждый из них сделал m бросков.
Если перенумеровать спортсменов числами от 0 до n-1, а попытки каждого из них – от 0 до m-1, то на вход программа получает массив int A[n][m], состоящий из неотрицательных чисел.
Будем считать, что побеждает спортсмен, у которого максимален наилучший бросок. Если таких несколько, то из них побеждает тот, у которого наилучшая сумма результатов по всем попыткам. Если и таких несколько, победителем считается спортсмен с минимальным номером. Определите номер победителя соревнований.
Тест:
Вход
4 3
8 8 8
5 9 3
9 4 7
6 6 2

Выход:
2

Вот мой код:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
#include <iostream>
#include <windows.h>
using namespace std;
 
int main()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    setlocale(LC_ALL, "Russian");
 
    int n, m;
    int i, j;
    int max = 0;
    int sum;
    int maxSum = 0;
  int maxSumRow;
    int k;
    bool maxElementInRow;
 
    cout << "n>";
    cin >> n;
    cout << "m>";
    cin >> m;
 
    int a[n][m];
 
    // Чтение массива
    for(i=0; i<n; i++)
    for(j=0; j<m; j++)
      cin >> a[i][j];
  // Ищем
  for(i=0; i<n; i++) {
    maxElementInRow = false;
    sum = 0;  // сумма в строке = 0
    for(j=0; j<m; j++) {
      sum += a[i][j]; // считаем сумму
      if(a[i][j] > max) {
        max = a[i][j];
        k = i;
      } else if(a[i][j]==max)
          maxElementInRow = true;
    }
    if(maxElementInRow) {   // если в строке есть элемент, равный максимуму
        if(sum > maxSum) {
          maxSum = sum;
          k = i;
        }
      }
  }
 
  cout << "\n" << k << "\n";
 
    return 0;
}

Но он работает неправильно, не могу понять, что я не учитываю

@polyaKIDze · 14.11.2012, 05:31

после 44 строки вставьте:

C
1
printf ("i=%d sum=%d maxSum=%d\n", i, sum, maxSum);

и должны понять, в чем ошибка.

Кликните здесь для просмотра всего текста

maxSum должна сравниваться между двумя строками, имеющими одинаковый максимальный элемент

@Illusive · 14.11.2012, 09:28

В чем ошибка то?
Ответ и должен быть 2 -> k = 0,1,2,3

А вот задание массива в 26 строке работать не должно
n и m должны быть константами

@Kuzia domovenok · 14.11.2012, 10:35

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

cout << "n>";
cin >> n;
cout << "m>";
cin >> m;
int a[n][m];

Разве так можно выделять динамический массив в Си?

@TopLoader · 14.11.2012, 13:17 **[ТС]**

polyaKIDze, разобрался. В данном случае нужно было обнулять maxSum после того, как изменится max.

Работающий код, вдруг кому пригодиться

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
/*
Будем считать, что побеждает спортсмен, у которого максимален
наилучший бросок. Если таких несколько, то из них побеждает
тот, у которого наилучшая сумма результатов по всем попыткам.
Если и таких несколько, победителем считается спортсмен с
минимальным номером. Определите номер победителя соревнований.
*/
#include <iostream>
#include <windows.h>
using namespace std;
 
int main()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    setlocale(LC_ALL, "Russian");
 
    int n, m;
    int i, j;
    int max = 0;
    int sum;
    int maxSum = 0;
    int k;
    bool maxElementInRow;
 
    cout << "n>";
    cin >> n;
    cout << "m>";
    cin >> m;
 
    int a[n][m];
 
    // Чтение массива
    for(i=0; i<n; i++)
    for(j=0; j<m; j++)
      cin >> a[i][j];
  // Ищем
  for(i=0; i<n; i++) {
    maxElementInRow = false;
    sum = 0;  // сумма в строке = 0
    for(j=0; j<m; j++) {
      sum += a[i][j]; // считаем сумму
      if(a[i][j] > max) {
        max = a[i][j];
        k = i;
        maxSum = 0;
      }
      if(a[i][j]==max)
        maxElementInRow = true;
    }
    if(maxElementInRow) {   // если в строке есть элемент, равный максимуму
        //cout << "i=" << i << " sum=" << sum << " maxSum=" << maxSum << "\n";
        if(sum >maxSum) {
          maxSum = sum;
          k = i;
        }
      }
  }
 
  cout << "\n" << k << "\n";
 
    return 0;
}

@Kuzia domovenok · 14.11.2012, 15:34

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

Работающий код, вдруг кому пригодиться

Скажи, где проверялся этот "работающий" код? Не знаю как другие IDE, но MSVS такое точно не потерпит.

@TopLoader · 15.11.2012, 00:29 **[ТС]**

Сообщение от Kuzia domovenok

Скажи, где проверялся этот "работающий" код? Не знаю как другие IDE, но MSVS такое точно не потерпит.

CodeBlocks: полёт нормальный.

@Kuzia domovenok · 15.11.2012, 00:34

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

CodeBlocks: полёт нормальный.

Значит сам CodeBlocks как раз ненормальный и вытворяет нестандартную отсебятину. Покажи мне, чтобы Страуструпп или Кериганн&Риттчи такое советовали и тогда это стало бы нормальным.

@polyaKIDze · 15.11.2012, 01:05

TopLoader, Kuzia domovenok, я согласен по поводу странного выделения памяти. Лучше воспользоваться malloc или просто выделить a[10][10]. Но, может быть, в документации codeBlocks оговаривается этот момент, и есть разумное объяснение. И хотелось бы напомнить, что проблема была заявлена, как ошибка в логике, а не синтаксисе. Считаю спор неконструктивным и не относящимся к теме.

ValeryS · 15.11.2012, 01:24

Сообщение от Kuzia domovenok

Разве так можно выделять динамический массив в Си?

не поверишь можно( но не у всех)
вот ответ от Evg в моей теме про константы
кое-что про const

@Kuzia domovenok · 15.11.2012, 01:27

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

не поверишь можно( но не у всех)
вот ответ от Evg в моей теме про константы
кое-что про const

Не поверишь, но если бы m и n были бы константы, я бы слова не сказал.

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

cin >> m;
int a[n][m];

ValeryS · 15.11.2012, 01:37

Сообщение от Kuzia domovenok

Не поверишь, но если бы m и n бы ли б константы, я бы слова не сказал.

ты видать тему не полностью поднял там как раз и рассуждалось о разном поведении констант
вот мой текст(стартер)

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

вот так ведь правильно

C++
1
2
const int d=5;
int n[d];

т.е размер массива известен до исполнения
и так правильно

C++
1
2
3
int x;
std::cin >> x;
const int XXX = x;

переменная XXX будет известна при исполнении
а вот так уже неправильно ошибка компиляциии

C++
1
2
3
std::cin >> x;
const int XXX = x;
int n[XXX];

на что ответили

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

во дела! а почему такое вообще компилится? ведь для создания массива его размер должен быть известен на этапе компиляции, разве не так?

и тогда уже Evg ответил что ГНУс имеет собственные виды на стандарт

Новые блоги и статьи Все статьи Все блоги /
Настройки VS Code Loafer 13.04.2026 { "cmake. configureOnOpen": false, "diffEditor. ignoreTrimWhitespace": true, "editor. guides. bracketPairs": "active", "extensions. ignoreRecommendations": true, . . .	Оптимизация кода на разграничение прав доступа к элементам формы Maks 13.04.2026 Алгоритм из решения ниже реализован на нетиповом документе, разработанного в конфигурации КА2. Задачи, как таковой, поставлено не было, проделанное ниже исключительно моя инициатива. Было так:. . .	Контроль заполнения и очистка дат в зависимости от значения перечислений Maks 12.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: реализовать контроль корректности заполнения дат назначения. . .	Архитектура слоя интернета для сервера-слоя. Hrethgir 11.04.2026 В продолжение https:/ / www. cyberforum. ru/ blogs/ 223907/ 10860. html Знаешь что я подумал? Раз мы все источники пишем в голове ветки, то ничего не мешает добавить в голову такой источник, который сам. . .
Подстановка значения реквизита справочника в табличную часть документа Maks 10.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: при выборе сотрудника (справочник Сотрудники) в ТЧ документа. . .	Очистка реквизитов документа при копировании Maks 09.04.2026 Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .	модель ЗдравоСохранения 8. Подготовка к разному выполнению заданий anaschu 08.04.2026 https:/ / github. com/ shumilovas/ med2. git main ветка * содержимое блока дэлэй из старой модели теперь внутри зайца новой модели 8ATzM_2aurI	Блокировка документа от изменений, если он открыт у другого пользователя Maks 08.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в конфигурации КА2. Задача: запретить редактирование документа, если он открыт у другого пользователя. / / . . .

@Illusive 18 / 18 / 4 Регистрация: 11.01.2008 Сообщений: 51
	14.11.2012, 09:28
	В чем ошибка то? Ответ и должен быть 2 -> k = 0,1,2,3 А вот задание массива в 26 строке работать не должно n и m должны быть константами 0

@polyaKIDze 64 / 64 / 20 Регистрация: 16.07.2012 Сообщений: 147
	15.11.2012, 01:05
	TopLoader, Kuzia domovenok, я согласен по поводу странного выделения памяти. Лучше воспользоваться malloc или просто выделить a[10][10]. Но, может быть, в документации codeBlocks оговаривается этот момент, и есть разумное объяснение. И хотелось бы напомнить, что проблема была заявлена, как ошибка в логике, а не синтаксисе. Считаю спор неконструктивным и не относящимся к теме. 1