Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.50/8: Рейтинг темы: голосов - 8, средняя оценка - 4.50
 Аватар для TopLoader
43 / 43 / 6
Регистрация: 10.12.2011
Сообщений: 128

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

14.11.2012, 03:20. Показов 1718. Ответов 11
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Решаю задачу:
В метании молота состязается 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;
}
Но он работает неправильно, не могу понять, что я не учитываю
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
14.11.2012, 03:20
Ответы с готовыми решениями:

Найти ошибку в программе: в visual studio выдает ошибку
#include &lt;math.h&gt; #include &lt;conio.h&gt; #include &lt;iostream&gt; using namespace std; int main() { double x, E, ypred, y, S; ...

Найти ошибку в программе
Найти к-е число Фибоначчи,такое,что x(k-1)&lt;a&lt;x(k). Числа Фибоначчи:x(1)=x(2)=1;x(k)=x(k-2)+x(k-1) #include &quot;stdafx.h&quot; ...

Найти ошибку в программе
#include&lt;iostream&gt; #include&lt;stdio.h&gt; #include&lt;vector&gt; using namespace std; int main(){ int n; cin&gt;&gt;n; vector&lt;int&gt;...

11
64 / 64 / 20
Регистрация: 16.07.2012
Сообщений: 147
14.11.2012, 05:31
после 44 строки вставьте:
C
1
printf ("i=%d sum=%d maxSum=%d\n", i, sum, maxSum);
и должны понять, в чем ошибка.
Кликните здесь для просмотра всего текста
maxSum должна сравниваться между двумя строками, имеющими одинаковый максимальный элемент
1
18 / 18 / 4
Регистрация: 11.01.2008
Сообщений: 51
14.11.2012, 09:28
В чем ошибка то?
Ответ и должен быть 2 -> k = 0,1,2,3

А вот задание массива в 26 строке работать не должно
n и m должны быть константами
0
 Аватар для Kuzia domovenok
4268 / 3327 / 926
Регистрация: 25.03.2012
Сообщений: 12,531
Записей в блоге: 1
14.11.2012, 10:35
Цитата Сообщение от TopLoader Посмотреть сообщение
cout << "n>";
cin >> n;
cout << "m>";
cin >> m;
int a[n][m];
Разве так можно выделять динамический массив в Си?
0
 Аватар для TopLoader
43 / 43 / 6
Регистрация: 10.12.2011
Сообщений: 128
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;
}
0
 Аватар для Kuzia domovenok
4268 / 3327 / 926
Регистрация: 25.03.2012
Сообщений: 12,531
Записей в блоге: 1
14.11.2012, 15:34
Цитата Сообщение от TopLoader Посмотреть сообщение
Работающий код, вдруг кому пригодиться
Скажи, где проверялся этот "работающий" код? Не знаю как другие IDE, но MSVS такое точно не потерпит.
0
 Аватар для TopLoader
43 / 43 / 6
Регистрация: 10.12.2011
Сообщений: 128
15.11.2012, 00:29  [ТС]
Цитата Сообщение от Kuzia domovenok Посмотреть сообщение
Скажи, где проверялся этот "работающий" код? Не знаю как другие IDE, но MSVS такое точно не потерпит.
CodeBlocks: полёт нормальный.
0
 Аватар для Kuzia domovenok
4268 / 3327 / 926
Регистрация: 25.03.2012
Сообщений: 12,531
Записей в блоге: 1
15.11.2012, 00:34
Цитата Сообщение от TopLoader Посмотреть сообщение
CodeBlocks: полёт нормальный.
Значит сам CodeBlocks как раз ненормальный и вытворяет нестандартную отсебятину. Покажи мне, чтобы Страуструпп или Кериганн&Риттчи такое советовали и тогда это стало бы нормальным.
0
64 / 64 / 20
Регистрация: 16.07.2012
Сообщений: 147
15.11.2012, 01:05
TopLoader, Kuzia domovenok, я согласен по поводу странного выделения памяти. Лучше воспользоваться malloc или просто выделить a[10][10]. Но, может быть, в документации codeBlocks оговаривается этот момент, и есть разумное объяснение. И хотелось бы напомнить, что проблема была заявлена, как ошибка в логике, а не синтаксисе. Считаю спор неконструктивным и не относящимся к теме.
1
Модератор
Эксперт по электронике
8978 / 6744 / 921
Регистрация: 14.02.2011
Сообщений: 23,854
15.11.2012, 01:24
Цитата Сообщение от Kuzia domovenok Посмотреть сообщение
Разве так можно выделять динамический массив в Си?
не поверишь можно( но не у всех)
вот ответ от Evg в моей теме про константы
кое-что про const
0
 Аватар для Kuzia domovenok
4268 / 3327 / 926
Регистрация: 25.03.2012
Сообщений: 12,531
Записей в блоге: 1
15.11.2012, 01:27
Цитата Сообщение от ValeryS Посмотреть сообщение
не поверишь можно( но не у всех)
вот ответ от Evg в моей теме про константы
кое-что про const
Не поверишь, но если бы m и n были бы константы, я бы слова не сказал.
Цитата Сообщение от TopLoader Посмотреть сообщение
cin >> m;
int a[n][m];
0
Модератор
Эксперт по электронике
8978 / 6744 / 921
Регистрация: 14.02.2011
Сообщений: 23,854
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 ответил что ГНУс имеет собственные виды на стандарт
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
15.11.2012, 01:37
Помогаю со студенческими работами здесь

Найти ошибку в программе
Найти ошибку в программе(связано с неправильным использованием указателя). Как можно исправить ошибку? #include &lt;iostream&gt; ...

Найти ошибку в программе
Программа должна используя Дерево выводить список игроков сначала всех играющих в основном составе ниже играющих в резерве.... она же...

найти ошибку в программе
Сортировка вещественных чисел из файла методом пузырька #include &quot;stdafx.h&quot; #include &lt;stdio.h&gt; #include &lt;stdlib.h&gt; ...

Найти ошибку в программе
Помогите найти ошибку. #include &lt;iostream&gt; using namespace std; int main() { int a,kol=0,koll=0;

Найти ошибку в программе
#include &lt;stdafx.h&gt; #include &lt;iostream&gt; #include &lt;cstdlib&gt; #include &lt;fstream&gt; #include &lt;ctime&gt; using namespace std; int...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
12
Ответ Создать тему
Новые блоги и статьи
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США.
Programma_Boinc 26.12.2025
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Нашел на реддите интересную статью под названием «Кто-нибудь знает, где получить бесплатный компьютер или. . .
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Как объединить две одинаковые БД Access с разными данными
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru