1

Olá, estou começando a estudar a linguagem JS e empaquei aqui, não sei qual meu erro, alguém poderia me explicar o que estou fazendo de errado?

<!DOCTYPE html>
<html>
  <head>
    <meta charset="utf-8">
    <title>PAGINA</title>
  </head>
  <body>


  <script type="text/javascript">
    function calculaRetangulo(b,h) {
      var area = (b*h);
      var perimetro = (b+h)*2;
      return[area,perimetro];
    };
  </script>

<button type="button" onclick=" var resposta = calculaRetangulo(5,10); alert("AREA: " + resultado[0]); alert("Perimetro" + resultado[1])">EXECUTAR</button>


  </body>
</html>
1
  • 1
    Ao invés de usar aspas duplas em alert(), use aspas simples. Você já está usando as aspas duplas ao redor do código, então quando você incluiu no alerta as aspas se abrem e fecham diversas vezes
    – RFL
    Commented 5/08/2017 às 2:04

2 Respostas 2

1

Nunca use aspas duplas (") dentro de aspas duplas, ou aspas simples (') dentro de aspas simples, ou se for necessário fazer isso, você terá que usar uma técnica para escapar a string usando algo do tipo \string para escapar\, caso contrario você estará fechando o atributo, se abrir o console do navegador pode ver como ele lê o trecho do código:

inserir a descrição da imagem aqui

Quando na verdade era pra ter sido lido assim:

inserir a descrição da imagem aqui

Alem disso você confundiu a variável resposta com a variável resultado que nem existia.

O Código correto poderia ser assim

function calculaRetangulo(b, h) {
  var area = (b * h);
  var perimetro = (b + h) * 2;
  return [area, perimetro];
};
<button type="button" onclick="var resposta = calculaRetangulo(5,10); alert('AREA: '+resposta[0]); alert('Perimetro: '+resposta[1])">EXECUTAR</button>

11
  • 1
    Contunia mesma coisa, console aponta isso -> Uncaught SyntaxError: Unexpected token }
    – Fernando
    Commented 5/08/2017 às 1:59
  • 1
    Leone, inclua a explicação do que o seu código faz e qual a diferença dele pro que está na pergunta. Assim fica fácil pra quem achar essa resposta depois. Commented 5/08/2017 às 2:02
  • 1
    Eu dei uma pesquisada sobre o que é parseInt
    – Fernando
    Commented 5/08/2017 às 2:04
  • 1
    Uhum, mais agora entendi quais foram meus erros, vlw pela ajuda
    – Fernando
    Commented 5/08/2017 às 2:13
  • 1
    Apesar de o código resolver o problema, a resposta em nada explica o que estava errado no código do AP, por isso dei -1, se explicar o erro eu retiro o voto -1 =) Commented 5/08/2017 às 3:22
1

Você esta usando aspas duplas para iniciar o onClick() e dentro do próprio onClick() você também esta usando aspas duplas que é o que esta gerando este erro, quando você faz onClick="... e coloca outra aspas duplas logo em seguida o que é interpretado é que o onClick() já terminou por exemplo:

<button onClick="alert("oi")">click</button>
                      /\ - o onClick termina aqui

O método correto seria:

<button onClick="alert('oi')">click</button>
                      /\ aspas simples

Leve isso sempre como uma regra, se você começar com aspas duplas use aspas simples dentro delas e virse e versa.

4
  • 1
    Entendi... Vlw pela ajuda
    – Fernando
    Commented 5/08/2017 às 2:12
  • 1
    Se a resposta está correta por favor marque como resolvido (o sinal de "v" ao lado esquerdo)
    – RFL
    Commented 5/08/2017 às 2:13
  • 1
    @LeoCaracciolo done.
    – RFL
    Commented 5/08/2017 às 13:28
  • 1
    agora sim, bem melhor, +1
    – user60252
    Commented 5/08/2017 às 14:47

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 .