1

Estou resolvendo um exercício de lógica e não consigo entender o erro:

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

int main(){
int TotalVinhos=0, t=0, b=0, r=0, fim = 0;
//float Porc;
char tipo;

while(fim == 0){
  printf("Tipo do vinho: ");
  scanf("%c", &tipo);

switch(tipo){
    case 'T':
        t++;
        TotalVinhos++;
        break;
    case 'B':
        b++;
        TotalVinhos++;
        break;
    case 'R':
        r++;
        TotalVinhos++;
        break;
    case 'F':
        fim = 1;
        break;
    default:
        printf("erro..");
        break;
  }

}

 printf("Total de vinhos: %d\n", TotalVinhos);
 printf("Quantida de vinho tipo T: %d\n", t);
 printf("Quantida de vinho tipo B: %d\n", b);
 printf("Quantida de vinho tipo R: %d", r); 
}

O erro que me é apresentado:

Ele executa 2x e já da a opção default

8
  • Esse código nem compila, pelo menos em compilador decente. Inclusive esse código não pode dar o resultado apresentado. Se você posta um código e está usando outro fica complicado ajudar.
    – Maniero
    Commented 13/05/2017 às 19:55
  • Qual compilador é descente?
    – WSS
    Commented 13/05/2017 às 19:58
  • @bigown veja, alterei o código.. agora é pra compilar porém mantem o erro ali
    – WSS
    Commented 13/05/2017 às 20:04
  • Piorou. Tem que postar algo que você sabe que pelo menos está compilando.
    – Maniero
    Commented 13/05/2017 às 20:15
  • 1
    Não pra mim i.sstatic.net/lX9ri.png
    – Maniero
    Commented 13/05/2017 às 20:24

1 Resposta 1

1

Um array de char termina com \0 então criei com tamanho 2 (um para o caractere e um para o \0). Na hora de passar o valor para o case, passei só o tipo[0], que é o valor necessário. Mudei também o bool porque não havia nenhum include para o tipo bool no código. Substitui por int com 1 e 0 que funciona da mesma forma.

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

int main()
{
    int TotalVinhos=0, t=0, b=0, r=0;
    //float Porc;
    int fim=1;
    char tipo[2];

    do
    {
        printf("Tipo do vinho: ");
        scanf("%s", &tipo);

        switch(tipo[0])
        {
        case 'T':
            t++;
            TotalVinhos++;
            break;
        case 'B':
            b++;
            TotalVinhos++;
            break;
        case 'R':
            r++;
            TotalVinhos++;
            break;
        case 'F':
            fim = 0;
            break;
        default:
            printf("erro..");
            break;
        }

    }
    while(fim);

    printf("Total de vinhos: %d\n", TotalVinhos);
    printf("Quantida de vinho tipo T: %d\n", t);
    printf("Quantida de vinho tipo B: %d\n", b);
    printf("Quantida de vinho tipo R: %d", r);
}

Não sei se há algum outro modo de fazer, mas assim está funcionando corretamente.

3
  • Bom funcionou, só não entendi porque no switch você coloca "tipo[0]"
    – WSS
    Commented 13/05/2017 às 20:28
  • 1
    Porque é a posição que está o caractere que você recebeu, ( a outra "tipo[1]" está com o "\0"). Já editei a resposta com uma explicação curta do que foi alterado. Commented 13/05/2017 às 20:33
  • Por exemplo, se eu recebi um 'T' no scanf, a variável tipo será: tipo = ['T', '/0'] Commented 13/05/2017 às 20:35

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .