Solucionado (ver solução)
Solucionado
(ver solução)
2
respostas

Resposta

Gostaria de saber se o jeito que fiz foi correto.

<meta charset="UTF-8">

<script>
    function pulaLinha(){
        document.write("<br>");
    }

    function mostra(frase){
        document.write(frase);
    }


    var anoPortugueses = 1500;
    var anoAtual = 2020;
    var anosPassaram = anoAtual - anoPortugueses;

    var mediaGeracoes = anosPassaram/28;

    mostra("Passaram-se " + mediaGeracoes.toFixed(1) + " gerações.")
</script>

Obrigada!

2 respostas
solução!

Olá Victória tudo bem ? Acabei de testar seu código aqui e vou te passar meu feedback . =)

A meneira como você fez está correta mas tem alguns pontos que podem ser implementados começando pela a váriavel anosPassaram veja que nessa linha

 var anosPassaram = anoAtual - anoPortugueses;

É uma boa prática fazer a utilização de parênteses para cálculo =)

var anosPassaram = (anoAtual - anoPortugueses);

Outro ponto também que é interessante é você fazer a declaração apenas com o valor da variavél antes de fazer direto a soma , desse modo acaba sendo mais fácil de vizualizar , oque você declara por exemplo :

 var mediaGeracoes = anosPassaram/28;

na váriavel mediaGeracoes nesse caso eu optaria em fazer apenas a declaração do valor dela :

var mediaGeracoes = 28;

Fazendo dessa maneira fica muito mais simples de você visualizar oque está acontecendo no seu código , claro que é opcional até por que da outra maneira que você declarou está correto também . =)

Então se fossemos optar em fazer a utilização da váriavel acima , nesse caso para fazer o cálculo você teria que criar outra váriavel :

var media = (anosPassaram/mediaGeracoes);

Aqui eu criei uma nova váriavel chamada media apenas para fazer a divisão.

Dessa forma sendo declarada teria que ocorrer uma mudança nessa linha :

 mostra("Passaram-se " + mediaGeracoes.toFixed(1) + " gerações.")

Observe que você fez a utilização do método toFixed() nesse caso eu aconselho você fazer a utilização do Math.round() esse método faz um arredondamento do valor de um número dado aproximado para o inteiro mais próximo. Então nesse caso o código ficara assim :


    mostra("Passaram-se " + Math.round(media) + " gerações.");

Então após essas colocações o código ficaria assim :

<meta charset="UTF-8">

<script>
    function pulaLinha(){
        document.write("<br>");
    }

    function mostra(frase){
        document.write(frase);
    }


    var anoPortugueses = 1500;
    var anoAtual = 2020;
    var anosPassaram = (anoAtual - anoPortugueses);

    var mediaGeracoes = 28;

    var media = (anosPassaram/mediaGeracoes);

    mostra("Passaram-se " + Math.round(media) + " gerações.")

</script>

Bom espero ter ajudado , lembre-se que da maneira que você fez seu código está correto =) mas é apenas um feedback que quis passar para você .

Que você tenha um execelente estudos ..

Abraços ....

Obrigada Suelen, me ajudou demais, é sempre bom ter feedback de melhoramento do código, agradeço.