Home / Questions / 33290
Accepted
Stevn
Asked: 2020-11-12 13:21:41 +0800 CST 2020-11-12 13:21:41 +0800 CST

Массив как параметр C++

  • 772

Уважаемый, я пытаюсь отправить динамический массив в качестве параметра функции на C++, но он сообщает мне как ошибку, что переменная N находится вне контекста или не определена, как я могу решить эту ошибку? Следите за обновлениями. Привет.

#include <iostream>

#include <stdlib.h>

#include <time.h>

using namespace std;

void generarArreglo(int matriz[n][n]){
  srand(time(NULL));
  for(int i=0;i<=6;i++){
    for(int j=0;j<=6;j++){
      matriz[i][j]=rand()%99+1;
    }
  }
}

void imprimirArreglo(int matriz[n][n]){
  for(int i=0;i<=6;i++){
    for(int j=0;j<=6;j++){
      cout<<matriz[i][j]<<"\t";
    }
    cout<<endl;
  }
}

int main(){

  int n;    
  cin >> n;
  int matriz[n][n];
  generarArreglo(matriz);
  imprimirArreglo(matriz);
  system("pause");
  return 0;
}
c++

4 Answers

  1. Мое решение:

    #include <iostream>
#include <stdlib.h>
#include <time.h>

using namespace std;

void generarArreglo(int **matriz, int rows, int cols){
    srand(time(NULL));
    for(int i=0;i<rows;i++){
        for(int j=0;j<cols;j++){
            matriz[i][j]=rand()%99+1;
        }
    }
}

void imprimirArreglo(int **matriz, int rows, int cols){
    for(int i=0;i< rows;i++){
        for(int j=0;j< cols;j++){
            cout<<matriz[i][j]<<"\t";
        }
        cout<<endl;
    }
}

int main(){
    int n;
    cin >> n;
    int rows , cols;
    rows = cols = n;
    int** matriz = new int*[rows];
    for (int i = 0; i < rows; ++i)
        matriz[i] = new int[cols];
    generarArreglo(matriz, rows, cols);
    imprimirArreglo(matriz, rows, cols);
    return 0;
}

    Вход:

    4

    Вылет из:

    98  79  6   19  
11  2   96  95  
63  33  11  93  
80  36  70  50
    • 2
  2. Best Answer

    Ошибка возникает здесь:

    void generarArreglo(int matriz[n][n]){ // <<--- AQUI!!!
  // ...
}

void imprimirArreglo(int matriz[n][n]){ // <<--- AQUI!!!

    И причина в том, что nон не существует для этих функций. Если бы это было постоянное значение, известное во время компиляции, вы могли бы исправить код так, чтобы:

    const int n = 5;

void generarArreglo(int matriz[n][n]){ // OK
  // ...
}

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

    void generarArreglo(std::vector<int>& matriz){
  srand(time(NULL));
  for( int i=0; i<matriz.size(); i++)
    matriz[i]=rand()%99+1;
}

void imprimirArreglo(std::vector<int>& matriz, int n){
  for(int i=0;i<n;i++){
    for(int j=0;j<n;j++){
      cout<<matriz[i*n+j]<<'\t';
    }
    cout<<'\n';
  }
}

int main(){
  int n;
  cin >> n;
  std::vector<int> matriz(n*n,0);
  generarArreglo(matriz);
  imprimirArreglo(matriz, n);
  system("pause");
  return 0;
}

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

    • Требуется только одно резервирование и одно освобождение
    • Вся память объединена в пул, что способствует использованию системного кеша.

    Пример с простым указателем:

    void generarArreglo(int* matriz, int length){
  srand(time(NULL));

  while( length-- )
  {
    *matriz=rand()%99+1;
    ++matriz;
  }
}

void imprimirArreglo(int* matriz, int n){
  for(int i=0;i<n;i++){
    for(int j=0;j<n;++j,++matriz){
      cout<<*matriz<<'\t';
    }
    cout<<'\n';
  }
}

int main(){

  int n;
  cin >> n;
  int* matriz = new int[n*n];
  generarArreglo(matriz, n*n);
  imprimirArreglo(matriz, n);
  delete[] matriz;

  system("pause");
  return 0;
}
    • 2

  3. Ваш код

    int matriz[n][n];

    недействителен в C++. Размеры массива должны быть константами, а не переменными. Вам придется объявить его как указатель и выделить память на лету:

    int *matriz;
matriz = calloc( n, n );

    И вам, конечно, придется модифицировать все ваши функции.

    РЕДАКТИРОВАТЬ

    Для доступа через указатели это то же самое (я уверен, что в C, бросьте меня к волкам, если то же самое не произойдет в C++):

    int *matriz = calloc( n, n );

matriz[0][0] = *matriz;
matriz[0][1] = matriz[2];
matriz[1][0] = matriz[n] = matriz[n][0]

    Если бы вместо уродливого и грязного C++ вы использовали простой C , вы могли бы использовать переменные для объявления массивов (в C11).

    В C а arrayи а punteroэквивалентны. Если вы объявляете a array[], компилятор выделяет память для указанного размера, и на самом деле вы используете указатель на эту память.

    РЕДАКТИРОВАТЬ 2

    Почему вы используете 2 forвложенных ? с чем вы используете

    int *idx;
for( idx = matriz; idx < n * n; ++ idx ) {
 . . .
}

    Блоки arrayпамяти и , возвращаемые функцией calloc(), malloc()и т.п. являются непрерывными в памяти . Просто представьте, что он array[]одномерный, но гораздо более длинный.

    РЕДАКТИРОВАТЬ 3

    По вашим комментариям я вижу, что вы хотите учиться.

    #include <iostream>
#include <stdlib.h>
#include <time.h>

void generarArreglo( int *matrix, int size ){
  srand( time( NULL ) );

  while( size -- )
    *matrix++ = rand( ) % 99 + 1;
}

void imprimirArreglo( int *matrix, int ancho, int size ) {
  int pos = 0;

  while( size-- ) {
    ::std::cout << "[" << *matrix++ << "]";
    if( ! ( ++pos % ancho ) )
      ::std::cout << ::std::endl;
  }

 ::std::cout << ::std::endl;
}

int main( ) {
  int *matriz;
  int n;

  ::std::cin >> n;
  matriz = (int *)calloc( n, n );

  generarArreglo( matriz, n * n );
  imprimirArreglo( matriz, n, n * n );

  system( "pause" );
  return 0;
}

    Версия 1.0. Патчи поддерживаются.

    PS . Говорят, что для такого кода C / C++ доступен только для записи .

    • 1

  4. Предлагаю объявить массив в виде:

    int mat[n][n];

    Только в качестве хорошей практики, потому что, если он объявлен динамически, хотя это делается автоматически, рекомендуется в конце концов память, которая была запрошена у системы, освобождаться вручную, однако, таким образом, это не обязательно. Другая вещь, которую я предлагаю, заключается в том, что передача массива в качестве параметра в виде указателя + размеры делает его гораздо более гибким при работе с массивом, если мы хотим внести последующие изменения в его размеры.

    • 0