2

Em um dos exercícios de um livro que estou lendo, é pedido para que eu trace o "perfil" de uma pessoa de acordo com o ano em que ela nasceu (como aqueles testes de Facebook) mas no intuito de limitar a possibilidade do que for digitado, coloquei um if para sinalizar um erro, mas mesmo com o valor dando positivo para o if, o bloco do else segue executando, como posso resolver?

int main(void){
    setlocale(LC_ALL,"");
    int y,a,b,c;
    printf("Digite seu ano de nascimento (quatro digitos)\n");
    scanf("%d",&y);
    if (y>9999 && y<1000) printf("Ano inválido");
        else {
        b=y%100;
        a=y/100;
        c=a+b;
        c=c%5;
        switch(c){
            case 0:printf("Tímido\n"); break;
            case 1:printf("Sonhador\n"); break;
            case 2:printf("Paquerador\n"); break;
            case 3:printf("Atraente\n"); break;
            case 4:printf("Irresistível\n"); break;
                }
            }
        return 0;
    }
6
  • Qual é seu problema?
    – Maniero
    5/04/2017 às 1:22
  • mesmo colocando um ano como 20000 ou 3, ainda é dado um resultado do bloco de else
    – motorola
    5/04/2017 às 1:23
  • não entendi, mas na pergunta foram cortadas algumas palavras do que coloquei, por isso ficou dificil de entender
    – motorola
    5/04/2017 às 1:24
  • @motorola Se tá falando da minha edição, eu não removi nada de mais. Você pode confirmar isso no histórico. 5/04/2017 às 1:25
  • não foi da tua edição, realmente algumas palavras que coloquei na pergunta na apareceram
    – motorola
    5/04/2017 às 1:26

2 Respostas 2

2

Esta linha

if (y>9999 && y<1000) printf("Ano inválido");

diz: Se Y é maior que 9999 E menor que 1000 imprima "Ano inválido"

Esta condição é sempre falsa. Ou seja, o else SEMPRE é executado.

Tente assim:

if (y>9999 || y<1000) printf("Ano inválido");
2

A comparação está usando um and && quando o correto seria um or ||, já que é inválido tanto um quanto o outro. Na verdade seria impossível ambos serem inválidos, porque **ou ** o número é maior que 9999 ou ele é menor que 1000, não tem como um número ser os dois ao mesmo tempo, que é uma exigência do and.

Dei uma melhorada, mas poderia melhorar mais, poderia dar nomes melhores para as variáveis e até eliminar algumas.

#include <stdio.h>

int main(void) {
    int y = 0;
    printf("Digite seu ano de nascimento (quatro digitos)\n");
    scanf("%d", &y);
    if (y > 9999 || y < 1000) { 
        printf("Ano inválido");
    } else {
        int b = y % 100;
        int a = y / 100;
        int c = a + b;
        c %= 5;
        switch (c) {
            case 0: printf("Tímido\n"); break;
            case 1: printf("Sonhador\n"); break;
            case 2: printf("Paquerador\n"); break;
            case 3: printf("Atraente\n"); break;
            case 4: printf("Irresistível\n"); break;
        }
    }
}

Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.

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 .