[C] Stampa delle informazioni acquisite su file

[caos-zero]

Salve ragazzi ho questo problema: il programma creato fa il suo dovere ma va in loop quando deve stampare delle informazioni. Di seguito posto il codice:

#include <stdio.h>
#include <string.h>
void scrivi();
void cerca();
struct azienda{
       char nome[20];
       char cognome[20];
       int stipendio;
       int eta;
       }dipendente;
FILE *p;
char r,nome1[20];
int main(int argc,char*argv[]){
    scrivi();
    cerca();
    system ("pause");
    return 0;
                               }

void scrivi(){
    p=fopen("Dipendente.txt","w");
    if(p!=NULL)
    {

      do
      {
        printf("Inserire nome: ");
        scanf("%s",&dipendente.nome);
        printf("Inserire cognome: ");
        scanf("%s",&dipendente.cognome);
        printf("Inserire stipendio: ");
        scanf("%d",&dipendente.stipendio);
        printf("Inserire eta': ");
        scanf("%d",&dipendente.eta);
        fprintf(p,"% s % s % d % d \n",dipendente.nome,dipendente.cognome,dipendente.stipendio,dipendente.eta);
       do
        {
          printf("Si vuole uscire dall'inserimento?");
          scanf("%s",&r);

        }while(r!='n' && r!='s');
      }while(r=='n');
      fclose(p);
      }
      else{
           printf("Errore");
      }
      }
      
void cerca(){
              p=fopen("Dipendente.txt","r");
     if(p!=NULL){
                 printf("Inserire nome del dipendente da cercare: ");
                 scanf("%s",&nome1);
                 fscanf(p,"% s % s % d % d \n",dipendente.nome,dipendente.cognome,dipendente.stipendio,dipendente.eta);
     while(!feof(p)){
                    if(!strcmp(nome1,dipendente.nome)){
                                                      printf("Il dipendente %s %s\n ha reddito di %d\n di eta' %d\n ",dipendente.nome,dipendente.cognome,dipendente.stipendio,dipendente.eta);
                                                      }
                    
                    fscanf(p,"% s % s % d % d \n",dipendente.nome,dipendente.cognome,dipendente.stipendio,dipendente.eta);
                    }
     fclose(p);
     }
     else{
          printf("Errore");
          }
            }

Aiutatemi

[Patrick Jane]

Per stampare delle informazioni intendi la funzione cerca()?

[caos-zero]

si, anche se non è propio ideale come nome...

[YuYevon]

Stai usando male le scanf() e le fscanf().

Quando leggi una stringa non hai bisogno di anteporre l'operatore & alla variabile stringa, questo perché di per sé una variabile del genere è già un indirizzo. Inoltre elimina gli spazi nei codici di formato: perché scrivi "% s" e non semplicemente "%s" ad esempio? Anche il "\n" in quel modo nella scanf() non ha utilità.

Considera poi che, quando leggi i dati da file, ai parametri "dipendente.stipendio" e "dipendente.eta" devi anteporre la & perché devi passarne l'indirizzo (è esattamente l'errore opposto a quello che commetti altrove per le stringhe).

Riscriverei così (apportando le modifiche minime):

printf("Inserire nome del dipendente da cercare: ");
scanf("%s", nome1);
fscanf(p,"%s %s %d %d", dipendente.nome, dipendente.cognome, &dipendente.stipendio, &dipendente.eta);
while (!feof(p)) {
    if (!strcmp(nome1,dipendente.nome)) {
        printf("Il dipendente %s %s\n ha reddito di %d\n di eta' %d\n ", dipendente.nome, dipendente.cognome, dipendente.stipendio, dipendente.eta);
        fscanf(p,"%s %s %d %d", dipendente.nome, dipendente.cognome, &dipendente.stipendio, &dipendente.eta);
    }
    fclose(p);
}

l'errore della & per le stringhe nelle scanf() è presente comunque anche altrove.

[caos-zero]

a questo punto per confondermi sarebbe meglio sostituire scanf con gets?

[YuYevon]

Originariamente inviato da caos-zero
sostituire scanf con gets?

Risolveresti *quel* problema ma ne introdurresti un altro ben più grave di sicurezza e affidabilità: la gets() è ufficialmente deprecata, questo perché non esegue alcun controllo sulla dimensione del buffer in cui va a scrivere dando quindi adito a buffer overflow. In realtà lo stesso problema si ha anche con la scanf() e il codice di formato %s, per quanto in quel caso ci siano alcuni work-around.

Puoi comunque ricorrere alla fgets(), generalizzazione di gets() per la lettura da input file stream (quindi anche da stdin) e buffer safe.