Autor
|
Tema: C:Calcular montañas en los datos almacenados en una matriz (Leído 7,683 veces)
|
Xandrete
Desconectado
Mensajes: 210
|
Supongo que en todo caso sería fscanf(f1 ,"%f", matriz [i ][j ]);
La función Muestra_montanas la tienes plagada de errores de este tipo. Recuerda que para acceder a un elemento de una matriz hacen falta dos índices. Saludos
|
|
« Última modificación: 18 Enero 2012, 23:05 pm por Xandrete »
|
En línea
|
|
|
|
deifk
Desconectado
Mensajes: 13
|
el caso es que al pegar el codigo todos los "i" de la matriz se han borrado ponen como que estan y luego no salen, asi que nose que podra ser
|
|
|
En línea
|
|
|
|
Xandrete
Desconectado
Mensajes: 210
|
Ah, claro. Eso es porque no utilizas las etiquetas para adjuntar código ([ code=c ] [ /code ]). La etiqueta [ i ] [ /i ] sirve para escribir en cursiva. Ya que estamos, a partir de ahora usa [ code ] para pegar código. Será más fácil de leer. Si puedes, hazlo ahora (no me gustaría tener que intuir cosas allá donde no las hay).
|
|
|
En línea
|
|
|
|
deifk
Desconectado
Mensajes: 13
|
Disculpas, he posteado escasas veces y no era consciente de estas herramientas, a ver si asi alguien me puede decir el porque del "error de segmentacion" Gracias! #include <stdio.h> #include <string.h> #include <stdlib.h> #define M 5 #define N 5 void Lee_Fichero(char [], float[][M]); void Muestra_montanas(float [][M]); main() { float m1[N][M]; Lee_Fichero("cartografia.dat",m1); Muestra_montanas(m1); } void Lee_Fichero(char nombre[],float matriz[N][M]) { FILE *f1; int i,j; if(f1!=NULL) { for(i=0; i<N;i++) { for(j=0;j<M;j++) { fscanf(f1 ,"%f", matriz [i ][j ]); } } } else { printf("Error en la apertura del fichero %s\n",nombre ); } } void Muestra_montanas(float m[N][M]) { float max_altura; int i,j; max_altura=m[i][j]; for(i=1;i<N-1;i++) { for(j=1; j<M-1;j++) { if(m[i][j]>m[i][j-1] && m[i][j]>m[i][j+1] &&m[i][j]>m[i-1][j-1] && m[i][j]>m[i+1][j-1] && m[i][j]>m[i-1][j] && m[i][j]>m[i+1][j] && m[i][j]>m[i-1][j+1] && m[i][j]>m[i+1][j+1 { max_altura=m[i][j]; printf("Las coordenadas de la montaña son %d y %d y su altura %f", i , j , max_altura ); } } }
|
|
|
En línea
|
|
|
|
Xandrete
Desconectado
Mensajes: 210
|
Tranqui, no pasa nada. Ya había visto el error antes de que postearas. Mira: float max_altura; int i,j; max_altura=m[i][j];
Ni i ni j están inicializadas, por tanto el valor que tienen es impredecible (o, dicho de otra manera, es basura). Por tanto, seguramente la violación de segmento se produce cuando accedes a m[ i][j] sin haberles dado un valor previo a j,i. Joder, he tenido que reeditar el post 100 veces porque esta vez fui yo quien se hizo la picha un lío con lo de las cursivas.
|
|
« Última modificación: 18 Enero 2012, 23:29 pm por Xandrete »
|
En línea
|
|
|
|
deifk
Desconectado
Mensajes: 13
|
max_altura=m[i][j];
esta sentencia creo que me sobraba, nose porque estaba ahí aun asi sigo teniendo error de segmentacion, ¿que es lo que que dices que es necesario inicializar? no se supone que i y j adquieren el valor -1 al ejecutarse el for?
|
|
|
En línea
|
|
|
|
Xandrete
Desconectado
Mensajes: 210
|
Oh, vaya. Antes copié la misma línea de código y no me di cuenta. Aquí tienes un error: fscanf(f1 ,"%f", matriz [i ][j ]);
Se te pasó poner el & antes de matriz[ i ][j]. El argumento de scanf tiene que ser el puntero al dato, no el dato. Y otra cosa. Es recomendable que el main devuelva un entero. Deberías poner int main() en la cabecera y no main() a secas. Y al final del main, pon return 0; Te aconsejo que compiles con -Wall y -Werror. Deberías considerar todos los warning como errores, y corregirlos convenientemente (y tu código da un par de warnings si los compilas con -Wall) max_altura=m[i][j];
esta sentencia creo que me sobraba, nose porque estaba ahí aun asi sigo teniendo error de segmentacion, ¿que es lo que que dices que es necesario inicializar? no se supone que i y j adquieren el valor -1 al ejecutarse el for? Me refería a que si querías acceder a m[ i ][j] antes del for, tenías que darle un valor concreto a i, j (inicializarlos). Saludos.
|
|
« Última modificación: 18 Enero 2012, 23:58 pm por Xandrete »
|
En línea
|
|
|
|
deifk
Desconectado
Mensajes: 13
|
Cierto,todo solucionado, muchas gracias de verdad, me han sido de gran ayuda, habia olvidado cosas como esas, llevaba unas semanas sin trabajar con matrices y se me olvido el ampersant.
Un saludo!
|
|
|
En línea
|
|
|
|
Xandrete
Desconectado
Mensajes: 210
|
¡De nada! Una última recomendación. Cuando escribas código, intenta que quede lo más legible posible. Conviene evitar sentencias if con megacondiciones (poniendo la evaluación de la expresión en una función aparte, por ejemplo). O, si aun así prefieres poner una megacondición, al menos distribúyela a lo largo de varias líneas, así: if( m[i][j] > m[i][j-1] && m[i][j] > m[i][j+1] && m[i][j] > m[i-1][j-1] && m[i][j] > m[i-1][j] && m[i][j] > m[i-1][j+1] && m[i][j] > m[i+1][j-1] && m[i][j] > m[i+1][j] && m[i][j] > m[i+1][j+1] )
|
|
|
En línea
|
|
|
|
|
Mensajes similares |
|
Asunto |
Iniciado por |
Respuestas |
Vistas |
Último mensaje |
|
|
[C] Calcular determinante de una matriz de orden 'n'
Programación C/C++
|
Kasswed
|
9
|
39,711
|
24 Enero 2013, 15:37 pm
por flony
|
|
|
No se imprimen los datos almacenados en un archivo
Programación C/C++
|
BJM
|
1
|
1,974
|
23 Noviembre 2012, 04:03 am
por durasno
|
|
|
Calcular determinante de una matriz NxN
Programación C/C++
|
amchacon
|
1
|
15,381
|
13 Febrero 2013, 20:35 pm
por leosansan
|
|
|
Calcular máximo de matriz, error
Programación C/C++
|
Puntoinfinito
|
7
|
5,004
|
18 Julio 2013, 11:58 am
por Puntoinfinito
|
|
|
[Python] - calcular el max y el min de una matriz de 4x4
Scripting
|
jhonnymn3
|
1
|
5,715
|
18 Noviembre 2014, 15:18 pm
por daryo
|
|