Segmentation Fault al realizar funcion sobre Matriz

[Manu_The_King]

Hola y muy buenas gente,

voy a intentar resumir un poco mi problema, a ver si podeis echarme un cable  

vereis estoy trabajando con una matriz base donde el usuario introduce la dimension y los valores, despues accedera a un menu con distintas opciones.

Ya he realizado 2 funciones distintas sin problema, pero ahora en la 3a, cuando intenta hacer algo con la matriz salta el segmentation fault, he llegado a la conclusión de que en la funcion, la matriz no tiene los valores que deberia y por eso salta.

La funcion en cuestion es:

Código [Seleccionar] Expandir


void Trasposada(int fil,int col,int mat[MAX][MAX])
{
   int i, j;
   int tras[MAX][MAX];

   for(i=0;i<fil;i++)
       for(j=0;j<col;j++)
           tras[i][j]=mat[i][j];

   for(j=0;j<fil;j++)
   {
       for(i=0;i<col;i++)
           printf("%d", mat[i][j]);
       printf("\n");
   }

}


Gracias de antemano ^^

[avesudra]

Es que así solo con la función no se que hace, esta función que es ¿para mostrar la matriz? , que valores debo pasarle a fil y a col, ¿el valor de MAX? Es que si no no puedo probar xD. Si pudieses poner el código entero lo miramos mejor porque igual el problema no es la función si no los valores que se pasan.

¡Un saludo!

[Manu_The_King]

te dejo el main (las otras funciones no intervienen)

Código [Seleccionar] Expandir


#include <stdio.h>
#include <stdlib.h>
#define MAX 10

void Trasposada(int fil,int col,int mat[MAX][MAX])
{
   int i, j;
   int tras[MAX][MAX];

   for(i=0;i<fil;i++)
       for(j=0;j<col;j++)
           tras[i][j]=mat[i][j];

   for(j=0;j<fil;j++)
   {
       for(i=0;i<col;i++)
           printf("%d", mat[i][j]);
       printf("\n");
   }

}


int main()
{
   int  i, j, fil=0, col=0, maxm, maxf;
   int mat[MAX][MAX];

//leer nº de filas y columnas

   printf("Indica les FILES que tindra la matriu(max 10):\n");
   scanf("%d", &fil);
   while(fil>MAX)
   {
       printf("Error. Has d'introduir un valor menor de 10\n");
       scanf("%d", &fil);
   }

   printf("Indica les COLUMNES que tindra la matriu(max 10):\n");
   scanf("%d", &col);
   while(col>MAX)
   {
       printf("Error. Has d'introduir un valor menor de 10\n");
       scanf("%d", &col);
   }

//meter valores

   for(i=0;i<fil;i++)
   {
       printf("Introdueix els valors de la fila %d\n", i);
       for(j=0;j<col;j++)
           scanf("%d", &mat[i][j]);
   }


   Trasposada(fil,col,mat[MAX][MAX]);



   return 0;
}


en si el problema es, que no entiendo porque dentro de la funcion Trasposada la matriz no mantiene los valores, mientras que en las otras funciones si lo hace

[avesudra]

Pues es un despiste, date cuenta que si le pasas a la función esto:

Código (cpp) [Seleccionar] Expandir

for(i=0;i<fil;i++)
   {
       printf("Introdueix els valors de la fila %d\n", i);
       for(j=0;j<col;j++)
           scanf("%d", &mat[i][j]);
   }


   Trasposada(fil,col,mat[MAX][MAX]);//<------------------------- Me refiero a eso
Es lo mismo que hacer esto:

Código (cpp) [Seleccionar] Expandir

Trasposada(fil,col,mat[10][10]);
Y tu le tienes que pasar la dirección inicial de la matriz es decir la 0,0 para que la vaya recorriendo:

Código (cpp) [Seleccionar] Expandir

Trasposada(fil,col,mat[0][0]);
O si lo prefieres así:

Código (cpp) [Seleccionar] Expandir

Trasposada(fil,col,&mat);
El código no mantenía los valores porque estaba accediendo a una memoria que estaba inicializada pero con valores aleatorios 
Así que el código quedaría así:

Código (c) [Seleccionar] Expandir

#include <stdio.h>
#include <stdlib.h>
#define MAX 10

void Trasposada(int fil,int col,int mat[MAX][MAX])
{
   int i, j;
   int tras[MAX][MAX];

   for(i=0; i<fil; i++)
       for(j=0; j<col; j++)
           tras[i][j]=mat[i][j];

   for(j=0; j<fil; j++)
   {
       for(i=0; i<col; i++)
           printf("%d",mat[i][j]);
       printf("\n");
   }

}

int main()
{
   int  i, j, fil=0, col=0, maxm, maxf;
   int mat[MAX][MAX];
   for(i=0; i!= MAX; ++i)
   {
       for(j=0; j!= MAX; ++j)
           mat[i][j]=0;
   }
   //leer nº de filas y columnas

   printf("Indica les FILES que tindra la matriu(max 10):\n");
   scanf("%d", &fil);
   while(fil>MAX)
   {
       printf("Error. Has d'introduir un valor menor de 10\n");
       scanf("%d", &fil);
   }

   printf("Indica les COLUMNES que tindra la matriu(max 10):\n");
   scanf("%d", &col);
   while(col>MAX)
   {
       printf("Error. Has d'introduir un valor menor de 10\n");
       scanf("%d", &col);
   }

  //meter valores

   for(i=0; i<fil; i++)
   {
       printf("Introdueix els valors de la fila %d\n", i);
       for(j=0; j<col; j++)
           scanf("%d", &mat[i][j]);
   }

   Trasposada(fil,col,&mat);

   return 0;
}

[Manu_The_King]

que grande eres y que mal estoy xD estos examenes me tienen el cerebro frito

muchas gracias por la ayuda!!

[rir3760]

En el caso de los arrays cuando su identificador se utiliza en una llamada a funcion no es necesario el operador "direccion de" (el '&'). Esta sentencia:

Código (c) [Seleccionar] Expandir

Trasposada(fil,col,&mat);
Se puede reducir a:

Código (c) [Seleccionar] Expandir

Trasposada(fil, col, mat);
Sin problemas.

Un saludo

[Manu_The_King]

Buenas, vuelvo a tener problemas con esta fucion -.-'

ahora ya lo tengo todo montado y funciona correctamente, el único problema es que en esta misma funcion (trasposada) no lee los valores de la matriz original, por lo que sigue teniendo valores aleatorios, os dejo el condigo con el main a ver si me podeis echar una mano

Código [Seleccionar] Expandir


int Trasposada(int tras[MAX][MAX],int fil,int col, int mat[MAX][MAX])
{
    int i, j;

    for (i=0;i<fil;i++)
        for (j=0;j<col;j++)
        {
            tras[j][i] = mat[i][j];

        }
    for (i=0;i<fil;i++)
    {
        for (j=0;j<col;j++)
            printf("%d ", &mat[i][j]);   //lo he puesto asi para probar si realmente leia la matriz
        printf("\n");
    }


}




int main()
{
    int  i, j, fil=0, col=0, maxm, maxf, x, opinf, opsup;
    int mat[MAX][MAX];
    int esc[MAX][MAX];
    int tras[MAX][MAX];

    printf("Benvingut al programa. MATRIUS.\n\n");


    printf("Indica les FILES que tindra la matriu(max 10):\n");
    scanf("%d", &fil);
    while(fil>MAX)
    {
        printf("Error. Has d'introduir un valor menor de 10\n");
        scanf("%d", &fil);
    }

    printf("Indica les COLUMNES que tindra la matriu(max 10):\n");
    scanf("%d", &col);
    while(col>MAX)
    {
        printf("Error. Has d'introduir un valor menor de 10\n");
        scanf("%d", &col);
    }

    for(i=0;i<fil;i++)
    {
        printf("Introdueix els valors de la fila %d\n", i);
        for(j=0;j<col;j++)
            scanf("%d", &mat[i][j]);
    }


//    dimensio(fil);
//    valors(fil, col, mat);
    printf("\n\nLa matriu introduida es:\n\n");
    for(i=0;i<fil;i++)
        {
            for(j=0;j<col;j++)
                printf("%d ",mat[i][j]);
            printf("\n");
        }

    while(x!=0)
    {




        menu();
        x=Opcio(opinf, opsup);

        switch(x)
        {
            case 1:Valor_Maxim_M(maxm, fil, col, mat);break;
            case 2:Valor_Maxim_F(maxf, fil, col, mat);break;
            case 3:Trasposada(tras,fil,col,mat);break;   
            case 4:Escalar(esc,fil,col,mat);break;
            case 5:Identitat(fil,col,mat);break;
            case 0: printf("FINALITZA EL PROGRAMA\n\n");break;
            default: printf("Has introduit malament la opcio. Torna a intentar-ho.\n\n");
        }
    }


    return 0;
}



Muchas gracias de antemano

[avesudra]

Pon los valores con los que pruebas por favor   porque con una matriz de 3x3 parece que funciona.

[Manu_The_King]

tal i como esta te da bien los valores por pantalla??? es que a mi me da valores aleatorios :S

he estado probando con

1 fila, 4 columnas, valores: 1234

2 filas, 3 columnas, valores 123
                                         456

[avesudra]

Tal y como está no porque me dan errores de compilacion porque me faltan la mitad de las funciones Está así:

Código (c) [Seleccionar] Expandir

#include <stdio.h>
#define MAX 10
int Trasposada(int tras[MAX][MAX],int fil,int col, int mat[MAX][MAX])
{
    int i, j;

    for (i=0; i<fil; i++)
        for (j=0; j<col; j++)
        {
            tras[j][i] = mat[i][j];

        }
    for (i=0; i<fil; i++)
    {
        for (j=0; j<col; j++)
            printf("%d ", mat[i][j]);   //lo he puesto asi para probar si realmente leia la matriz
        printf("\n");
    }


}
int main()
{
    int  i, j, fil=0, col=0, maxm, maxf, x, opinf, opsup;
    int mat[MAX][MAX];
    int esc[MAX][MAX];
    int tras[MAX][MAX];

    printf("Benvingut al programa. MATRIUS.\n\n");


    printf("Indica les FILES que tindra la matriu(max 10):\n");
    scanf("%d", &fil);
    while(fil>MAX)
    {
        printf("Error. Has d'introduir un valor menor de 10\n");
        scanf("%d", &fil);
    }

    printf("Indica les COLUMNES que tindra la matriu(max 10):\n");
    scanf("%d", &col);
    while(col>MAX)
    {
        printf("Error. Has d'introduir un valor menor de 10\n");
        scanf("%d", &col);
    }

    for(i=0; i<fil; i++)
    {
        printf("Introdueix els valors de la fila %d\n", i);
        for(j=0; j<col; j++)
            scanf("%d", &mat[i][j]);
    }
//    dimensio(fil);
//    valors(fil, col, mat);
    printf("\n\nLa matriu introduida es:\n\n");
    for(i=0; i<fil; i++)
    {
        for(j=0; j<col; j++)
            printf("%d ",mat[i][j]);
        printf("\n");
    }

    x=3;
    switch(x)
    {
        /* case 1:Valor_Maxim_M(maxm, fil, col, mat);break;
         case 2:Valor_Maxim_F(maxf, fil, col, mat);break;*/
    case 3:
        Trasposada(tras,fil,col,mat);
        break;
        /*case 4:Escalar(esc,fil,col,mat);break;*/
        /* case 5:Identitat(fil,col,mat);break;
         case 0: printf("FINALITZA EL PROGRAMA\n\n");break;*/
    default:
        printf("Has introduit malament la opcio. Torna a intentar-ho.\n\n");
    }
    return 0;
}
La salida de ese código me da esto:

Código [Seleccionar] Expandir


Indica les FILES que tindra la matriu(max 10):
2
Indica les COLUMNES que tindra la matriu(max 10):
3
Introdueix els valors de la fila 0
1
2
3
Introdueix els valors de la fila 1
4
5
6


La matriu introduida es:

1 2 3
4 5 6
1 2 3
4 5 6

Process returned 0 (0x0)   execution time : 22.177 s
Press any key to continue.