Can't get do while loop working

Code:

---


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

#define MAX_STRING_LENGTH 1

int main(void) {

    /* Variáveis. {{{ */
    char estado_civil;
    char resp[MAX_STRING_LENGTH];
    int idade;
    int casad = 0; /* Casado(a). */
    int solt = 0; /* Solteiro(a). */
    int viuv = 0; /* Viúvo(a). */
    int sep = 0; /* Desquitado ou separado. }}} */

   
    do {
        printf("Idade?\n");
        scanf("%d", &idade);
        printf("%d.\n", idade); getchar();

        printf("Etado civíl ?\n"
            "C, casado(a); S, solteiro(a); V, viúvo(a); D, desquitado(a):\n");
        estado_civil = getchar();
       
        switch (estado_civil) { /* {{{ */
            case 'C': case 'c':
                casad++;
                break;
            case 'S': case 's':
                solt++;
                break;
            case 'V': case 'v':
                viuv++;
                break;
            case 'D': case 'd':
                sep++;
                break;
            default:
                printf("Caractere inválido.\n");
                break;
        } /* }}} */

        printf("Mais uma pessoa? (s/n)\n");
        scanf("%s", resp);
        /* scanf("%1c", resp); */
        /* getchar(); */
        /* resp = getchar(); */

    } while (strcmp(resp, "s") != 0);

    printf("C = %d, S = %d, V = %d, D = %d.\n", casad, solt, viuv, sep);

    return 0;

}

/*
 * vim: foldmethod=marker foldmarker={{{,}}}
 */

---

No matter the resp is "s" or not, the do while loop always stops..

After you do this:

Code:

---

estado_civil = getchar();

---

You hit enter. Then you do this:

Code:

---

scanf("%s", resp);

---

I bet if after that line you put in:

Code:

---

printf( "\'%s\'\n", resp );

---

You will find your missing enter.


Quzah.

Are you by chance working with your caps lock turned on?

As it is it will only exit on lower case "s" a capital "S" would not be detected.

Also you don't even need strcmp for single characters...

Try... while(tolower(resp) != 's');

Quote:

---

Originally Posted by CommonTater

Try... while(tolower(resp) != 's');

---

resp is a string.

Actually, no it's not:

Code:

---

#define MAX_STRING_LENGTH 1

int main(void) {

    /* Variáveis. {{{ */
    char estado_civil;
    char resp[MAX_STRING_LENGTH];

---

There isn't enough room in resp to store the nul character, so it's not valid treatment to use it as a string. So even if the extra enter key isn't causing the issue, the fact that it's running off the end of the buffer when it's used as a string could be making it fail the strcmp.


Quzah.

Are you by chance working with your caps lock turned on?

As it is it will only exit on lower case "s" a capital "S" would not be detected.

Also you don't even need strcmp for single characters...

Try...

Code:

---


        printf("Mais uma pessoa? (s/n)\n");
    } while (tolower(getchar()) != 's');

---

Quote:

---

Originally Posted by quzah

resp is a string.

---

You're too quick for me, my friend... I was mid-way through fixing it when you posted :D

I am sure I don't have caps lock on.

Quote:

---

Originally Posted by FernandoBasso

I am sure I don't have caps lock on.

---

Then go back and take another look at message #5 in the thread...

Quote:

---

Originally Posted by CommonTater

Are you by chance working with your caps lock turned on?

As it is it will only exit on lower case "s" a capital "S" would not be detected.

Also you don't even need strcmp for single characters...

Try...

Code:

---


        printf("Mais uma pessoa? (s/n)\n");
    } while (tolower(getchar()) != 's');

---

---

It did the trick. Thanks.