[SOLUCIONADO] Violación de segmento con matriz

[Metus]

Hola a todos, espero me puedan ayudar.

Estoy intentando llenar una matriz de [5][5] de forma aleatoria y que luego imprima el resultado.
El código es el siguiente.

Código C:

#include <stdio.h>
#include <stdlib.h>
 
void introduce(int matriz[][5]);
void leer(int matriz[][5]);
 
int main(int argc, char const *argv[])
{
    int matriz[5][5];
 
    introduce(matriz);
    leer(matriz);
 
    return 0;
}
 
void introduce(int matriz[][5]){
    int i, j;
 
    srand(time(NULL));
 
    for (i = 0; i < 5; ++i){
        for (j = 0; j < 5; ++j){
            matriz[i][j] = rand()%10;
        }
    }
}
 
void leer(int matriz[][5]){
    int i, j;
 
    for (i = 0; i < 5; ++i){
        for (j = 0; j < 5; ++i){
            printf("%i ", matriz[i][j]);
        }
        printf("\n");
    }
}
 

Al compilar no me manda ningún error

Código BASH:

metus@metus ~ $ gcc -o matriz matriz.c
metus@metus ~ $
 

pero al ejecutarlo me salen estos resultados:

Código BASH:

metus@metus ~ $ ./matriz
 

Código Otro:

0 6 0 0 9 32767 1496751813 32767 0 0 0 1761281253 0 32767 0 32767 -1759235576 0 -1759231058 32767 -1759230922 32767 -1759230773 32767 -1759229309 32767 -1759229037 32767 -1759228731 32767 -1759228555 32767 -1759228461 32767 -1759228264 32767 -1759228102 32767 -1759227967 0 16 0 100 0 5 0 0 0 12 0 1000 0 31 0 0 -1920327830 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 1769108577 1397052255 1414415687 1212370517 1027885392 1028605518 1769109861 1163132977 1647275625 1263488835 1697998177 808530993 825636400 1431327829 926104900 1634296933 1936020015 1852397359 1414418243 1798254640 1162108755 995250002 1397489779 859517232 1030320186 825245039 876438627 976302139 1633892915 859666287 993079357 2053600302 708456755 993079357 1819553322 976302907 859517232 993079357 809335404 2053254698 708456755 825244978 1650811950 708456755 993079357 1918989102 708456755 993079357 2050436650 774519345 993079357 1886216750 708457779 993079357 1835169838 708457779 825245030 1987259946 892549937 809334883 1831742010 859517232 1031170925 708457779 993079357 879783214 976565051 993079357 1718837550 774519349 993079357 1768711726 708457779 892549937 809329783 1663969850 892549937 809334894 1865296442 909327152 809329505 1831742010 859516976 1029927021 774519350 909327152 809328999 2016291386 1230128223 1764713065 1966029417 762869102 1129272159 2036689711 1397966163 1701654380 796420462 1886221359 1380275029 1599296588 1852793703 1634741876 1263748420 1937077349 1969318957 1953723747 1869900659 1697594707 792360044 1831822701 795631971 980314466 792358505 1936026977 1162084467 1819631974 1230258499 1831822701 1735357040 1852139876 3681606 942751044 1966030152 1969317477 1145897064 1815957071 1634100580 1313818963 133Violación de segmento
 

Espero me puedan ayudar.

[Drewermerc]

Hola amigo.
bueno lo único que te faltaba era incluir la librería time.h ya que usas una de sus funciones que es time(NULL) para la semilla de srand y al leer en el segundo for te hacia falta la letra j por que pusiste la i y buen tu código que daría así.

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
 
void introduce(int matriz[5][5]);
void leer(int matriz[5][5]);
 
int main(int argc, char const *argv[])
{
    int matriz[5][5];
 
    introduce(matriz);
    leer(matriz);
 
    return 0;
}
 
void introduce(int matriz[5][5]){
    int i, j;
 
    srand(time(NULL));
 
    for (i = 0; i < 5; i++){
        for (j = 0; j < 5; j++){
            matriz[i][j] = rand()%10;
        }
    }
}
 
void leer(int matriz[5][5]){
    int i, j;
 
    for (i = 0; i < 5; i++){
        for (j = 0; j < 5; j++){
            printf("%d ",matriz[i][j]);
        }
        printf("\n");
    }
}
 

Bueno espero que te sirva.
Saludos.
Drewermerc.

[vosk]

Antes de hacer nada mas revisa tranquilamente tu codigo, concretamente la linea 33:

Código C:

for (j = 0; j < 5; ++i){
 

Te oriento un poco con una pregunta: para que el ++i si estas iterando las j?

Ahora la parte tecnica: para enviar un puntero al array tienes que recibir un puntero al primer elemento del array:

Código C:

void leer(int (*matriz)[5]) {
    ...
}
 

Compara esta declaracion de funcion con la que tu tienes en el codigo.

Y ademas supongo que te interesará reutilizar el codigo haciendo que la funcion no se limite a una matriz de 5x5, sino de nxm, solo tienes que recibir el nº de filas, de columnas y el array de filas x columnas:

Código C:

void leer(int n, int m, int a[n][m]) {
    ...
}
 

Espero que te sea de ayuda

Saludos
vosk

[vosk]

Ui perdon Drewermerc, es la segunda vez en dos dias que mientras escribo el post alguien responde antes :))

Saludos
vosk

[Drewermerc]

No te preocupes Vosk también le hiciste recomendaciones que yo no, así se hace una mejor respuesta.
Saludos.
Drewermerc.

[Metus]

Muchas gracias a los 2, creo que son errores de novato, llevo pocos días intentando aprender C, supongo que suele suceder aveces.

Revisare el código en un rato y les comento como me fue.

Me fue bien, funciono tal cual los cambios que me dijeron que hiciera.