Foro de elhacker.net

Hola a todo el mundo, he credo el siguiente código en el cual se le pide al usuario cuantos números aleatorios quiere y a partir de estos se les de dos archivos donde el primero los ordene de menor a mayor y el segundo de mayor a menor, después se saque la frecuencia de dichos números, el problema es que al calcular la frecuencia me esta dando puros ceros.

//PROGRAMA CRISTINA!!//
 
#include<stdio.h>
#include<stdlib.h>
#include<time.h>
 
int i,f,g,x,p;//Estas variables son usadas en los dos FILE//
int j;//Estas variables son usadas en el apartado de la frecuencia relativa//
 
int main ()
{
	printf ("%4s\n%4s\n%4s\n","**********","CRISTINA","**********");
 
	printf("¿Cuantos numeros quieres que te genere?:   ");
	scanf("%d",&x);
	printf("¿Cuantos numeros quieres que te genere?:   ");
	scanf("%d",&p);
 
	int a[x],b[p], luca[x],r,s,y;//Las variables locales que seran usadas tanto para ordenar los datos como para sacar la frecuencia//
 
	srand( time(NULL) );
	     for (i=0;i<=x-1;i=i+1)
		 {
	     	a[i]=rand();
 
		 }
 
	  FILE *m;
         m= fopen ("mayor.dat","w");
 
		 for(f=1;f<=x-1;f++) /*En este for, es importante que no se vuelva ha establecer la variable i 
		                       pues al momento de hacer la cuenta lo confunde y no hace el ordenamiento.*/
         {
 
			 for (i=0;i<=x-2;i++)
			 {
 
 
 
                 if (a[i]<a[i+1])
			    {
                     g=a[i];
                     a[i]=a[i+1];
                     a[i+1]=g;
                }
			 }
         }
 
		 for (i=0;i<=x-1;i++)
            {
			 fprintf(m,"\n %4d\n",a[i]);
             fprintf(m,"\n");
             b[i]=a[i];
             }  
 
	 fclose (m); //El primer archivo ya ha sido creado, este tiene un ordenamiento de mayor a menor//               
 
	 //Para crear el segundo ordenamiento se puede usar la misma estructura solo se cambia la desigualdad y el nombre interno der archivo// 	  
	 FILE *n;
         n= fopen ("menor.dat","w");
 
		 for(f=1;f<=x-1;f++) 
         {
 
			 for (i=0;i<=x-2;i++)
			 {
 
 
 
                 if (a[i]>a[i+1])
			    {
                     g=a[i];
                     a[i]=a[i+1];
                     a[i+1]=g;
                }
			 }
         }
 
		 for (i=0;i<=x-1;i++)
            {
			 fprintf(n,"\n %4d\n",a[i]);
             fprintf(n,"\n");
             }  
 
	 fclose (n); 	     
FILE *t;
t=fopen("frecuencia.dat","w");
for (i=0;i<x;i++){
	for(j=1;j<p;j++){
		if(a[i]==j){
 
		y=a[i]=a[i]+1;
       }
	}
 
}
for (i=0;i<x;i++){
fprintf(t," %7d",y);
 
}
 
fclose(t);
 
return 0;
}  

---

· Los códigos deben ir en etiquetas GeSHi
· No se debe escribir en mayúsculas
>aquí las reglas del foro (http://foro.elhacker.net/reglas.htm)
-Engel Lex

A ver Cristina Gusigusa...

De entrada, no entiendo bien por qué pides dos veces la cantidad de números que se deben generar al azar...
   
Luego parece que el array 'a', tendrá el tamaño que el usuario da a 'x', pero el array 'b' tendrá el tamaño dado a 'p'... 'x' y 'p', va a ser que no sean el mismo valor (salvo que tu misma seas quien lo introduce o rara casualidad...
Entonces si los arrays no van a ser del mismo tamaño y especialmente si no hay garantías de que el array 'b' sea mayor que el array 'a', acabará dando un error en la línea 53
Que además, no veo que se vuelva a usar el array 'b' en parte alguna... Luego elimina el array 'b', elimina la petición de 'p', y si no quieres eliminar 'p, al menos haz: p == x

La indentación aparece confusa, y no queda claro (de un simple vistazo), que queda dentro de qué... sin un esfuerzo extra revisando llaves (que obvio hacer)...
El ordenamiento inverso es redunndante, una vez hecho de mayor a menor o de menor a mayor, el inverso, basta con recorrer el array desde el final hacia atrás, hasta llegar a 0.

Así el código queda resumido así para menor.dat
De hecho si se abren dos canales de escritura, pueden guardarse a fichero los dos al mismo tiempo (en el mismo bucle)


Aunque para ordenar usas un algoritmo de burbuja... que son muy lentos, pero para ejemplos son perfectamente válidos...

Finalmente llegamos a la parte de contar las frecuencias...
De entrada, date cuenta que si el array ya está ordenado es superfluo recorrer todo el array, las veces que un valor aparezca, estarán contiguos... ...porque el array está ordenado. Como no sabemos cuantos valores distintos hay, usaremos un bucle indefinido (Mientras... condición...repetir)


Finalmente una aclaración... aquí se están contando el número de apariciones de cada valor, no la frecuencia. La frecuencia en realidad sería dividir el tamaño del array entre la cantidad de apariciones de cada valor... yendo más lejor se podría luego  dar en porcentaje. (con luego, no quiero decir en otro bucle si no modificarlo en otro momento).

Ojo, que como para Rand, no pones límites... si alguien pide pongamos 20 números es fácil que no se repita ninguna vez... acota Rand a un rango de valores, por ejemplo entre 100 y 110 y que la cantidad de números a generar sea (obligatorio) por lo menos el doble o el triple de lo que es el rango, para que haya 'repes'...
...es decir si 'x' (que introduce el usuario) es menor que el rango entre MinVal y MaxVal, repite la petición... MinVal y MaxVal serían los límites menor y mayor entre los que elegir al azar...

for (i=0;i<x;i++){
    fprintf(t," %7d",y);
}

Aquí estás escribiendo "x" veces la misma variable "y".