C:Calcular montañas en los datos almacenados en una matriz

[Xandrete]

Código (c) [Seleccionar] Expandir

fscanf(f1,"%f", matriz[j]);

Supongo que en todo caso sería

Código (c) [Seleccionar] Expandir

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

[deifk]

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

[Xandrete]

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).

[deifk]

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!

Código (c) [Seleccionar] Expandir


#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;
 
 f1=fopen(nombre,"r");
 
 if(f1!=NULL)
 {
   for(i=0; i<N;i++)
   {
    for(j=0;j<M;j++)
     {

     fscanf(f1,"%f", matriz[i][j]);
     }
   }
   fclose(f1);
 }  
 else
 {
   printf("Error en la apertura del fichero %s\n",nombre);
   exit(-1);
 }
}
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);
       }
   }
 }



[Xandrete]

Tranqui, no pasa nada.

Ya había visto el error antes de que postearas. Mira:

Código (c) [Seleccionar] Expandir

 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.

[deifk]

Código (c) [Seleccionar] Expandir

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?

[Xandrete]

Oh, vaya. Antes copié la misma línea de código y no me di cuenta.

Aquí tienes un error:

Código (c) [Seleccionar] Expandir

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)

Código (c) [Seleccionar] Expandir

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.

[deifk]

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!

[Xandrete]

¡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í:

Código (c) [Seleccionar] Expandir

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] )