1

Я создал два динамических массива, двумерный и одномерный (arrA[n][m] и arrB[n]) Помогите найти ошибку в цикле. В некоторых строках находит максимальный элемент, во других же находит мусор.

int maxv = A - 1, nextv; // A - минимальный возможный элемент массива при генерации
for (int i = 0; i < n; i++)
{
    for (int j = 0; j < m; j++)
    {
        if (maxv < (nextv = arrA[i][j]))
        {
            maxv = nextv;
            arrB[i] = maxv;
        }
    }
}
1
  • можете скинуть полный код если не трудно Commented 6 дек. 2021 в 22:43

3 ответа 3

2

У вас после того, как в строке найден максимальный элемент, он и остается в maxv... Эту переменную надо инициализировать для каждой строки:

int maxv = A - 1, nextv; // A - минимальный возможный элемент массива при генерации
for (int i = 0; i < n; i++)
{
    maxv = A - 1;
    for (int j = 0; j < m; j++)
    {
        if (maxv < (nextv = arrA[i][j]))
        {
            maxv = nextv;
            arrB[i] = maxv;
        }
    }
}

Но без этой переменной можно и обойтись совсем :)

2

Вообще то это должен быть комментарий, но пока мне не дозволено (не хватает рейтинга), чтобы их оставлять.

Ваша проблема в том, что Вы не производите сброс maxv перед входом во внутренний цикл, если конечно я правильно понял задачу.

1

Вообще для таких вещей есть отладчик. Но вот исправленный вариант:

int maxv = A - 1, nextv; // На самом деле эти переменные не нужны
for (int i = 0; i < n; i++)
{
    arrB[i] = arrA[i][0];
    for (int j = 1; j < m; j++)
    {
        if (arrB[i] < arrA[i][j])
        {
            arrB[i] = arrA[i][j];
        }
    }
}

Необходимости использовать дополнительные переменные тут нет.

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.