Foro de elhacker.net

Hola!, podrían ayudarme a encontrar el error de este programa?

#include<stdio.h>
#include<stdlib.h>
#include<ctype.h>
#include<string.h>
 
int a;
char nombres[1000][25],apellidos[1000][30];
FILE*arch;
int cargar_arch(){
                  if(fopen("dato.txt","r")){
                  arch=fopen("dato.txt","r");
                  fscanf(arch,"%i",&a);
                  for(int c=0;c<a;c++){
                                        fprintf(arch,"%s\n",nombres[c]);
                                        fprintf(arch,"%s\n",apellidos[c]);
                                        }
                  fclose(arch);
                  return 1;
                  }                
                  else if (!fopen("dato.txt","r")){
                                                   return 0;
                                                   }
                          }
void cargar(){
              printf("Ingrese cantidad de empleados\n");
              printf("Cantidad: ");
              do{
                 scanf("%i",&a);
                 }
                while(a<=0||a>=1000);
             for(int c=0;c<a;c++){system("cls");
                                  printf("Empleado Numero %i\n",c+1);
                                  printf("Nombre: ",nombres[c]);
                                  scanf("%s",&nombres[c]);
                                  printf("Apellido: ",apellidos[c]);
                                  scanf("%s",&apellidos[c]);
                                  system("cls");
                                  }
 
             }
int guardar_arch(){
                   arch=fopen("dato.txt","w");
                   fprintf(arch,"%i\n",&a);
                   for(int c=0;c<a;c++){
                                        for(int c2=0;c2<a;c2++){
                                                                fprintf(arch,"%s\n",&nombres[c]);
                                                                fprintf(arch,"%s\n",&apellidos[c]);
                                                                }
                                        }
 
                   fclose(arch);
                   }
 
main(){
       int op;
       char aux[25];
       if(cargar_arch()==1){printf("Se cargaron datos anteriores\n");}
       else printf("No hay datos anteriores\n");
       do{
          printf("1)Cargar\n");
          printf("2)Ver\n");
          printf("3)Salir\n");
          printf("Opcion: ");
          scanf("%i",&op);
           switch(op){
                      case 1:
                      cargar();       
                      break;
                      case 2:
                      printf("EMPLEADOS\n\n");
                       for(int con=0;con<a;con++){
                                                  for(int con2=0;con2<a;con2++){
                                                                                if(strcmp(apellidos[con],apellidos[con2])<0){
                                                                                strcpy(aux,apellidos[con]);
                                                                                strcpy(apellidos[con],apellidos[con2]);
                                                                                strcpy(apellidos[con2],aux);
                                                                                strcpy(aux,nombres[con]);
                                                                                strcpy(nombres[con],nombres[con2]);
                                                                                strcpy(nombres[con2],aux);
                                                                                                         }
                                                                               }
                                                }
                       for(int con=0;con<a;con++){
                                                  printf("%s ,%s\n",apellidos[con],nombres[con]);
                                                 }                                 
                       break;
                       case 3:
                       guardar_arch();
                       break;
                       system("cls");
                       system("pause");
                       }
   }
   while(op!=3);
}

Salta error en el segundo corchete y no entiendo porque!
Muchas gracias

podrías publicar exactamente el error que salta? los errores son útiles porque dicen cual fue el error y sabiendo el mismo puedes arreglarlo, lo estoy compilando en un compilador online y compila bien

No se si a esto te referis

Otra cosa muy importante comenta tu codigo escribir

//No cuesta mucho : )

1. No ha creado el archivo .txt pero si hace su funcion.

(http://1.bp.blogspot.com/-1InFOuvxkVg/UxgR0WN8F9I/AAAAAAAAAi8/urbrcNXVLpY/s1600/Untitled.png)

2. salgo de la programa me crea el .txt pero al volver entrar me salta este error

(http://3.bp.blogspot.com/-S4TtfbiOj2w/UxgSlLYacEI/AAAAAAAAAjE/bygGb1MA1i0/s1600/Untitled1.png)

3. es el error que te salta a ti, y como comenta engelx podrias ser un poco mas explicito.

Una pregunta por que creas arreglos bidimencionales para nombre y apellido? revisa eso.

Saludos.

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| (http://foro.elhacker.net/programacion_cc/lo_que_no_hay_que_hacer_en_cc_nivel_basico-t277729.0.html).

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( aux, nombres[con] );
      strcpy( nombres[con], nombres[con2] );
      strcpy( nombres[con2], aux );
    }
  }
}

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];
  strcpy( aux, string1 );
  strcpy( string1 , string2  );
  strcpy( string2 , aux );
}
 
// ...
 
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",&nombres[c]);
    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]);
scanf("%s",&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:

printf("Nombre: " );

* 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");
fprintf(arch,"%i\n",&a);

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

Hay un error lógico en este bloque:

int cargar_arch()
{
   if (fopen("dato.txt","r")){
      arch = fopen("dato.txt","r"); /* modo "r", lectura */
      fscanf(arch, "%i", &a);       /* OK */
 
      for (int c = 0; c < a; c++){
         fprintf(arch, "%s\n", nombres[c]);   /* <== */
         fprintf(arch, "%s\n", apellidos[c]); /* <== */
      }
      fclose(arch);
 
      return 1;
   }

Considerando el estado del programa tal vez haya mas (no lo revise a fondo).

Un saludo

Cierto... no me había fijado en las dos llamadas consecutivas.

Bien visto.

No quiero ni pensar que habrías escrito si hubieras entrado en detalle  :laugh:

Jajajajaja.

Bueno, lo cierto es que cuando ya has puesto una docena de comentarios con cosas a corregir sobre un programa tan corto no tiene sentido seguir poniendo cosas porque entonces los comentarios se empiezan a solapar. Tampoco es cuestión de echar para atrás a nadie.

desde luego ya te digo yo que si fuese profesor, visto lo visto, me centraría bastante en conseguir que la gente aprendiese a estructurar un programa... me da la sensación que la inmensa mayoría únicamente buscan que el programa funcione y eso me parece un error.

El problema está en que muchas veces lo único que se busca es eso. Yo estoy estudiando ingeniería informática y una compañera que ha venido de telecomunicaciones me comentó que allí cuando programaban lo único que les pedían era que el programa funcionase, y que la organización y eficiencia del código eran completamente secundarias. Supongo que también dependerá del lugar en el que estudies y de lo que estudies, pero a mi me crujen si les entrego un código con nombres de variables como las de este señor.

En mi universidad pasaba igual... te daban un guión y el resto era cosa tuya... ellos al final solo querían ver que funcionaba y que no había lagunas de memoria... así pasa que luego te encuentras proyectos reales que te dan ganas de arrancarte los ojos.