necesito ayuda para ordenar en c

Iniciado por marrison, 22 Diciembre 2012, 14:28 PM

0 Miembros y 1 Visitante están viendo este tema.

Imprimir
Ir Abajo Páginas1
Acciones del Usuario

marrison

22 Diciembre 2012, 14:28 PM
Hola buenas, estoy aprendiendo a programar en c, y estoy haciendo una base de datos, que coje 3 datos de un archivo, nombre, nº de registro y nº de serie.

Bueno el problema es que al querer ordenarlo, usando el metodo de burbuja, que es el unico que se, solo me ordena uno de los valores, me explico, yo pido los datos, el usuario los teclea y se guardan en 3 variables distintas, y al ordenarlo solo me ordena una de ella, pero las otras se quedan igual, y lo que quiero es que a partir de cada una de las variables se me ordenen todas, tambien las otras dos, bueno no se si eso es posible, he pensado que a lo mejor habria que crear otra variable con todos los datos de cada archivo, pero no se muy bien como programarlo...

Aqui os dejo el codigo del programa para ver si me podeis ayudar...

Código (c) [Seleccionar]
#include <stdio.h>
#include <string.h>
struct{
char nombreFich[41];
unsigned long numeroregistro;
unsigned long numeroserie;
} fichas[1000000];
int numeroFichas=0;
int i;
int opcion;
char textoTemporal[40];
unsigned long numeroTemporal;

main()
{
do {
printf("Escoja una opción: (Teclee el numero de la opcion)\n");
printf("1.- Añadir datos de un nuevo fichero\n");
printf("2.- Mostrar todos los ficheros\n");
printf("3.- Mostrar ficheros a partir de un cierto numero de registro\n");
printf("4.- Mostrar ficheros a partir de un cierto numero de serie\n");
printf("5.- Ver datos de un fichero\n");
printf("6.- Salir\n");
gets (textoTemporal);
sscanf(textoTemporal, "%d", &opcion);
switch(opcion){
case 1:
if (numeroFichas < 1000000) {
printf("Introduce el nombre del fichero: ");
gets(fichas[numeroFichas].nombreFich);
printf("Introduce el numero de registro(si el valor no es un numero no aparecera o aparecera 0): ");
gets(textoTemporal);
sscanf(textoTemporal,"%ld",&fichas[numeroFichas].numeroregistro);
printf("Introduce el numero de serie(si el valor no es un numero no aparecera o aparecera 0): ");
gets(textoTemporal);
sscanf(textoTemporal,"%ld",&fichas[numeroFichas].numeroserie);
numeroFichas++;
} else
printf("Máximo de fichas alcanzado (1000000)!\n");
break;
case 2:
for (i=0; i<numeroFichas; i++)
printf("Nombre: %s; Nº Registro: %ld; Nº Serie: \n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
break;
case 3:
printf("¿A partir de que nº de registro quieres que se muestre?");
gets(textoTemporal);
sscanf(textoTemporal, "%ld", &numeroTemporal);
for (i=0; i<numeroFichas; i++)
if (fichas[i].numeroregistro >= numeroTemporal)
printf("Nombre: %s; Nº Registro: %ld; Nº Serie: %1d; \n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
else printf("No hay ficheros o no hay ficheros con un nº de registro mayor que el introducido.\n");
break;
case 4:
printf("¿A partir de que nº de serie quieres que se muestre?");
gets(textoTemporal);
sscanf(textoTemporal, "%ld", &numeroTemporal);
for (i=0; i<numeroFichas; i++)
if (fichas[i].numeroserie >= numeroTemporal)
printf("Nombre: %s; Nº Registro: %ld; Nº Serie: %1d; \n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
else printf("No hay ficheros o no hay ficheros con un nº de serie mayor que el introducido.\n");
break;
case 5:
printf("¿De qué fichero quiere ver todos los datos?");
gets(textoTemporal);
for (i=0; i<numeroFichas; i++)
if (strcmp(fichas[i].nombreFich, textoTemporal) == 0)
printf("Nombre: %s; Nº registro: %ld; Nº serie: %1d \n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
for (i=0; i<numeroFichas; i++)
if (strcmp(fichas[i].nombreFich, textoTemporal) != 0)
if (strstr (fichas[i].nombreFich, textoTemporal) != NULL)
printf("Nombre: %s; Nº registro: %ld; Nº serie: %1d (Busqueda a partir de la palabra tecleada, archivos coincidentes)\n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
break;
case 6:
printf("Fin del programa\n");
break;
default:
printf("Opción desconocida!\n");
break;
}
} while (opcion != 6);
}


Y aqui os dejo la forma de la que lo ordeno, para ver si se puede hacer algo:

Código (c) [Seleccionar]
for(i=0; i<(numeroFichas-1); i++) {
for (j=i+1; j<numeroFichas; j++) {
if(fichas[j].numeroregistro<fichas[i].numeroregistro) {
temp = fichas[j].numeroregistro;
fichas[j].numeroregistro = fichas[i].numeroregistro;
fichas[i].numeroregistro = temp;
}
if(fichas[j].numeroregistro<fichas[i].numeroregistro) {
temp = fichas[j].numeroserie;
fichas[j].numeroserie = fichas[i].numeroserie;
fichas[i].numeroserie = temp;
}
}
}
for(i=0; i<numeroFichas ; i++) {
printf("Nombre: %s; Nº registro: %ld; Nº serie: %1d\n", fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
}

Muchas gracias de antemano, Feliz Navidad y Feliz año  :)
"Es genial trabajar con ordenadores. No discuten, lo recuerdan todo y no se beben tu cerveza" (Paul Leary)

"Controlar la complejidad es la esencia de la programación" (Brian Kernigan)

"Primero resuelve el problema. Entonces, escribe el código" (John Johnson)

"640K deberían ser suficientes para todo el mundo" (Bill Gates, 1981)

rir3760

#1
23 Diciembre 2012, 02:34 AM
Cuando tengas problemas con un programa por favor publicalo completo, no en partes.

Para empezar el algoritmo que utilizas no es Burbuja, es Selección.

En cuanto a este un problema es que solo intercambias el campo "numeroregistro" cuando deberías intercambiar los elementos (estructuras). Ese fragmento con los cambios:
Código (c) [Seleccionar]
struct {
   char nombreFich[41];
   unsigned long numeroregistro;
   unsigned long numeroserie;
} fichas[10], temp; /* temp debe ser del mismo tipo */

/* ... */

for (i = 0; i < numeroFichas - 1; i++)
   for (j = i + 1; j < numeroFichas; j++) {
      if (fichas[j].numeroregistro < fichas[i].numeroregistro){
         temp = fichas[j];
         fichas[j] = fichas[i];
         fichas[i] = temp;
      }
Así debe funcionar correctamente siempre y cuando no haya otros errores en la parte del programa que no publicaste (de ahí la recomendación de publicar el código fuente completo).

Un saludo
C retains the basic philosophy that programmers know what they are doing; it only requires that they state their intentions explicitly.
--
Kernighan & Ritchie, The C programming language

marrison

#2
23 Diciembre 2012, 12:23 PM Ultima modificación: 23 Diciembre 2012, 12:46 PM por marrison
Cita de: rir3760 en 23 Diciembre 2012, 02:34 AM
Cuando tengas problemas con un programa por favor publicalo completo, no en partes.

Para empezar el algoritmo que utilizas no es Burbuja, es Selección.

En cuanto a este un problema es que solo intercambias el campo "numeroregistro" cuando deberías intercambiar los elementos (estructuras). Ese fragmento con los cambios:
Código (c) [Seleccionar]
struct {
  char nombreFich[41];
  unsigned long numeroregistro;
  unsigned long numeroserie;
} fichas[10], temp; /* temp debe ser del mismo tipo */

/* ... */

for (i = 0; i < numeroFichas - 1; i++)
  for (j = i + 1; j < numeroFichas; j++) {
     if (fichas[j].numeroregistro < fichas[i].numeroregistro){
        temp = fichas[j];
        fichas[j] = fichas[i];
        fichas[i] = temp;
     }
Así debe funcionar correctamente siempre y cuando no haya otros errores en la parte del programa que no publicaste (de ahí la recomendación de publicar el código fuente completo).

Un saludo

Lo primero gracias por contestar amigo,

Decirte que el codigo del programa esta entero, es la primera parte, lo segundo es lo que añado pero no va, pero el codigo es el mismo, con una opcion mas y dos variables, pero no cambia nada.

He probado el codigo e ir va, me aparecen ordenadas, todas, menos la primera que no apareec, es decir, si pongo 10 fichas, la primera que escriba no aparece, por que puede ser?

Aqui os dejo el codigo con la parte modificada tambien, es decir completo.
Código (c) [Seleccionar]
#include <stdio.h>
#include <string.h>

struct{
char nombreFich[41];
unsigned long numeroregistro;
unsigned long numeroserie;
} fichas[1000000], temp;
int numeroFichas=0;
int i;
int opcion;
char textoTemporal[40];
unsigned long numeroTemporal;
int j;


main()
{
do {
printf("Escoja una opción: (Teclee el numero de la opcion)\n");
printf("1.- Añadir datos de un nuevo fichero\n");
printf("2.- Mostrar los nombres de todos los ficheros\n");
printf("3.- Mostrar ficheros a partir de un cierto numero de registro\n");
printf("4.- Mostrar ficheros a partir de un cierto numero de serie\n");
printf("5.- Ordenar por numero de registro\n");
printf("6.- Ver datos de un fichero\n");
printf("7.- Salir\n");
gets (textoTemporal);
sscanf(textoTemporal, "%d", &opcion);
switch(opcion){
case 1:
if (numeroFichas < 1000000) {
printf("Introduce el nombre del fichero: ");
gets(fichas[numeroFichas].nombreFich);
printf("Introduce el numero de registro: ");
gets(textoTemporal);
sscanf(textoTemporal,"%ld",&fichas[numeroFichas].numeroregistro);
printf("Introduce el numero de serie: ");
gets(textoTemporal);
sscanf(textoTemporal,"%ld",&fichas[numeroFichas].numeroserie);
numeroFichas++;
} else
printf("Máximo de fichas alcanzado (1000000)!\n");
break;
case 2:
for (i=0; i<numeroFichas; i++)
printf("Nombre: %s; Nº Registro: %ld; Nº Registro: \n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
break;
case 3:
printf("¿A partir de que nº de registro quieres que se muestre?");
gets(textoTemporal);
sscanf(textoTemporal, "%ld", &numeroTemporal);
for (i=0; i<numeroFichas; i++)
if (fichas[i].numeroregistro >= numeroTemporal)
printf("Nombre: %s; Nº Registro: %ld; Nº Serie: %1d; \n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
break;
case 4:
printf("¿A partir de que nº de serir quieres que se muestre?");
gets(textoTemporal);
sscanf(textoTemporal, "%ld", &numeroTemporal);
for (i=0; i<numeroFichas; i++)
if (fichas[i].numeroserie >= numeroTemporal)
printf("Nombre: %s; Nº Registro: %ld; Nº Serie: %1d; \n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
break;
case 5:
for(i=0; i<(numeroFichas-1); i++) {
  for(j=i+1; j<numeroFichas; j++) {
     if (fichas[j].numeroregistro<fichas[i].numeroregistro) {
        temp = fichas[j];
        fichas[j] = fichas[i];
        fichas[i] = temp;
   
}
}
printf("Nombre: %s; Nº registro: %ld; Nº serie: %1d\n", fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
}
break;
case 6:
printf("¿De qué fichero quiere ver todos los datos?");
gets(textoTemporal);
for (i=0; i<numeroFichas; i++)
if (strcmp(fichas[i].nombreFich, textoTemporal) == 0)
printf("Nombre: %s; Nº registro: %ld; Nº serie: %1d \n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
for (i=0; i<numeroFichas; i++)
if (strcmp(fichas[i].nombreFich, textoTemporal) != 0)
if (strstr (fichas[i].nombreFich, textoTemporal) != NULL)
printf("Nombre: %s; Nº registro: %ld; Nº serie: %1d (Busqueda a partir de la palabra tecleada, archivos coincidentes)\n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
break;
case 7:
printf("Fin del programa\n");
break;
default:
printf("Opción desconocida!\n");
break;
}
}
while (opcion != 7);
}
"Es genial trabajar con ordenadores. No discuten, lo recuerdan todo y no se beben tu cerveza" (Paul Leary)

"Controlar la complejidad es la esencia de la programación" (Brian Kernigan)

"Primero resuelve el problema. Entonces, escribe el código" (John Johnson)

"640K deberían ser suficientes para todo el mundo" (Bill Gates, 1981)

marrison

#3
23 Diciembre 2012, 13:08 PM
Bueno ya lo he arreglado :D

El problema estaba aqui:
Código (c) [Seleccionar]
case 5:
for(i=0; i<(numeroFichas-1); i++) {
  for(j=i+1; j<numeroFichas; j++) {
     if (fichas[j].numeroregistro<fichas[i].numeroregistro) {
        temp = fichas[j];
        fichas[j] = fichas[i];
        fichas[i] = temp;
   
}
}
printf("Nombre: %s; Nº registro: %ld; Nº serie: %1d\n", fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
}
break;

En el primer for he suprimido el -1 de numeroFichas-1 de manera que quedaria asi:

Código (c) [Seleccionar]
case 5:
for(i=0; i<(numeroFichas); i++) {
  for(j=i+1; j<numeroFichas; j++) {
     if (fichas[j].numeroregistro<fichas[i].numeroregistro) {
        temp = fichas[j];
        fichas[j] = fichas[i];
        fichas[i] = temp;
   
}
}
printf("Nombre: %s; Nº registro: %ld; Nº serie: %1d\n", fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
}
break;

Y el codigo completo:

Código (c) [Seleccionar]
#include <stdio.h>
#include <string.h>

struct{
char nombreFich[41];
unsigned long numeroregistro;
unsigned long numeroserie;
} fichas[1000000], temp;
int numeroFichas=0;
int i;
int opcion;
char textoTemporal[40];
unsigned long numeroTemporal;
int j;


main()
{
do {
printf("Escoja una opción: (Teclee el numero de la opcion)\n");
printf("1.- Añadir datos de un nuevo fichero\n");
printf("2.- Mostrar los nombres de todos los ficheros\n");
printf("3.- Mostrar ficheros a partir de un cierto numero de registro\n");
printf("4.- Mostrar ficheros a partir de un cierto numero de serie\n");
printf("5.- Ordenar por numero de registro\n");
printf("6.- Ver datos de un fichero\n");
printf("7.- Salir\n");
gets (textoTemporal);
sscanf(textoTemporal, "%d", &opcion);
switch(opcion){
case 1:
if (numeroFichas < 1000000) {
printf("Introduce el nombre del fichero: ");
gets(fichas[numeroFichas].nombreFich);
printf("Introduce el numero de registro: ");
gets(textoTemporal);
sscanf(textoTemporal,"%ld",&fichas[numeroFichas].numeroregistro);
printf("Introduce el numero de serie: ");
gets(textoTemporal);
sscanf(textoTemporal,"%ld",&fichas[numeroFichas].numeroserie);
numeroFichas++;
} else
printf("Máximo de fichas alcanzado (1000000)!\n");
break;
case 2:
for (i=0; i<numeroFichas; i++)
printf("Nombre: %s; Nº Registro: %ld; Nº Registro: \n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
break;
case 3:
printf("¿A partir de que nº de registro quieres que se muestre?");
gets(textoTemporal);
sscanf(textoTemporal, "%ld", &numeroTemporal);
for (i=0; i<numeroFichas; i++)
if (fichas[i].numeroregistro >= numeroTemporal)
printf("Nombre: %s; Nº Registro: %ld; Nº Serie: %1d; \n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
break;
case 4:
printf("¿A partir de que nº de serir quieres que se muestre?");
gets(textoTemporal);
sscanf(textoTemporal, "%ld", &numeroTemporal);
for (i=0; i<numeroFichas; i++)
if (fichas[i].numeroserie >= numeroTemporal)
printf("Nombre: %s; Nº Registro: %ld; Nº Serie: %1d; \n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
break;
case 5:
for(i=0; i<(numeroFichas); i++) {
  for(j=i+1; j<numeroFichas; j++) {
     if (fichas[j].numeroregistro<fichas[i].numeroregistro) {
        temp = fichas[j];
        fichas[j] = fichas[i];
        fichas[i] = temp;
   
}
}
printf("Nombre: %s; Nº registro: %ld; Nº serie: %1d\n", fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
}
break;
case 6:
printf("¿De qué fichero quiere ver todos los datos?");
gets(textoTemporal);
for (i=0; i<numeroFichas; i++)
if (strcmp(fichas[i].nombreFich, textoTemporal) == 0)
printf("Nombre: %s; Nº registro: %ld; Nº serie: %1d \n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
for (i=0; i<numeroFichas; i++)
if (strcmp(fichas[i].nombreFich, textoTemporal) != 0)
if (strstr (fichas[i].nombreFich, textoTemporal) != NULL)
printf("Nombre: %s; Nº registro: %ld; Nº serie: %1d (Busqueda a partir de la palabra tecleada, archivos coincidentes)\n",
fichas[i].nombreFich, fichas[i].numeroregistro, fichas[i].numeroserie);
break;
case 7:
printf("Fin del programa\n");
break;
default:
printf("Opción desconocida!\n");
break;
}
}
while (opcion != 7);
}


Ahora voy a añadir una opcion para ordenarlo por nº de serie (la otra variable) e intentare ordenarlo por orden alfabetico, aunque nose porque me da que por orden alfabetico me va a costar mas  ;D

Muchas gracias por tu ayuda, Un saludo, feliz navidad y Feliz Año :)
"Es genial trabajar con ordenadores. No discuten, lo recuerdan todo y no se beben tu cerveza" (Paul Leary)

"Controlar la complejidad es la esencia de la programación" (Brian Kernigan)

"Primero resuelve el problema. Entonces, escribe el código" (John Johnson)

"640K deberían ser suficientes para todo el mundo" (Bill Gates, 1981)
Imprimir
Ir Arriba Páginas1
Acciones del Usuario