problemas con punteros y matrices

[aguml]

Hola amigos, estoy intentando hacer un ejercicio donde se pide la dimension de unas matrices y luego se suman y muestro el resultado de la matriz final. El problema lo tengo en la parte donde solicito memoria dinamica para las matrices ya que da una excepcion. Este es mi codigo:

Código C++:

#include <iostream>
 
using namespace std;
 
void solicitar_dimensiones( int *f, int *c)
{
    cout << "Ingrese el numero de filas para la matriz : ";
    cin >> *f;
    cout << "\nIngrese el numero de columnas para la matriz : ";
    cin >> *c;
}
 
void pedir_memoria(int ***a,int f,int c)
{
    *a= new int*[f];
    for (int i = 0; i <f; i++)
    {
        *(a[i]) = new int[c];
    }
}
 
void rellenar_matriz(int ***a,int indice,int f,int c)
{
    cout << "\nIngrese los valores de la matriz " << indice <<" : ";
    for (int i = 0; i < f; i++)
    {
        for (int j = 0; j < c; j++)
        {
            cout << "\nIngrese el elemento [ " << i << ", " << j << " ]: ";
            cin >> *(a[i][j]);
        }
        printf("\n");
    }
}
 
void sumar_matrices (int ***a,int ***puntm2,int ***puntm,int f, int c)
{
    for (int i = 0; i < f; i++)
    {
        for (int j = 0; j < c; j++)
        {
            *(a[i][j])= *(puntm[i][j]) + *(puntm2[i][j]);
        }
    }
}
 
void liberar_memoria(int **a,int f)
{
    for (int i = 0; i <f; i++)
    {
        delete[] a[i];
    }
    delete[] a;
}
 
void imprimir_matriz(int **a, int f, int c )
{
    cout << "\nLa matriz final es: \n";
    for (int i = 0; i < f; i++)
    {
        for (int j = 0; j < c; j++)
        {
            cout << a[i][j] <<endl;
        }
    }
}
 
int main()
{
    int f,c;
    int **psumas;
    int **pmatriz2;
    int **pmatriz1;
 
    solicitar_dimensiones(&f,&c);
    pedir_memoria(&pmatriz1,f,c);
    pedir_memoria(&pmatriz2,f,c);
    pedir_memoria(&psumas,f,c);
 
    rellenar_matriz(&pmatriz1,1,f,c);
    rellenar_matriz(&pmatriz2,2,f,c);
 
    sumar_matrices(&psumas,&pmatriz2,&pmatriz1,f,c);
 
    imprimir_matriz(psumas,f,c);
 
    liberar_memoria(pmatriz1,f);
    liberar_memoria(pmatriz2,f);
    liberar_memoria(psumas,f);
    return 0;
}
 

¿alguien podria ayudarme indicando donde esta el fallo y como solucionarlo?

[aguml]

He estado intentando solucionarlo pero tengo un lio muy gordo. Ahora mismo lo tengo asi pero al rellenar el array lo desbordo y no veo como hacerlo de la forma correcta:

Código C++:

#include <iostream>
 
using namespace std;
 
void solicitar_dimensiones( int *f, int *c)
{
    cout << "Ingrese el numero de filas para la matriz : ";
    cin >> *f;
    cout << "\nIngrese el numero de columnas para la matriz : ";
    cin >> *c;
}
 
void pedir_memoria(int ***a,int f,int c)
{
    a[0]= new int*[f];
    for (int i = 0; i <f; i++)
    {
        a[0][i] = new int[c];
    }
}
 
void rellenar_matriz(int ***a,int indice,int f,int c)
{
    cout << "\nIngrese los valores de la matriz " << indice <<" : ";
    for (int i = 0; i < f; i++)
    {
        for (int j = 0; j < c; j++)
        {
            cout << "\nIngrese el elemento [ " << i << ", " << j << " ]: ";
            cin >> *a[i][j];
        }
        printf("\n");
    }
}
 
void sumar_matrices (int ***a,int ***puntm2,int ***puntm,int f, int c)
{
    for (int i = 0; i < f; i++)
    {
        for (int j = 0; j < c; j++)
        {
            *(a[i][j])= *(puntm[i][j]) + *(puntm2[i][j]);
        }
    }
}
 
void liberar_memoria(int **a,int f)
{
    for (int i = 0; i <f; i++)
    {
        delete[] a[i];
    }
    delete[] a;
}
 
void imprimir_matriz(int **a, int f, int c )
{
    cout << "\nLa matriz final es: \n";
    for (int i = 0; i < f; i++)
    {
        for (int j = 0; j < c; j++)
        {
            cout << a[i][j] <<endl;
        }
    }
}
 
int main()
{
    int f,c;
    int **psumas;
    int **pmatriz2;
    int **pmatriz1;
 
    solicitar_dimensiones(&f,&c);
    pedir_memoria(&pmatriz1,f,c);
    pedir_memoria(&pmatriz2,f,c);
    pedir_memoria(&psumas,f,c);
 
    rellenar_matriz(&pmatriz1,1,f,c);
    rellenar_matriz(&pmatriz2,2,f,c);
 
    sumar_matrices(&psumas,&pmatriz2,&pmatriz1,f,c);
 
    imprimir_matriz(psumas,f,c);
 
    liberar_memoria(pmatriz1,f);
    liberar_memoria(pmatriz2,f);
    liberar_memoria(psumas,f);
    return 0;
}
 

[Fw190]

Quizá te estás complicando la vida con tu manera de definir las matrices. En lugar de utilizar triples y dobles punteros, yo te sugiero el empleo de una estructura de datos, por ejemplo:

Código C++:

typedef struct
{
    int fil;
    int col;
    int **A;
}
T_Matriz;
 
void pedir_memoria(T_Matriz *M, int f, int c)
{
    M->fil = f;
    M->col = c;
    M->A = new int*[f];
    for (int i=0; i<c; i++)
        M->A[i] = new int[c];
}
 

y de un modo similar para todas las funciones que vayas a crear para operar con matrices.

Si quieres, aprovecha para comprobar (en función de lo que quieras hacer) que el número de filas y/o columnas es el correcto, si el apuntador A corresponde o no a un bloque de datos válido, etc.

Ya que estás empleando C++, ¿Por qué no implementas las matrices y sus operaciones como una clase?

Saludos,

[aguml]

Es que es un ejercicio precisamente de punteros y matrices y la idea es acerlo así como hago para entender bien su uso. Tiene que ser como lo hago.
He visto que hacen algo similar en una duda de un foro y lo ponen asi:

Código C++:

*(*(*matS+i)+j) = *(*(matA + i) + j) + *(*(matB + i) + j);
 

Sí cojo ese mismo código y lo convierto usando indices:

Código C++:

*matS[i][j] = matA[i][j] + matB[i][j];
 

Ya no funciona. ¿alguien me puede explicar esto? ¿acaso no puedo usar índices para desplazarme por una matriz de este tipo cuando se usa memoria dinámica?

[Fw190]

Quieres representar una matriz mediante un doble puntero, con el fin de poder utilizar la notación clásica para los elementos de la matriz.

El primer fallo está en la función pedir_memoria():

Código C++:

void pedir_memoria(int ***a,int f,int c)
 

Con esa declaración, lo que estoy diciendo es "paso la dirección de un doble puntero para poderlo modificar". Es decir, el parámetro a representa un puntero a un doble puntero.

Para poder modificar un pàrámetro por referencia, tengo que decir "lo apuntado por a es igual a ...", luego ya tenemos fallo en la primera línea de la función. Debería ser:

Código C++:

*a = new int*[f]
 

o bien, empleando una variable intermedia (creo que te quedará más claro), dejar tu función pedir_memoria() como:

Código C++:

void pedir_memoria(int ***a,int f,int c)
{
    int **b;
 
    b = new int*[f];
    for (int i = 0; i <f; i++)
    {
        b[i] = new int[c];
    }
    *a = b;
}
 

Ves que dentro del bucle solamente se emplea un índice. Ese era otro fallo en tu código original, ten en cuenta que, ahora, b es un arreglo de punteros, cada uno de ellos apuntará a un arreglo de enteros. que es lo que constituirán las filas.

Una vez establecido que las matrices las estabas generando mal, ya sabes por qué te daba la excepción: estabas metiendo en memoria lo que no debías en donde no debías.

Para las demás funciones ya no necesitas el triple puntero puesto que ya no necesitas modificar el doble puntero. Pasando éste directamente, ya tienes acceso a los elementos sin necesidad de complicarte la vida (ya estás pasando las direcciones al pasar el puntero). Por ejemplo:

Código C++:

void rellenar_matriz(int **a, int indice, int f, int c)
{
    cout << "\nIngrese los valores de la matriz " << indice <<" : ";
    for (int i = 0; i < f; i++)
    {
        for (int j = 0; j < c; j++)
        {
            cout << "\nIngrese el elemento [ " << i << ", " << j << " ]: ";
            cin >> a[i][j];
        }
        printf("\n");
    }
}
 

y de una forma similar deberás modificar la función sumar_matrices().

En tu función main(), las llamadas a pedir_memoria() serían correctas, pero no las llamadas a rellenar_matriz() y sumar_matrices(). También serán correctas las llamadas a imprimir_matriz() y liberar_memoria(). Pero, en el caso de la función liberar_memoria(), ten en cuenta que pmatriz1, pmatriz2 y psumas siguen apuntando a un bloque de memoria que ya se liberó. En este caso no tiene importancia, dado que el programa se acaba, pero puede dar lugar a problemas si no tienes cuidado en programas grandes en donde pretendas reutilizar esas variables.

Sobre la conversión usando índices... bueno, si partíamos de una matriz mal creada... pero, de todos modos, con lo que te he contado, no necesitarás el triple índice más que en la creación de la matriz.

Espero haber arrojado algo de claridad sobre el tema.

Saludos,

[aguml]

No termino de entenderlo porque en mi primer código lo tenia como indicas y usaba un solo indice y fallaba igual. ¿te importaría coger la función pedir_memoria de mi primer código y corregir lo que veas mal? Y así puedo ver el fallo porque me tiene loco y no lo termino de entender. Me gustaría hacerlo sin usar una variable intermedia porque la idea es enterarse bien de como hacerlo directamente. Gracias por tu tiempo.

[Fw190]

Te incluyo un programa de ejemplo, hecho con Codeblocks versión 16.01 y MinGW 4.9.2 (32 bits)

Las funciones pedir_memoria_1() y borrar_memoria_1(), que utilizan directamente el puntero, me provocan ambas un error de segmentación.

Las funciones pedir_memoria_2() y borrar_memoria_2(), que (en teoría) funcionan como las versiones _1, pero utilizando una variable intermedia, me funcionan perfectamente. La función borrar_memoria_2() lo que hace es poner el puntero original a NULL una vez que libera la memoria (ya te comenté por qué hacerlo así).

La función borrar_memoria_3() es la tuya original.

La función matriz_prueba() es para comprobar que la matriz se ha creado correctamente.

¿Por qué funcionan las versiones que usan una variable intermedia y las otras no? Francamente, no lo sé, pero habiendo un modo de que funcione...

Código C++:

#include <iostream>
 
using namespace std;
 
void pedir_dimension (int *f, int *c);
void pedir_memoria_1 (int ***a, int f, int c);
void pedir_memoria_2 (int ***a, int f, int c);
void matriz_prueba   (int **a,  int f, int c);
void imprimir_matriz (int **a,  int f, int c);
void borrar_matriz_1 (int ***a, int f, int c);
void borrar_matriz_2 (int ***a, int f, int c);
void borrar_matriz_3 (int **a, int f, int c);
 
int main()
{
    int    fils, cols;
    int    **A;
 
    pedir_dimension(&fils, &cols);
    //pedir_memoria_1(&A, fils, cols);
    pedir_memoria_2(&A, fils, cols);
    matriz_prueba(A, fils, cols);
    imprimir_matriz(A, fils, cols);
    //borrar_matriz_1(&A, fils, cols);
    borrar_matriz_2(&A, fils, cols);
    //borrar_matriz_3(A, fils, cols);
 
    return 0;
}
 
void pedir_dimension (int *f, int *c)
{
    cin >> "Filas: "
    cin >> *f;
    cin >> "Columnas: ";
    cin >> *c;
 
    cout << endl;
}
 
void pedir_memoria_1 (int ***a, int f, int c)
{
    *a = new int*[f];
    for (int i=0; i<f; i++)
    {
        *a[i] = new int[c];
    }
}
 
void pedir_memoria_2 (int ***a, int f, int c)
{
    int **b;
 
    b = new int*[f];
    for (int i=0; i<f; i++)
    {
        b[i] = new int[c];
    }
    *a = b;
}
 
void matriz_prueba (int **a, int f, int c)
{
    for (int i=0; i<f; i++)
        for (int j=0; j<c; j++)
            a[i][j] = 10*(i+1) + (j+1);
}
 
void imprimir_matriz (int **a, int f, int c)
{
    for (int i=0; i<f; i++)
    {
        for (int j=0; j<c; j++)
        {
            cout << a[i][j] << " ";
        }
        cout << endl;
    }
}
 
void borrar_matriz_1 (int ***a, int f, int c)
{
    for (int i=0; i<f; i++)
        delete[] *a[i];
    delete[] *a;
    *a = NULL;
}
 
void borrar_matriz_2 (int ***a, int f, int c)
{
    int **b;
 
    b = *a;
    for (int i=0; i<f; i++)
        delete[] b[i];
    delete[] b;
    *a = NULL;
}
 
void borrar_matriz_3 (int **a, int f, int c)
{
    for (int i=0; i<f; i++)
        delete[] a[i];
    delete[] a;
}
 

De todos modos, me pica la curiosidad, voy a probar utilizando C puro y duro con malloc() a ver qué pasa. Ya te comentaré.

Saludos,

[PercevalCiro]

Fw190, esta muy bueno, pero marco un detalle, escribis:

Código C:

cin >> "Filas: "
    cin >> *f;
    cin >> "Columnas: ";
    cin >> *c;
 

debería ser:

Código C:

cout >> "Filas: "
    cin >> *f;
    cout >> "Columnas: ";
    cin >> *c;
 

Sino tira muchos errores...

[Fw190]

Tienes razón, no fue más que un simple error de copia/pega.