Botão "voltar" quiz em JavaScript

geoleandro · Novembro 20, 2019

O botão voltar funciona, mas ele adiciona uma questão na variável "numQ".

Onde está o erro no código abaixo:

<!DOCTYPE html>
<html>
<head>

	<meta name="viewport" content="width=device-width, initial-scale=1.0">

<style>

.imgbox {
  float: left;
  text-align: center;
  width: 150px;
  border: 1px solid #ddd;
  margin: 4px;
  padding: 5px;
}

#mylabel {

text-align: left;
}


</style>

<body onload="loadquestion()">


 
  <p id="numQ">

  <p id="mylabel" name="questao">Questões</p><br>
  
  
  <div class="imgbox" id="imgbox1"><br>
    <input type="image" src=" " width =auto height= 80px   id="btn0"  value="option0" name="opt0" onclick="checkans(1)"><br><br>
  </div>
  
  <div class="imgbox" id="imgbox2"><br>
  <input type="image" src=" " width =auto  height= 80px  id="btn1"  value="option1" name="opt1" onclick="checkans(2)"><br><br>
    </div>
  
  <br><br><br><br><br><br><br><br><br>
  
  <div class="imgbox" id="imgbox3"><br>
  <input type="image" src=" " width =auto  height= 80px  id="btn2"  value="option2" name="opt2" onclick="checkans(3)"><br><br>
  </div>
  
  
  <div class="imgbox" id="imgbox4"><br>
  <input type="image" src=" " width =auto  height= 80px  id="btn3"  value="option3" name="opt3" onclick="checkans(4)"><br><br>
  </div>
  
  

  <br><br><br><br><br><br>  <br><br><br><br>
  
     
  <input type="button" id="next" value="Próximo" name="nxtbtn" onclick="changequestion()"><br><br>

   <input type="button" id="back" value="voltar" name="bkbtn" onclick="backQ()"><br><br>
   
   
<p id="erro">
<p id="pontos">
<p id="questAtual">

<script type="text/javascript">

   i=0;
   var pontos = 0;
   var numQ = 1;
  

   
 myqs=[["Clique na foto 3 ?","img3.gif", "img2.gif","img1.gif","img1a.gif","3"],
      ["Clique na foto 2 ?","icone.png","icone2.png","certo.png","certo.png","2"],
      ["Clique na foto 4","icone.png","errado.png","certo.png","certo.png","4"]
 ];

 function loadquestion() {

 
document.getElementById("mylabel").innerHTML= myqs[i][0];
document.getElementById("btn0").src= myqs[i][1];
document.getElementById("btn1").src= myqs[i][2];
document.getElementById("btn2").src= myqs[i][3];
document.getElementById("btn3").src= myqs[i][4];
document.getElementById ("numQ").innerHTML = "Questão " + numQ + " de " + myqs.length;

document.getElementById("next").value = "Próximo";
document.getElementById("next").style.backgroundColor = "lightgray";
   numQ++;
   
document.getElementById("imgbox1").style.backgroundColor = "white";
document.getElementById("imgbox2").style.backgroundColor = "white";
document.getElementById("imgbox3").style.backgroundColor = "white";
document.getElementById("imgbox4").style.backgroundColor = "white";
document.getElementById("erro").innerHTML = "";
document.getElementById("erro").style.color = "";
	
}
 
 
 
function changequestion(){
 i=i+1;
 loadquestion();
	
	
	
}


function backQ(){

 loadquestion();
 i=i-1
 
}


function checkans(a){
 respostas =parseInt(myqs[i][5]);
 
 if(respostas==a && respostas==3){
  pontos++;
  
document.getElementById ("pontos").innerHTML = "Você acertou " + pontos ;
document.getElementById("imgbox3").style.backgroundColor = "#99ff99";
  
 }
  else if(respostas==a && respostas==2){
  pontos++;
  
document.getElementById ("pontos").innerHTML = "Você acertou " + pontos ;
document.getElementById("imgbox2").style.backgroundColor = "#99ff99";
 }
 
  else if(respostas==a && respostas==4){
  pontos++;
  
document.getElementById ("pontos").innerHTML = "Você acertou " + pontos ;
document.getElementById("imgbox4").style.backgroundColor = "#99ff99";
 }
 
 else {
 
 document.getElementById("erro").innerHTML = "Incorreta";
 document.getElementById("erro").style.color = "red";
 }


}

 </script>
</head>
</body>
</html>

wanderval · Novembro 24, 2019

O erro no seu código e que você não manipula a variável que é incrementada pra exibir o estado da questão atual:

Before:

function backQ(){
 loadquestion();
 i = i - 1; // ainda não entendi pq decrementou depois de carregar
}

After:

function backQ(){
 i = i - 1;
 numQ = i;
 loadquestion();
}

JsBin: https://jsbin.com/zobofakija/edit?html,js,output

PS: Seu html tá triste precisa estudar CSS aqueles "<br>" da um certo desgosto você colocou dentro do <head> todo o código e não está se preocupando com os espaçamentos tornado seu código sujo e seu Javascript tem que começar a pensar em reaproveitar código.

fiz um outro código com um refactor:

i = 0;
var pontos = 0;
var numQ = 1;
  
var myqs = [
  ["Clique na foto 3 ?","img3.gif", "img2.gif","img1.gif","img1a.gif","3"],
  ["Clique na foto 2 ?","icone.png","icone2.png","certo.png","certo.png","2"],
  ["Clique na foto 4","icone.png","errado.png","certo.png","certo.png","4"]
 ];

function initApp() {
  setValue('next', 'Próximo');
  setBackground('next', 'lightgray');
  loadQuestion();
}

function loadQuestion(numQ = 1) {
  setContent('mylabel', myqs[i][0]);
  setImagesOnBox();
  updateStateQuestion(numQ);
  defaultStateBox();
  clearMessage('erro');
}

function setImagesOnBox() {
  const targetList = ['btn0', 'btn1', 'btn2', 'btn3'];
  targetList.forEach((item, index) => setSrc(item, myqs[i][index+1]));
}

function updateStateQuestion(numQ) {
  setContent("numQ", "Questão " + numQ + " de " + myqs.length);
}

function defaultStateBox() {
  const targetList = ['imgbox1', 'imgbox2', 'imgbox3', 'imgbox4'];
  targetList.forEach(item => setBackground(item, 'white'));
}

function nextQuestion(){
 i = i + 1;
 numQ = numQ + 1;
 loadQuestion(numQ);
}

function backQuestion(){
 i = i - 1;
 numQ = numQ - 1;
 loadQuestion(numQ);
}

function checkans(a){
 clearMessage('erro');
 respostas = parseInt(myqs[i][5]);

 if(respostas === a){
  pontos++;
  
  setContent("pontos", "Você acertou " + pontos);
  setBackground("imgbox"+respostas, "#99ff99"); 
 }
 else {
  showIncorrectMessage();
 }
}

function clearMessage(fieldId) {
  setContent(fieldId, "");
  setColor(fieldId, "");	
}

function showIncorrectMessage() {
  setContent("erro", "Incorreta");
  setColor("erro", "red");
}

// Utils methods
function refElement(fieldId) {
  return document.getElementById(fieldId);
}

function setValue(fieldId, value) {
  refElement(fieldId).value = value;
}

function setContent(fieldId, content) {
  refElement(fieldId).innerHTML = content;
}

function setBackground(fieldId, color) {
  refElement(fieldId).style.backgroundColor = color;
}

function setColor(fieldId, color) {
  refElement(fieldId).style.color = color;
}

function setSrc(fieldId, pathFile) {
  refElement(fieldId).src = pathFile;
}