С Новым годом! Форум программистов, компьютерный форум, киберфорум
Наши страницы

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
TopLoader
42 / 42 / 4
Регистрация: 10.12.2011
Сообщений: 128
#1

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

14.11.2012, 03:20. Просмотров 779. Ответов 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
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
14.11.2012, 03:20
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Найти ошибку в программе (C++):

Найти ошибку в программе: в visual studio выдает ошибку - C++
#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; ...

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

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

Найти ошибку в программе - C++
Где ошибка в программе: #include &quot;stdafx.h&quot; #include &lt;iostream&gt; using namespace std; #include &quot;conio.h&quot; #include &quot;string&quot; ...

Найти ошибку в программе - C++
Найти к-е число Фибоначчи,такое,что 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...

Найти ошибку в программе - C++
#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...

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

А вот задание массива в 26 строке работать не должно
n и m должны быть константами
0
Kuzia domovenok
2060 / 1905 / 174
Регистрация: 25.03.2012
Сообщений: 6,567
Записей в блоге: 1
14.11.2012, 10:35 #4
Цитата Сообщение от TopLoader Посмотреть сообщение
cout << "n>";
cin >> n;
cout << "m>";
cin >> m;
int a[n][m];
Разве так можно выделять динамический массив в Си?
0
TopLoader
42 / 42 / 4
Регистрация: 10.12.2011
Сообщений: 128
14.11.2012, 13:17  [ТС] #5
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
2060 / 1905 / 174
Регистрация: 25.03.2012
Сообщений: 6,567
Записей в блоге: 1
14.11.2012, 15:34 #6
Цитата Сообщение от TopLoader Посмотреть сообщение
Работающий код, вдруг кому пригодиться
Скажи, где проверялся этот "работающий" код? Не знаю как другие IDE, но MSVS такое точно не потерпит.
0
TopLoader
42 / 42 / 4
Регистрация: 10.12.2011
Сообщений: 128
15.11.2012, 00:29  [ТС] #7
Цитата Сообщение от Kuzia domovenok Посмотреть сообщение
Скажи, где проверялся этот "работающий" код? Не знаю как другие IDE, но MSVS такое точно не потерпит.
CodeBlocks: полёт нормальный.
0
Kuzia domovenok
2060 / 1905 / 174
Регистрация: 25.03.2012
Сообщений: 6,567
Записей в блоге: 1
15.11.2012, 00:34 #8
Цитата Сообщение от TopLoader Посмотреть сообщение
CodeBlocks: полёт нормальный.
Значит сам CodeBlocks как раз ненормальный и вытворяет нестандартную отсебятину. Покажи мне, чтобы Страуструпп или Кериганн&Риттчи такое советовали и тогда это стало бы нормальным.
0
polyaKIDze
63 / 63 / 12
Регистрация: 16.07.2012
Сообщений: 147
15.11.2012, 01:05 #9
TopLoader, Kuzia domovenok, я согласен по поводу странного выделения памяти. Лучше воспользоваться malloc или просто выделить a[10][10]. Но, может быть, в документации codeBlocks оговаривается этот момент, и есть разумное объяснение. И хотелось бы напомнить, что проблема была заявлена, как ошибка в логике, а не синтаксисе. Считаю спор неконструктивным и не относящимся к теме.
1
ValeryS
Модератор
6729 / 5138 / 485
Регистрация: 14.02.2011
Сообщений: 17,246
15.11.2012, 01:24 #10
Цитата Сообщение от Kuzia domovenok Посмотреть сообщение
Разве так можно выделять динамический массив в Си?
не поверишь можно( но не у всех)
вот ответ от Evg в моей теме про константы
кое-что про const
0
Kuzia domovenok
2060 / 1905 / 174
Регистрация: 25.03.2012
Сообщений: 6,567
Записей в блоге: 1
15.11.2012, 01:27 #11
Цитата Сообщение от ValeryS Посмотреть сообщение
не поверишь можно( но не у всех)
вот ответ от Evg в моей теме про константы
кое-что про const
Не поверишь, но если бы m и n были бы константы, я бы слова не сказал.
Цитата Сообщение от TopLoader Посмотреть сообщение
cin >> m;
int a[n][m];
0
ValeryS
Модератор
6729 / 5138 / 485
Регистрация: 14.02.2011
Сообщений: 17,246
15.11.2012, 01:37 #12
Цитата Сообщение от 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
15.11.2012, 01:37
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
15.11.2012, 01:37
Привет! Вот еще темы с ответами:

Найти ошибку в программе - C++
#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;...

найти ошибку в программе - C++
#include &quot;stdafx.h&quot; #include &lt;conio.h&gt; #include &lt;iostream&gt; #include &lt;stdio.h&gt; using namespace std; void _tmain() { char...

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

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


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

Или воспользуйтесь поиском по форуму:
12
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.