为什么fgets()工作错误?

时间:2015-04-30 15:03:32

标签: c eclipse fgets

这是我的代码

if(passwordCorretta(ds_sock)){
   printf("CLIENT: password aggiungi corretta\n");

   do{
      printf("Cognome >> ");
      fgets(cognome,sizeof(cognome),stdin);
      //scanf("%s", cognome);
      printf("Nome >> ");
      fgets(nome,sizeof(nome),stdin);
      //scanf("%s", nome);
      printf("Telefono >> ");
      fgets(telefono,sizeof(telefono),stdin);
      //scanf("%s", telefono);

在输出中首先打印两个printf,跳过第一个fgets(),为什么会这样?

这是我的输出

CLIENT: password aggiungi corretta
Cognome >> Nome >> **my input
Telefono >> **my input

任何想法为什么?感谢

这是我的整个功能

void main(){
int ds_sock, length, res;
struct sockaddr_in client;
struct hostent *hp;
char oper[2];

int risPwd;
int op;

char cognome[30];
char nome[20];
char telefono[12];

char continua[3];
char ris[2];
int trovato;
int aggiunto;

ds_sock = socket(AF_INET, SOCK_STREAM, 0);

client.sin_family = AF_INET;
client.sin_port = 1999;

hp = gethostbyname("localhost");  //indirizzo del server
memcpy(&client.sin_addr, hp->h_addr, 4);

res = connect(ds_sock, &client, sizeof(client));
if(res==-1) {
    perror("Errore nella connessione");
}

signal(SIGPIPE, gest_broken_pipe);
signal(SIGINT, gest_interruzione);

do{
    op=scelta();
    sprintf(oper,"%d",op);
    if(write(ds_sock, oper, sizeof(oper))<0){
        if(errno!=EINTR) perror("Errore di scrittura");
    }

    switch(op){
            case 1:
                printf("INSERIMENTO NUOVO CONTATTO\n");

                if(passwordCorretta(ds_sock)){
                    printf("CLIENT: password aggiungi corretta\n");

                    do{
                        printf("Cognome >> ");
                        fgets(cognome,sizeof(cognome),stdin);
                        //scanf("%s", cognome);
                        printf("Nome >> ");
                        fgets(nome,sizeof(nome),stdin);
                        //scanf("%s", nome);
                        printf("Telefono >> ");
                        fgets(telefono,sizeof(telefono),stdin);
                        //scanf("%s", telefono);
                        if(write(ds_sock, cognome, sizeof(cognome))<0){
                            if(errno!=EINTR) perror("Errore di scrittura");
                        }
                        if(write(ds_sock, nome, sizeof(nome))<0){
                            if(errno!=EINTR) perror("Errore di scrittura");
                        }
                        if(write(ds_sock, telefono, sizeof(telefono))<0){
                            if(errno!=EINTR) perror("Errore di scrittura");
                        }

                        if(read(ds_sock, ris, sizeof(ris))<0){
                            if(errno!=EINTR) perror("Errore di lettura");
                        }
                        trovato=atoi(ris);

                        switch(trovato){
                            case 0:
                                printf("Errore di lettura nel Server\n");
                                break;
                            case 1:
                                printf("Il contatto è già presente nell'elenco\n");
                                break;
                            case 2:
                                if(read(ds_sock, ris, sizeof(ris))<0){
                                    if(errno!=EINTR) perror("Errore di lettura");
                                }
                                aggiunto=atoi(ris);
                                if(aggiunto==0) printf("Errore di scrittura nel Server\n");
                                else printf("Il contatto è stato correttamente inserito nell'elenco\n");
                                break;
                        }

                        printf("Vuoi aggiungere un altro contatto? [SI/NO]\n");
                        scanf("%s", continua);
                        if(write(ds_sock, continua, sizeof(continua))<0){
                            if(errno!=EINTR) perror("Errore di scrittura");
                        }
                    }while(strcmp(continua, "SI")==0);

                }
                break;

            case 2:
                printf("RICERCA DI UN NUMERO TELEFONICO\n");

                if(passwordCorretta(ds_sock)){
                    printf("CLIENT: password cerca corretta\n");

                    do{
                        printf("Cognome >> ");
                        scanf("%s", cognome);
                        printf("Nome >> ");
                        scanf("%s", nome);
                        if(write(ds_sock, cognome, sizeof(cognome))<0){
                            if(errno!=EINTR) perror("Errore di scrittura");
                        }
                        if(write(ds_sock, nome, sizeof(nome))<0){
                            if(errno!=EINTR) perror("Errore di scrittura");
                        }

                        if(read(ds_sock, ris, sizeof(ris))<0){
                            if(errno!=EINTR) perror("Errore di lettura");
                        }
                        trovato=atoi(ris);

                        switch(trovato){
                            case 0:
                                printf("Errore di lettura nel Server\n");
                                break;
                            case 1:
                                if(read(ds_sock, telefono, sizeof(telefono))<0){
                                    if(errno!=EINTR) perror("Errore di lettura");
                                }
                                if(strcmp(telefono, "errore")==0) printf("Errore di lettura nel Server\n");
                                else printf("Il telefono del contatto richiesto è: %s\n", telefono);
                                break;
                            case 2:
                                printf("Il contatto non è presente nell'elenco\n");
                                break;
                        }

                        printf("Vuoi cercare un altro numero di telefono? [SI/NO]\n");
                        scanf("%s", continua);
                        if(write(ds_sock, continua, sizeof(continua))<0){
                            if(errno!=EINTR) perror("Errore di scrittura");
                        }
                    }while(strcmp(continua, "SI")==0);

                }
                break;

            case 3:
                printf("USCITA\n");
                break;

            case 0:
                printf("Eseguito lo SHUTDOWN del Server\n");
                break;
    }

}while(op!=3 && op!=0);

close(ds_sock);

}

我使用名为cognome,nome,telefono的3个缓冲区。想法?

2 个答案:

答案 0 :(得分:1)

问题出在您的scelta()功能中。它的两个版本(scanf()fgets())都是错误的。

首先,我将解释您的fgets()版本错误的原因。

您将comando声明为int,而fgets()则需要char *。这是未定义的行为,你很幸运(或不幸)它并没有崩溃。

至于你的scanf()版本:这正是我在评论中所预测的。 scanf()仅读取整数输入,并将新行(ENTER键)保留在输入缓冲区中。这意味着您在fgets()中的第一个main()来电正在读取此左转ENTER键。

正确执行此操作的一种方法如下:

long int scelta() {
    char input[20];
    fgets(input, sizeof input, stdin); // this reads input
    while(strchr(input, '\n') == NULL) {
        // the input has not ended
        // it is probably not a long int anyway but you still need to consume it
        // choose your appropriate action, e.g.
        while(getchar() != '\n'); // read everything in input buffer up to (and including) the ENTER key
        puts("error");
        fgets(input, sizeof input, stdin); // read input again
    }
    char *endptr;
    long int comando = strtol(input, &endptr, 10);
    // then check range and check for errors with endptr
    return comando;
}

有关strtol()的详细说明和示例代码(带错误检查),请参阅Linux Programmer's Manual

答案 1 :(得分:0)

如果您的系统支持它,则函数fpurge会清除输入流,并且可以在这种情况下使用:

if(passwordCorretta(ds_sock)){
   fpurge(stdin);
   printf("CLIENT: password aggiungi corretta\n");

   do{

并且不要尝试使用fflush,因为它只对输出流起作用。

但无论如何,您应该尝试了解不需要的输入来自何处。这距离快速而肮脏的黑客不远,但至少是无害的,缓冲区中没有任何东西。