Entrar Ainda não tem acesso?
3
respostas

A fim de correção

Referente ao curso JavaScript e HTML: desenvolva um jogo e pratique lógica de programação, no capítulo Crie suas próprias funcionalidades e atividade O problema das gerações

por André Luiz
| 66.1k xp | 7 posts
Analista de Programação Pleno 
<!DOCTYPE html>
<html>
<head>
    <meta charset="utf-8">
    <title>Aprendendo Programação</title>
</head>
<body>
    <script>
    function paragrafo(parametro){
        document.write("<p>" + parametro + "</p>");
    }

    function titulo(parametro){
        document.write("<h1>" + parametro + "</h1>");
    }

    var mediaGeracao = 28;
    var primeiroAno = 1500;
    var anoAtual = 2018;
    var tempoQuePassou = (anoAtual - primeiroAno);
    var geracoes = (tempoQuePassou / mediaGeracao);

    titulo("Quantas gerações?");
    paragrafo("Já passamos por " + Math.round(geracoes) + " gerações.");
    </script>
</body>
</html>

Estaria correto desta forma? Alguma melhoria?

3 respostas
por Alberto Souza
| 2676.6k xp | 5740 posts
Instrutor Desenvolvedor e Instrutor

Oi André, tudo bem?

Não tem problema o jeito que você fez.. .A ideia das variáveis é trazer contexto para os valores... O único ponto que você deve olhar é que uma explosão de variáveis pode confundir um pouco na hora da manutenção do código.

por DOUGLAS ALVES MARCELINO
| 156.2k xp | 39 posts
<!DOCTYPE html>
<html>
<head>
    <meta charset="utf-8">
    <title>Aprendendo Programação</title>
</head>
<body>
    <script>
    function paragrafo(parametro){
        document.write("<p>" + parametro + "</p>");
    }

    function titulo(parametro){
        document.write("<h1>" + parametro + "</h1>");
    }

    let mediaGeracao = 28;
    let primeiroAno = 1500;
    let anoAtual = 2018;
    let tempoQuePassou = (anoAtual - primeiroAno);
    let geracoes = (tempoQuePassou / mediaGeracao);

    titulo("Quantas gerações?");
    paragrafo(`Já passamos por ${Math.round(geracoes)} gerações.`);
    </script>
</body>
</html>

Eu faria dessa forma, trocaria todos os "var" por "let" , pois o var seria "uma variavel global", e não precisa disso, com o let você pode declarar as variáveis com escopo de blocos. E eu utilizei acento grave `` no parametro da segunda função para facilitar a concatenação, e desse modo, o código fica mais fácil de ser lido.

Mas o seu código está correto de qualquer forma :)

Espero ter ajudado

por Carlos Antonio Silva do Nascimento
| 343.4k xp | 747 posts
Analista de Sistema

Muito bem, ficou muito bom para o começo, mais em frente você vai aprender um pouco mais para enxugar esse código!