Na sua pergunta anterior eu já tinha dado uma boa organizada no código, agora está pior de novo. Não tem muito o que simplificar, só dá para organizar melhor.
Um dos erros que você comete é não manter a consistência. Hora usa de um jeito e hora usa de outro. Hora organiza, hora amontoa tudo. Hora usa cosias desnecessárias. Preste atenção caractere por caractere do que eu escrevi para ver como fica melhor.
O que eu "simplifiquei":
Tirei os protótipos, eles só são necessários em três situações: a) vai usar em outro arquivo, b) há referências circulares entres as funções ou c) o código está desorganizado demais. Como não há nenhum desses casos nesse código, eu os retirei (já havia feito antes na pergunta anterior).
Tirei as declarações de variáveis separadas, isso é estilo antigo e desnecessário. As declarações devem estar mais perto possível do seu uso. Tem que diminuir o escopo na medida do possível.
Tirei os comentários porque eles não estão sendo úteis para nada. Comentários devem explicar porque fez algo que parece estranho e não dizer o que o código faz. Há mitos sobre o uso de comentários. Se acha que um trecho de código merece um comentário, ou ele está confuso ou ele merece estar em uma função separada. Preferi não separar o menu, mas é uma possibilidade.
Um dos motivos de eu não ter mexido no menu é que me parece que o código está pela metade. Esse menu não parece fazer sentido algum. Se for pra simplificar mesmo, do jeito que está eu só tiraria ele. Não tirei porque acho que vai fazer algo útil com ele depois. Se não fizer, basta tirá-lo. Não coloque nada no código que não acrescente algo realmente útil para a experiência do usuário. No momento ele só causa transtorno.
Tirei uma declaração de variável. No outra pergunta eu tinha dito que ela era desnecessária, mas tinha deixado porque ele poderia servir para alguma coisa caso mexesse no código. Fiz agora para simplificar.
Tirei um break
depois de do return
, ele nunca seria executado. Na verdade era para tirar outras coisas redundantes, mas aí o switch
todo teria que ser tirado porque ele não tem função nenhum no código. E finalmente a mensagem de menu também teria que ser tirada.
Obviamente fica difícil fazer um code review em código que está pela metade. Na próxima vez que pedir isso, apresente um código completo, mesmo que tenha que apresentar só um trecho.
#include <stdio.h>
float converterDiasEmAnos(float dias) {
return dias / 365;
}
float converterHorasEmDias(int horas) {
return 10000 / horas;
}
int obterMeses(float anos) {
return (anos - (int)anos) * 12;
}
int main() {
printf("\t\tPrograma para saber quanto demora a tornar-se génio!\n\n");
int escolha;
printf("\nDigite a opcao correta:\n1-Continuar com o programa\n0-Para sair!\n\n");
scanf("%d", &escolha);
switch(escolha) {
case 1:
int num_horas;
printf("Quantas horas vai dedicar por dia para ser genio?");
scanf("%d", &num_horas);
float dias = converterHorasEmDias(num_horas);
float anos = converterDiasEmAnos(dias);
printf("Voce vai demorar %.0f dias ou seja aproximadamente %.0f anos\n%d meses para ficar genio\n", dias, anos, obterMeses(anos));
break;
case 0:
return 0;
default:
printf("Digite corretamente!");
break;
}
}
Vou colocar aqui o código mais simplificado tirando as partes inúteis. Você pode não gostar, mas estou fazendo o que pede a pergunta, tirar tudo que é inútil simplifica o código:
#include <stdio.h>
float converterDiasEmAnos(float dias) {
return dias / 365;
}
float converterHorasEmDias(int horas) {
return 10000 / horas;
}
int obterMeses(float anos) {
return (anos - (int)anos) * 12;
}
int main() {
printf("\t\tPrograma para saber quanto demora a tornar-se génio!\n\n");
int num_horas;
printf("Quantas horas vai dedicar por dia para ser genio?");
scanf("%d", &num_horas);
float dias = converterHorasEmDias(num_horas);
float anos = converterDiasEmAnos(dias);
printf("Voce vai demorar %.0f dias ou seja aproximadamente %.0f anos\n%d meses para ficar genio\n", dias, anos, obterMeses(anos));
}
Uma última simplificação já que as funções estão sendo usadas só uma vez. Eu sei que pode estar querendo usar para outras coisas, mas nesse caso não está sendo necessário ter funções e simplificar costuma ser eliminar tudo o que não precisa ser usado:
#include <stdio.h>
int main() {
printf("\t\tPrograma para saber quanto demora a tornar-se génio!\n\n");
int num_horas;
printf("Quantas horas vai dedicar por dia para ser genio?");
scanf("%d", &num_horas);
float dias = num_horas / 365;
float anos = 10000 / dias;
printf("Voce vai demorar %.0f dias ou seja aproximadamente %.0f anos\n%d meses para ficar genio\n", dias, anos, (anos - (int)anos) * 12);
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.
Simplificação pode dar mais legibilidade, mas pode piorá-la também. Eu poderia fazer outras que certamente pioraria. Algumas que eu fiz pode ter piorado dependendo do contexto. Olhando só para este código não piorou. Ele é bem compreensível, mas poderia não ser em um contexto maior.
double
para valores de virgula flutuante. Nao ha vantagem em usarfloat
.