[C] - programma registrazione utente

[software-bloat]

ciao a tutti ^^
sto facendo un programma dove dò la possibilità all'utente di registrarsi o accedere.
per ora ho buttato giù queste righe,ma la funziona accedi non va

#include <stdio.h>
#include <stdlib.h>

void accedi();
void registrati();

int main()
{
    char scelta;
	
    while (1) {	
	    printf("Scegli una delle seguenti opzioni\n");
	    printf("[1] - Accedi\n");
	    printf("[2] - Registrati\n");
	    printf("[3] - Esci\n");
	    	
		
	    scanf("%c", &scelta);
while(getchar() != '\n');
	
	    switch (scelta){
	        case '1':
	            accedi();
	            exit(0);
	        case '2':
	            registrati();
	            exit(0);
	        case '3':
	            exit(0);
	        default:
	            printf("\aOPZIONE NON VALIDA\n");
	    }
    }	    

    return 0;
}

void accedi()
{
	FILE * fp;
	char username[20],password[20],Cuser[20],Cpass[20],str[50];
	
	printf("USERNAME: ");
	gets(username);
	printf("PASSWORD: ");
	gets(password);
	
	while(!feof(fp)){
		fscanf(fp,"%s %s",Cuser,Cpass);
		if((strcmp(Cuser,username)==0) && (strcmp(Cpass,password)==0)){
			printf("BENVENUTO %s",Cuser);
			
			}
		
		else{
			printf("\aERRORE,UTENTE NON REGISTRATO");
			main();
			}
		}

}

void registrati()
{
    char username[20];
    char password[20];
    FILE * fp;

    fp = fopen("db_utenti.txt", "a");

    printf("USERNAME: ");
    gets(username);
    fputs(username,fp);

    fputs(" ",fp);

    printf("PASSWORD: ");
    gets(password);
    fputs(password,fp);

    fputs("\n", fp);

}

mi sa che sbaglio qualcosa nel ciclo...

[yorkeiser]

Nella funzione accedi() ti sei dimenticato di aprire il file:

fp = fopen("db_utenti.txt", "r");

Inoltre nel while, dopo che hai trovato l'utente, dovresti aggiungere un

break;

altrimenti il programma continua a leggere utenze dal file di testo e ritornarti messaggi di "non registrato".
Inoltre è buona norma chiudere il puntatore al file, con una fclose, dopo averlo utilizzato.

[software-bloat]

che imbecille che sono....non so perchè non ho aperto il canale...xD
invece del break; avevo messo exit(0),va bene lo stesso?

[yorkeiser]

Dove li hai messi tu vanno pure bene (dipende: se vuoi terminare l'esecuzione del programma dopo la stampa del messaggio, vanno bene), ma devi comunque forzare l'uscita nel while di accedi() (appunto con un "break;"), subito dopo la printf di benvenuto.

[software-bloat]

ok ho fatto ma non funziona...da sempre errore,funziona solo se inserisci i dati del primo utente della lista

[yorkeiser]

Non funziona perchè è sbagliata la gestione della stampa dei messaggi nel while e la chiamata al main() nell'else: praticamente, appena vai nel ramo else, richiami il main uscendo quindi dal while, anche se non sei a fine file.
Ti incollo una possibile versione, che tra l'altro utilizza un flag (trovato) al posto del break, che personalmente odio come la morte.

int trovato=0;
while(!feof(fp) && !trovato)
{
    fscanf(fp,"%s %s\n",Cuser,Cpass);
    if((strcmp(Cuser,username)==0) && (strcmp(Cpass,password)==0))
    {
	trovato=1;
    }


}

if (trovato)
    printf("BENVENUTO %s",Cuser);
else
{
    printf("\aERRORE,UTENTE NON REGISTRATO");
    main();
}

[software-bloat]

grazie,così sembra funzionare,se avrò altri problemi scriverò qui ^^

PS: perchè lo odi?