C++ Programa para seguro de vida

[Vitor Fredie Dreher Galvão]

Boa noite meus querido, 

Não sei onde errei nesse código, ele roda normalmente mas não da os resultados certos, fiz todos os cases do switch separadinhos com os sinais certos e conferi, se foi isso eu estou cego.

Grato desde já.

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



main ()
{

	int idade;
	char nome;
	



	printf ("Digite a sua idade: \n");
	scanf ("%d",&idade);

	printf ("Digite o seu nome: \n");
	scanf ("%s",&nome);
	
	{
	
		if (idade<10)
		printf ("O %s devera pagar R$30,00",&nome); 
	
		else if ((idade>10) && (idade<=29))
		printf ("O %s devera pagar R$60,00",&nome); 
	
		else if ((idade>29) && (idade<=45))
		printf ("O %s devera pagar R$120,00",&nome); 
	
		else if ((idade>45) && (idade<=59))
		printf ("O %s devera pagar R$150,00",&nome); 
	
		else if ((idade>59) && (idade<=65))
		printf ("O %s devera pagar R$250,00",&nome); 
	
		else if (idade>65)
		printf ("O %s devera pagar R$400,00",&nome); 
	
	
	
	
	
	
	
	}

}

 

[Lucca Rodrigues]

@Vitor Fredie Dreher Galvão Você declarou um char, não uma string (vetor de caracteres).

Use algo como char nome[50].

Não precisa usar & no scanf() (no caso de strings), tampouco no printf() (para os casos em geral). Para strings, colocar "nome" e "&nome[0]" no scanf() da no mesmo.

Use também, no primeiro if(), idade<=10, pois o 10 está excluído nas suas estruturas condicionais.

[herbertbahia]

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



main ()
{

	int idade;
	char nome[50];
	



	printf ("Digite a sua idade: \n");
	scanf ("%d",&idade);

	printf ("Digite o seu nome: \n");
	scanf ("%s",nome);
	
	{
	
		if (idade<10)
		printf ("O %s devera pagar R$30,00",nome); 
	
		else if ((idade>10) && (idade<=29))
		printf ("O %s devera pagar R$60,00",nome); 
	
		else if ((idade>29) && (idade<=45))
		printf ("O %s devera pagar R$120,00",nome); 
	
		else if ((idade>45) && (idade<=59))
		printf ("O %s devera pagar R$150,00",nome); 
	
		else if ((idade>59) && (idade<=65))
		printf ("O %s devera pagar R$250,00",nome); 
	
		else if (idade>65)
		printf ("O %s devera pagar R$400,00",nome); 
	
	
	
	
	
	
	
	}

}

 

adicionado 0 minutos depois

tente usar o "=" porque se por exemplo eu digitar 10 nao vai rolar.

adicionado 1 minuto depois

so use & em scanf nunca use em printf isso significa endereco

[Vitor Fredie Dreher Galvão]

Entendi agora onde eu estava errando, obrigado pelos comentários e dicas sobre a questão do string, aos poucos vou pegando, porque meu professor sequer nem citou string nas aulas.

Quanto ao 10 eu utilizei "idade<=10" e agora ela está dando.

Obrigado.

[herbertbahia]

pois é não são todos professores que entram em mínimos detalhes principalmente se for em c++ onde naturalmente se utiliza mais o cin para leitura. mas são os mínimos detalhes que fazem a diferença no codigo. talvez seu professor nao quis citar porque existe varias maneiras e comandos de ler palavras e  frases .

[arfneto]

        if (idade<=10)
		printf ("O %s devera pagar R$30,00",nome); 
		else if ((idade>10) && (idade<=29))
		printf ("O %s devera pagar R$60,00",nome); 	
		else if ((idade>29) && (idade<=45))
		printf ("O %s devera pagar R$120,00",nome); 

No trecho acima você vê que está propagando uma redundância em todos os testes. Qual chance de uma idade que não é <- 29 ser > 29? Total. Ela não tem opção. Não precisava testar 2 duas condições a cada vez. O limite inferior já foi testado.

 

E fica difícil, mesmo num programa tão pequeno, descobrir que ele tem 6 pontos de saída. Essa não é uma boa prática.

 

main() é uma função, e ao retornar encerra seu programa. O normal. Então você podia ter escrito algo assim

	if (idade < 10)
	{	printf("O %s devera pagar R$30,00", nome);
		return;
	};

	if (idade <= 29)
	{	printf("O %s devera pagar R$60,00", nome);
		return;
	};

	if (idade <= 45)
	{	printf("O %s devera pagar R$120,00", nome);
		return;
	};

	if (idade <= 59)
	{	printf("O %s devera pagar R$150,00", nome);
		return;
	}

	if (idade <= 65)
	{	printf("O %s devera pagar R$250,00", nome);
		return;
	};

	printf("O %s devera pagar R$400,00", nome);
	return;

Que talvez concorde que é muito mais legível.

 

Ao encontrar o fim de uma função retorne explicitamente. É mais simples e seguro. E nesse caso nem teria tido o problema com o teste de <10  que te custou umas horas.

 

Outro ponto: essas { } que ficaram a mais no meio do programa às vezes não são assim inofensivas. Em C++ por exemplo elas encerram o tempo de vida (scope) de todas as variáveis que estejam declaradas dentro do bloco { }  e isso se você não fez de propósito pode ser desastroso em seu programa

 

 

 

 

[Vitor Fredie Dreher Galvão]

@arfneto De fato muito mais limpo e simples o código tanto de se entender a operacionalidade, quanto para escreve-lo. Obrigado pelas correções.