0

Criei uma função pularLinha() para não precisar ficar repetindo <br> constantemente, contudo para fazer a linha pular mais de uma vez dentro da função mostra(), eu chamo pularLinha() por duas vezes.

Ai penso e se fosse necessário pular de 5 em 5 linhas, chamaria a mesma função por 5 vezes dentro da função mostra()? Isso ficaria horrível!

Tentei colocar uma implementação para melhorar o código e ficar funcional, mas não consegui.

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

    let mostra = function(frase){
        document.write(frase);
        pulaLinha(); pulaLinha();

    }

    let ano = 2012; 
    mostra("Eu nasci em : " + (ano - 25)); 
    mostra("Adriano nasceu em : " + (ano - 26));
    mostra("Paulo nasceu em : " + (ano - 32)); 
2
  • Se precisa repetir várias vezes a mesma coisa, use um for. Outro detalhe, para declarar funções, faça simplesmente function mostra(frase) { etc}. Nesse caso não há vantagem nenhuma em fazer mostra = function(etc..): pt.stackoverflow.com/q/13364/112052 | stackoverflow.com/a/33040926
    – hkotsubo
    Commented 28/04/2020 às 14:36
  • Outra coisa que acho que está errada aí é que fica dando um document.write em cima de outro, document.write sobrescreve todo html.
    – LeAndrade
    Commented 28/04/2020 às 14:39

1 Resposta 1

4

Primeiro, vamos arrumar esse abuso de lambda onde uma função normal funciona melhor. Se alguém ensinou assim, fuja.

Eu não acho horrível um código mostrar que faz exatamente o que é intenção dele. Se ele deve pular duas linhas então chame o código que faz o que deseja duas vezes. Se quer 5 vezes faça isso, embora eu acho exagero usar 5 vezes. Mas se realmente precisa e são muitas vezes (não faça para 2) então é só fazer um laço.

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

function mostra(frase) {
    document.write(frase);
    for (let i = 0; i < 5; i++) pulaLinha();
}

let ano = 2012; 
mostra("Eu nasci em : " + (ano - 25)); 
mostra("Adriano nasceu em : " + (ano - 26));
mostra("Paulo nasceu em : " + (ano - 32)); 

Coloquei no GitHub para referência futura.

Acho um enorme exagero, mas se quiser pode parametrizar a função que pula linha com um número de repetição:

function pulaLinha(repeticao = 1) {
    for (let i = 0; i < repeticao; i++) document.write("<br>");
};

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

let ano = 2012; 
mostra("Eu nasci em : " + (ano - 25)); 
mostra("Adriano nasceu em : " + (ano - 26));
mostra("Paulo nasceu em : " + (ano - 32)); 

Coloquei no GitHub para referência futura.

Só lembrando que o let e o argumento default funciona em interpretadores modernos, então se usará em Node, Deno ou coisas semelhantes ou pode garantir que rodará em navegador moderno ou usará um transpilador para EcmaScipt mais antigo, então pode usar, se não vá de var.

1
  • 1
    +1 pelo "arrumar esse abuso de lambda", nada como um bom codigo vanilla, fácil de entender :) Commented 28/04/2020 às 16:20

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 .