-1

Ошибка компиляции на cols во вложенном цикле:

void FillArr(int arr[], const int ROW, const int COL) {
    for (int cols = 0, num = 0; cols < COL; cols++) {
        for (int rows = 0; rows < ROW; rows++, num++) {
            arr[rows][cols] = num;
        }
    }
}
10
  • 3
    У вас передается в функцию одномерный массив, а обращаетесь вы с ним, как с двумерным...
    – Harry
    11 янв 2020 в 11:07
  • 1
    У вас передается в функцию опциональный указатель на 0 или более элементов типа int, наверное тут подразумевается, что он указывает на массив ROW x COL элементов, соответственно вам надо считать индекс. 11 янв 2020 в 11:09
  • @Harry я исправил код, но теперь ошибку на третью скобку выдаёт void FillAr ( int arr [] []) На третью квадратную 11 янв 2020 в 11:14
  • 2
    Нельзя так просто передать в функцию двухмерный массив не константного размера. Лучше используйте вместо него одномерный размера ROW * COL. 11 янв 2020 в 11:18
  • Потому что при передаче двумерного массива именно в таком виде первая размерность должна быть указана. Как я думаю (нужную информацию вы в вопросе не привели), вам нужен первый вариант, но обращение не arr[rows][cols], а arr[rows*COLS+cols].
    – Harry
    11 янв 2020 в 11:19

1 ответ 1

3

(ответа нет, комментарии есть, так что попробую написать ответ) Вы пытаетесь обратиться к одномерному массиву как к двумерному:

// здесь int arr[] тоже самое, что и int *arr

void FillArr(int arr[], const int ROW, const int COL) {
    ...
    arr[rows][cols] = ...;
    ...
}

Из-за чего компилятор не понимает ваших намерений и выдает ошибку компиляции.

Исправить проблему можно двумя способами, в зависимости от того, что у вас представляет из себя двумерный массив.

  1. Изменить сигнатуру метода, то есть сделать в функции из arr двумерный массив. Использовать, если входной массив был создан как int *arr[y] (y -> количество строк) или int **arr.
//           int *arr[],

void FillArr(int **arr, const int ROW, const int COL) {
    for (int cols = 0, num = 0; cols < COL; cols++) {
        for (int rows = 0; rows < ROW; rows++, num++) {
            arr[rows][cols] = num;
        }
    }
}
  1. Использовать математику, чтобы превратить обращение к одномерному массиву так, будто бы это двумерный массив. Использовать, если входной массив был создан как int arr[x], int *arr; еще можно ожидать int arr[y][x], однако придется будет преобразовывать его в int* -> reinterpret_cast<int*>(arr) при вызове.
//           int arr[],

void FillArr(int *arr, const int ROW, const int COL) {
    for (int cols = 0, num = 0; cols < COL; cols++) {
        for (int rows = 0; rows < ROW; rows++, num++) {
            // rows * COL позволит нам перепрыгнуть определённое количество
            // элементов (равняещееся количеству колонок в двумерном
            // массиве умноженное на количество пройденный строчек),
            // а суммирование просто сделает отступ от этого прыжка.
            // Прыжком мы дойдем до нужной строки, а отступ
            // позволит получить элемент в этой строке.

            arr[rows * COL + cols] = num;
        }
    }
}

Проверял свои суждения на gcc 8.3.0; gcc -xc++ -lstdc++ -shared-libgcc -std=C++14.

Ваш ответ

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

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