Pasar array bidimensional por parámetro

[losmuertosdecristo]

No me aclaro con C++... Estoy intentando pasar por parámetro un array bidimensional y no puedo. Tengo una función así:

Código C++:

void Ingame::mapTo(int map1[50][50]) 
{ 
   for(int i = 0;i<50;i++) 
   { 
      for(int j = 0;j<50;j++) 
      { 
         actualMap[i][j] = map1[i][j]; 
      } 
   } 
}
 

Lo que quiero efectivamente es que el array bidimensional map1 se copie a actualMap, pero no me funciona, sin embargo el compilador no me dice que esté mal. He parado en ese punto en tiempo de ejecución y la variable map1 se muestra como un array de una dimensión.

Así llamo a la función:

Código C++:

mapTo(map1);
 

Creo que el fallo se puede ver fácilmente en esta imagen de depuración:



Espero que alguien sepa ayudarme.

Gracias.

[carbon]

A mi me funciona perfectamente.

[losmuertosdecristo]

Bueno, hay un pequeño matiz, empecé haciéndolo en un array de 50x50 y luego utilizo el de 8x8, pero en mi programa está en los dos sitios igual.

¿Me puedes decir en qué compilador lo has hecho? No sé si puede ser algo de eso.

¿Y cómo declaraste el array bidimensional?

Gracias.

[carbon]

Con C y MinGW:

Código C:

#include <stdio.h>
 
void copyArray(int map1[50][50])
{ 
    int actualMap[50][50];
    int i, j;
    
    for(i = 0;i<50;i++)
    {
        for(j = 0;j<50;j++)
        {
            actualMap[i][j] = map1[i][j];
            /* Verificar */
            printf("actualMap[%d][%d] = %d\n", i, j, actualMap[i][j]);
        }
    }
}
 
int main(int agrc, char **argv)
{
    int map[50][50];
    int i, j;
    
    for (i = 0; i < 50; i++)
    {
        for (j = 0; j < 50; j++)
        {
            map[i][j] = 15;
        }
    }
    
    copyArray(map);
    
    return 0;
}
 

[FjZOOM]

ES QUE SI TE FIJAS cuando tu pasas map1[50][50] a la funcion copyArray no estas pasando la matriz, mas bien si te fijas estas pasando el DATO que hay en la posicion de la matriz en las coordenadas (50,50)... ES POR ESO QUE NO TE MARCA ERROR EL COMPILADOR por que tu error es de logica (no estas planteando correctamente lo que tu programa necesita para llevar acabo correctamente el proceso de esa funcion). La solucion para pasar la matriz seria mandar como parametros el nombre de la matriz, el numero de columnas y el numero de filas algo asi Ejmplo: copyArray(map1, filas,columans); (como consejo define el tamaño de tu matriz en esas dos variables(int filas =50,int columnas=50) desde el incio de tu programa en el main o como variables globales digo no es muy recomendable pero por practicidad hazlo asi y ya despues que sepas un poquillo mas lo vas hacer mas elegentemente :) ).

pd espero a verte ayduado

[carbon]

Me parece que no es como vos decis. Un array es un puntero al primer elemento del array, por lo qur en realidad sí estoy pasando el array y no únicamente su valor. Además, el código funciona perfectamente bien.

[FjZOOM]

Carbon: tu codigo esta muy bien CARBON simpre y cuando sea en C, nuestro amigo lo que ocupa es hacerlo mismo que tu hiciste con tu codigo pero... en C++ y recuerda que C y C++ son lenguajes muy diferentes por que manejan tipo de programacion distinta en C++ se ocupa una programacion orientada a objetos el codigo que le ayudaria a nuestro amigo seria el siguiente.

Código C++:

#include <iostream> 
#include <cstdlib>
 
using namespace std;
 
 
      
      
      class Ingame{
            
            public:
                   Ingame();
                   void mapTo();
                   
            private:
                    int filas;
                    int columnas;
                    int map1[50][50];
            
            
            };
  Ingame::Ingame()
{
 
  
    int i, j;
    
    this->filas = 50;
    this->columnas = 50;
    
    for (i = 0; i < filas; i++)
    {
        for (j = 0; j < columnas; j++)
        {
            this->map1[i][j] = 15;
        }
    }
    
}
 
 
 
void Ingame::mapTo()
{ 
    int actualMap[filas][columnas];
    int i, j;
    
    for(i = 0;i<filas;i++)
    {
        for(j = 0;j<columnas;j++)
        {
            actualMap[i][j] = map1[i][j];
             
            cout << "actualMap" << "[" << i+1 << "]" << "[" << j+1 << "]" << "=" << actualMap[i][j] << endl;
        }
    }
}
 
  
 
int main(int agrc, char **argv)
{
    
    Ingame obj;
    
    obj.mapTo();
    
    system("pause");
  
    
    return 0;
}
 

Desarrolle este peque;o codigo considerando que pues el amigo esta usando la clase Ingame y la funcion miembro mapTo

Aclaro que me faltaron muchas practicas de buena programación pero en esencia asi es como se lleva acabo la construcción de un programa en C++ Y q tanto "el constructor",y demás variables que use unas se pueden declarar const y otras define etc pero debido a que solo me estoy basando en el peque;o ejemplo que el amigo puso considero que por ahi va lo que desea por que segun yo interpreto que esta usando clases y funciones miembro .El IDE que uso es Netbeans y lenguaje pos C++.

[L3m0n]

No es obligatorio programar con objetos en C++, ni es una mejor o peor práctica. C++ es un lenguaje a gusto del consumidor, quieres usar objetos, usalos, no quieres, pues no. Y así con muchas cosas.

[FjZOOM]

Bueno no creo que hayan creado C++ para seguir utilizando programcion estructurada en ese caso se hubieran quedado en C.

[carbon]

Iniciado por FjZOOM

Carbon: tu codigo esta muy bien CARBON simpre y cuando sea en C, nuestro amigo lo que ocupa es hacerlo mismo que tu hiciste con tu codigo pero... en C++ y recuerda que C y C++ son lenguajes muy diferentes por que manejan tipo de programacion distinta en C++ se ocupa una programacion orientada a objetos el codigo que le ayudaria a nuestro amigo seria el siguiente.

Código C++:

#include <iostream> 
#include <cstdlib>
 
using namespace std;
 
 
      
      
      class Ingame{
            
            public:
                   Ingame();
                   void mapTo();
                   
            private:
                    int filas;
                    int columnas;
                    int map1[50][50];
            
            
            };
  Ingame::Ingame()
{
 
  
    int i, j;
    
    this->filas = 50;
    this->columnas = 50;
    
    for (i = 0; i < filas; i++)
    {
        for (j = 0; j < columnas; j++)
        {
            this->map1[i][j] = 15;
        }
    }
    
}
 
 
 
void Ingame::mapTo()
{ 
    int actualMap[filas][columnas];
    int i, j;
    
    for(i = 0;i<filas;i++)
    {
        for(j = 0;j<columnas;j++)
        {
            actualMap[i][j] = map1[i][j];
             
            cout << "actualMap" << "[" << i+1 << "]" << "[" << j+1 << "]" << "=" << actualMap[i][j] << endl;
        }
    }
}
 
  
 
int main(int agrc, char **argv)
{
    
    Ingame obj;
    
    obj.mapTo();
    
    system("pause");
  
    
    return 0;
}
 

Desarrolle este peque;o codigo considerando que pues el amigo esta usando la clase Ingame y la funcion miembro mapTo

Aclaro que me faltaron muchas practicas de buena programación pero en esencia asi es como se lleva acabo la construcción de un programa en C++ Y q tanto "el constructor",y demás variables que use unas se pueden declarar const y otras define etc pero debido a que solo me estoy basando en el peque;o ejemplo que el amigo puso considero que por ahi va lo que desea por que segun yo interpreto que esta usando clases y funciones miembro .El IDE que uso es Netbeans y lenguaje pos C++.

Perdón, pensé que te referías a mi código, como te referiste a la función copyArray.

[polpolpol]

bueno estoy en pañales en esto de programar pero a mi me parece que deves hacer esto

void copyArray(int map1[50][50]) ---->> void copyArray(int map1[][])

borrar las dimensiones de la matriz, ya me diras si eso funciona
estoy estudiando struct y asi paso los parametros y me funcionan bien

[losmuertosdecristo]

Gracias a todos por comentar.

Carbon es verdad que tu código me ha funcionado en un proyecto nuevo, pero así no me funciona en mi programa tal como lo tengo estructurado, quizás por ahí venga el fallo.

FjZOOM el problema de hacerlo como lo haces es que necesito pasar un mapa por parámetro ya que tengo muchos y utilizo esta función para decirle al programa que mapa toca pintar. Al principio puse un switch para cada mapa, pero era enorme y trabajoso, por eso opté por una forma similar a esta, pero no termino de lograrlo.

polpolpol no me funcionó eso.

Seguramente en mi código haya muchas cosas mejorables, pero es mi primer intento de juego en C++ y voy mejorándolo según voy consiguiendo las cosas.