Varias cosas:
1. int a???? que co** es a????, vale, si inviertes unos minutillos en analizar el código descubres que a es una variable global que la usas para almacenar la cantidad de registros... ¿es necesario llamarla a?? ¿no había un nombre un poco más apropiado? no se algo quizás del tipo 'numeroDeRegistros'. No se, yo creo que si alguien se encuentra con una variable que se llama 'numeroDeRegistros' se hace una idea bastante aproximada del uso que tiene esa variable y puede llegar a entender el código con más rapidez.
2. No uses variables globales. No cuesta nada poner un argumento más en las funciones y con eso consigues que seguir el programa sea más sencillo.
3. Por si no te has dado cuenta, no has inicializado tu querida 'a'... si ejecutas el programa y no pasas por la opción 1 el resultado no te va a gustar.
4. En cargar_arch... si falla la carga del archivo TAMPOCO actualizas el valor de 'a'...
5. ¿Por qué el buffer de nombres es de 25 y el de apellidos de 30?? ¿por qué 'aux' es de 25 si también va a almacenar apellidos? Lo mejor para evitar este tipo de inconsistencias es usar macros para definir valore fijos:
#define MAX_REGS 1000
#define LONG_REG 30
char nombres[MAX_REGS][LONG_REG],apellidos[MAX_REGS][LONG_REG];
// ...
char aux[LONG_REG];
// ...
6. fscanf y scanf pueden provocar desbordamiento de buffer si intentas cargar una cadena más grande que el buffer... en tu caso 25 o 30 caracteres... deberías hechar un vistazo a
|Lo que no hay que hacer en C/C++. Nivel basico|.
7. Da la sensación de que en la opción 2 intentas ordenar el vector antes de mostrarlo... No hace falta comprobar un registro contra sí mismo para hacerlo.
for(int con=0;con<a;con++)
{
for(int con2=con+1;con2<a;con2++)
{
if(strcmp(apellidos
[con
],apellidos
[con2
])<0) {
strcpy( aux
, apellidos
[con
] ); strcpy( apellidos
[con
], apellidos
[con2
] ); strcpy( apellidos
[con2
], aux
); strcpy( nombres
[con
], nombres
[con2
] ); }
}
}
Si además metes ese código repetitivo en una función, el código se simplifica:
void SwapStrings( char* string1, char* string2 )
{
char aux[30];
}
// ...
for(int con=0;con<a;con++)
{
for(int con2=con+1;con2<a;con2++)
{
if(strcmp(apellidos
[con
],apellidos
[con2
])<0) {
SwapStrings( apellidos[con], apellidos[con2] );
SwapStrings( nombres[con], nombres[con2] );
}
}
}
Y no parece buena idea que este código se ejecute SIEMPRE que se vayan a visualizar los registros... deberías aislar este código en una función y llamarlo después de que el usuario introduzca los datos y después de cargar los datos del archivo.
8. Tu programa no está en C++, está en C puro y duro.
9. Al guardar tienes el siguiente código:
for(int c=0;c<a;c++)
{
for(int c2=0;c2<a;c2++)
{
fprintf(arch
,"%s\n",&apellidos
[c
]); }
}
el bucle de c2 no solo es redundante, sino que tiene un efecto dañino y es que la lista va a aparecer repetida en el archivo tantas veces como elementos tengas en la lista, 2 nombres aparecerán 2 veces, 20 nombres aparecerán 20 veces cada uno... 1000 nombres....
10. El siguiente código está mal:
printf("Nombre: ",nombres
[c
]); printf("Apellido: ",apellidos
[c
]); scanf("%s",&apellidos
[c
]);
Por varios motivos:
* para sacar el nombre y los apellidos por pantalla te falta '%s':
printf( "Nombre: %s", nombres
[c
] );
Y lo mismo para los apellidos... además si no añades ni espacios ni saltos de línea te van a salir pegados y el efecto es feo.
Otra cosa es que no desees mostrar el nombre que había anteriormente en la lista, lo cual es más lógico... en ese caso te puedes ahorrar pasar por argumentos en el printf el nombre y los apellidos:
* Los scanf, en el caso de vectores, matrices y punteros en general, no necesitan el '&'. Esto es así porque scanf necesita punteros para almacenar los valores... y un vector es un puntero... no necesitas pasar referencias:
scanf( "%s", nombres
[c
] );
11. Fíjate en la función carga_arch:
int cargar_arch()
{
if(fopen("dato.txt","r")) {
// ...
return 1;
}
else if (!fopen("dato.txt","r")) {
return 0;
}
}
No tiene sentido que intentes abrir dos veces el archivo, porque... ¿Y si falla la primera vez pero funciona la segunda? el else se inventó precisamente para tratar el caso del "y si no...". Y no sufras por esto, el compilador, a diferencia del programador, no se confunde con estas cosas.
int cargar_arch()
{
if(fopen("dato.txt","r")) {
// ...
return 1;
}
else
{
return 0;
}
}
12. En guardar_arch...
arch
=fopen("dato.txt","w");
¿Y si no se puede abrir el archivo? Aprende y aplica esta regla de oro: "Valida que los punteros son válidos antes de usarlos".
Y bueno, con esto creo que a grandes rasgos esto es todo. Espero que no se me haya olvidado nada.
Un saludo.
Autor
En línea
