scissors cant win vs paper :(

memo333 · 10-25-2012, 01:46 AM

Code:


function ppt()
{
	

var userChoice = prompt("Escoje piedra, papel o tijeras");

var computerChoice = Math.random();

if (computerChoice <0.34){

computerChoice = "piedra";
}else if(computerChoice <=0.67){
computerChoice = "papel";
}else{
computerChoice = "tijeras";
}



function comparar(choice1,choice2){

if(choice1===choice2){return alert("EMPATE!");}//empate


if (choice1==="piedra"&&choice2=== "tijeras"){return alert("GANASTE");}
else{return alert("PERDISTE!");}


if (choice1==="papel"&&choice2=== "tijeras"){return alert("PERDISTE");}
else{return alert("GANASTE!");}


if (choice1==="tijeras"&&choice2=== "piedra"){return alert("PERDISTE!");}
else{return alert("GANASTE");}



}//fin funcion comparar

comparar(userChoice,computerChoice);

alert("Tu elejiste " +userChoice +" y la computadora elijio " +computerChoice); 




}//fin de funcion ppt

devnull69 · 10-25-2012, 01:11 PM

Your very first comparison

Code:

if (choice1==="piedra"&&choice2=== "tijeras"){return alert("GANASTE");}
else{return alert("PERDISTE!");}

means: Only return "GANASTE", if choice1 is piedra and choice2 is tijeras, in EVERY OTHER CASE it will return "PERDISTE" and will never do the other checks.

Try this

Code:

if ((choice1==="piedra"&&choice2=== "tijeras")||(choice1==="papel"&&choice2=== "tijeras")||(choice1==="tijeras"&&choice2=== "piedra")){return alert("GANASTE");}
else{return alert("PERDISTE!");}