Insertionsort compila pero no funca

[extremoo]

Explico: Leo el tamaño del arreglo desde un archivo con numeros aleatorios
ejem.txt
4
2
1
4
2
el tam del arr es 4 y genera 4 nº aleatorios leo el archivo y creo uno nuevo e aqui el cod.

Código C:

#include<stdio.h>
#include<stdlib.h>
#include<time.h>
#include<string.h>
 
void insertion_sort(int x[],int length)
{
  int key,i, j;
  for(j=1 ;j<length; j++)
  {
     key=x[j];
     i=j-1;
     while(x[i]>key && i>=0)
     {
         x[i]=x[i+1];
         i--;
     }
         x[i+1]=key;
  }
}
 
int main()
{
 
  float total,inicio, final;
  inicio=clock();
  FILE *ARCH, *ARCHI;
 
    int N, aux, i;
    ARCH = fopen("aleatorio.txt","r");
  
    fscanf(ARCH,"%i", &N);
    int d[N];
 
    ARCHI = fopen("insertionsort.txt","w");
    i=0;
    for(i=0;i<N;i++)
      {
      fscanf(ARCHI,"%i", &d[i]);
      }
   insertion_sort(d,N);
 
  for(i=0;i<N;i++)
  {
 
   fprintf(ARCHI,"%i \n",d[i]);
  }
 
 
  final=clock();
  total=(final-inicio)/(double) CLOCKS_PER_SEC;
  printf("%f\n",total);
 
  fclose(ARCH);  
  fclose(ARCHI);
return 0;
}
 

mi archivo de salida arroja esto:

-1217354048
-1217354048
-1217354048
-1217354048

que puede fallar? saludos

[gvite666]

pues mira en este pedazo de codigo esta tue "error"

for(i=0;i<N;i++)

{

fscanf(ARCHI,"%i", &d[i]);

}
¿por que? porque estas creando el archivo "insertionsort.txt" ¿y que pasa? q el archivo esta vacio, entonces al momento de intentar leer una dato fscanf te retorna un -1 que indica que no se pudo realizar la lectura y deja el valor que tenga d[i] en este caso como no las has utilizado te deja a lo que se le llama basura que te da por default "-1217354048 ".

otra cosa importante, tu compilador no deberia dejarte correr la instruccion "int d[N];" ya que esto es una, por llamarlo asi, "falta de programacion" ya que no es estandar y podria fallar tu aplicacion al momento de querer correr tus programas en otro equipo, por decir, los buenos programas y compiladores se les debe decir el tamaño del arreglo al definirlo.
Espero te sirva.
Saludos

[extremoo]

me parece que no es eso ya que en esa parte lee el N total del archivo que es el primer numero que aparece en el archivo y lo guarda. por ejemplo el archivo generador antes de crearlo
me pide N yo ingreso 5 y luego guarda en un aleatorio.txt lo siguiente

5 //este es el total que toma en fscanf
3 //de este numero hacia abajo son los 5 aleatorios que debe recorrer
4
2
5
1

=/

[sam90]

Fiajte que lees de un archivo que esta abierto para escribir.... Proba imprimiendo lo que lees y te vas a dar cuenta.

[gvite666]

por eso, mira te lo explico con mas detalle
int main(){
float total,inicio, final;
inicio=clock();
FILE *ARCH, *ARCHI;
int N, aux, i;
ARCH = fopen("aleatorio.txt","r"); //abres el archivo "aleatorio.txt", que supongo ya existe y que tiene tu "N".
fscanf(ARCH,"%i", &N); int d[N]; //lees tu "N" hasta aqui na hay porque tener problema
ARCHI = fopen("insertionsort.txt","w"); //Creas un nuevo archivo "insertionsort.txt" el cual esta vacio, aqui puedes poner "w+" esto te permite leer y escribir
i=0;
for(i=0;i<N;i++)
{
fscanf(ARCHI,"%i", &d[i]); //aqui es donde esta el error ya que tu archivo esta vacio y al momento de intentar leer algo, pues no se puede. y te deja la variable con el valor por default
}
insertion_sort(d,N); //todo lo demas, ya no tiene que ver con este problema.
for(i=0;i<N;i++)
{
fprintf(ARCHI,"%i \n",d[i]);
}
final=clock();
total=(final-inicio)/(double) CLOCKS_PER_SEC;
printf("%f\n",total);
fclose(ARCH);
fclose(ARCHI);
return 0;
}