Vejamos...
Vou listar os erros observados um a um, separados por cada função. Tente verificar se consegue percebê-los para que aprenda para seu futuro, OK?
cria_matriz(...)
1 - Não tem por que você usar o tipo Matriz *
para a função cria_matriz(...)
. Por quê? Simplesmente por que você passou o endereço de m
como argumento a função. Toda modificação será feita no próprio m
passado como argumento. Assim sendo, sua função fica definida assim:
void cria_matriz( Matriz *m, int mat[5][5] ){ ... }
2 - É sempre uma boa prática fazer um type casting do tipo do ponteiro recebendo alocação de memória. Suas alocações ficariam:
m->elementos = (int **) malloc( m->n_elementos * sizeof(int *) )
...
m->elementos[i] = (int *) calloc( m->n_elementos, sizeof(int) );
3 - m->n_elementos
não foi definido ao ser criado. Isso por que m
é um ponteiro. Logo, podemos assumir que seu tamanho é 0
. Isso vai fazer com que você crie nenhuma memória para m->elementos
ou m->elementos[n]
. Assim, antes de alocar memória com malloc(...)
ou calloc(...)
, você precisa definir o tamanho de m->n_elementos
:
m->n_elementos = 5 * 5; /* 5 linhas * 5 colunas */
4 - Em verdade, você não quer definir o espaço de cada linha e coluna da matriz m
com m->n_elementos
já que este é o total de elementos. Sua matriz[5][5]
tem um total de 25
elementos porém divididos em 5 linhas e 5 colunas.
Pensando nisso, em malloc(...)
você deve substituir por 5 linhas e em calloc(...)
deve substituí-lo por 5 colunas. Efetivamente, você cria o espaço para 5 linhas com malloc(...)
e cada linha recebe 5 elementos de calloc(...)
. Portanto:
m->elementos = (int **) malloc( 5 * sizeof(int *) ); /* 5 linhas */
...
m->elementos[i] = (int *) calloc( 5, sizeof(int) ); /* 5 colunas */
Cada um dos laços for
utilizados também precisam ser alterados:
for (int i = 0; i < 5; i++) {...} /* 5 linhas */
...
for (int j = 0; j < 5; j++) {...} /* 5 colunas */
5 - Tanto LINHAS
como COLUNAS
foram criados com a diretiva de pré-processamento #define
. Foram usados para tornar o código mais legível. São colocadas logo após os #include
:
#include ...
#define LINHAS 5
#define COLUNAS 5
Foram substitutos de m->n_elementos em
em malloc(...)
, calloc(...)
e laços for
, mencionados no item 4, como segue:
m->elementos = (int **) malloc( LINHAS * sizeof(int *) );
...
for (int i = 0; i < LINHAS; i++) {...}
...
m->elementos[i] = (int *) calloc( COLUNAS, sizeof(int) );
...
for (int j = 0; j < COLUNAS; j++) {...}
O mesmo foi feito para m->n_elementos
mencionado no item 3:
m->n_elementos = LINHAS * COLUNAS;
Nos casos seguintes, serei mais econômico com exemplos de código. Não há motivo para chover no molhado pois os exemplos são os mesmos. No final também será mostrado o código completo e funcional.
imprimir_matriz(...);
1 - Aqui a recomendação é mudar o parâmetro de imprimir_matriz(...) para ponteiro, evitando criar memória extra. Se for considerar que toda a temática é alocação dinâmica, mais correto ainda. Veja:
void imprimir_matriz(Matriz *m){...}
2 - Como agora m
internamente a imprimir_matriz(...)
é ponteiro, deve-se trocar .
por ->
.
3 - Seguindo a mesma lógica apontada no item 4 e melhorado no item 5 de cria_matriz(...)
, teríamos mudanças nos laços for
conforme apontado.
main()
1 - Recomendo trocar a definição de matriz[5][5]
para matriz[LINHAS][COLUNAS]
.
2 - Recomendo atribuir os valores da matriz separados por cada linha e coluna. Isso pois -Wall
é muito usado ao compilar um código C pois fornece avisos de estruturas não muito aceitas por certos desenvolvedores. Tem-se:
int mat[LINHAS][COLUNAS] = {
{ 0, 185, 119, 152, 133 },
{ 185, 0, 121, 150, 200 },
{ 199, 121, 0, 174, 120 },
{ 152, 150, 174, 0, 199 },
{ 133, 200, 120, 199, 0 }
};
3 - Precisa-se alterar a chamada de imprimir_matriz(...)
para fornecer o endereço do de m
:
imprimir_matriz( &m );
4 - Umas das primeiras coisas que devemos criar é a função que vai liberar a memórias do que foi alocado pois já temos planejada a estrutura de dados a ser criada. Presumo que esteja na lista de afazeres.
Código funcional
Segue seu código de maneira funcional, com os problemas corrigidos. Era uma cadeia de erros possíveis de serem corrigidos por você mas que fatores outros podem ter atrapalhado a percepção dos erros presentes. Sem mais delongas:
Diretivas de pré processamento:
#include <stdio.h>
#include <stdlib.h>
#define LINHAS 5
#define COLUNAS 5
Estrutura de dados:
typedef struct matriz
{
int n_elementos;
int **elementos;
} Matriz;
Função cria_matriz(...)
:
void cria_matriz( Matriz *m, int mat[5][5] ){
int i, j;
m->n_elementos = LINHAS * COLUNAS;
m->elementos = (int **) malloc( LINHAS * sizeof(int *) );
for ( i = 0; i < LINHAS; i++ ){
m->elementos[i] = (int *) calloc( COLUNAS , sizeof(int) );
for ( j = 0; j < COLUNAS; j++ ) m->elementos[i][j] = mat[i][j];
}
}
Função imprimir_matriz(...)
:
void imprimir_matriz( Matriz *m ){
int i, j;
printf("\nMatriz\n\n");
for( i = 0; i < LINHAS; i++ ){
for( j = 0; j < COLUNAS; j++ ) printf("%d ", m->elementos[i][j]);
printf("\n");
}
printf("\n");
}
Função main()
:
int main(){
Matriz m;
int mat[LINHAS][COLUNAS] = {
{ 0, 185, 119, 152, 133 },
{ 185, 0, 121, 150, 200 },
{ 199, 121, 0, 174, 120 },
{ 152, 150, 174, 0, 199 },
{ 133, 200, 120, 199, 0 }
};
cria_matriz( &m, mat );
imprimir_matriz( &m );
/* Crie uma função limpar_matriz( &m ) */
return 0;
}
Saída:
Matriz
0 185 119 152 133
185 0 121 150 200
199 121 0 174 120
152 150 174 0 199
133 200 120 199 0
Este código pode ser encontrado em uma versão um tiquinho melhor e compilável com -pedantic -ansi -Wall -Wextra -Werror
aqui. Recomendo olhar só depois de fazer sua própria função de limpeza de memória. Também recomendo procurar sobre o Valgrind (tutorial aqui) para auxiliar em vazamento de memória.