Aparte de que no te funcione bien, tienes otro problema: estás modificando el valor de las cartas, lo que deberías evitar, porque si: Si quieres comprobar el valor dos veces, la segunda vez volverá a multiplicar por dos el valor de las cartas rojas.
Usando la estrategia divide y vencerás, puedes crear primero una función que te calcule el valor de una carta sin modificarla y luego usarla para sumar un conjunto de cartas.
En mi solución estoy usando el método Array.reduce()
por lo simple que queda, ahorrándote hasta los bucles:
let carta1 = {PALO:"c",VALOR:1}
let carta2 = {PALO:"d",VALOR:7}
let carta3 = {PALO:"p",VALOR:5}
let cartas = [carta1,carta2,carta3];
function puntuaCarta(c) {
let v = c.VALOR === 1 ? 20: c.VALOR;
if (c.PALO === 'c' || c.PALO === 'd') {
v = v * 2; //duplicamos
}
console.log('Vale',v);
return v;
}
function puntua(cartas){
var mano = 0;
return cartas.reduce((
(acumulador,c ) => acumulador + puntuaCarta(c)), //función que acumula
0 //valor inicial del acumulador
);
}
console.log('Puntuación',puntua(cartas));
Pero si quieres un clásico bucle (totalmente válido, es cuestión de gustos), puedes hacerlo así:
let carta1 = {PALO:"c",VALOR:1}
let carta2 = {PALO:"d",VALOR:7}
let carta3 = {PALO:"p",VALOR:5}
let cartas = [carta1,carta2,carta3];
function puntuaCarta(c) {
let v = c.VALOR === 1 ? 20: c.VALOR;
if (c.PALO === 'c' || c.PALO === 'd') {
v = v * 2; //duplicamos
}
console.log('Vale',v);
return v;
}
function puntua(cartas){
var mano = 0;
for (let i=0;i<cartas.length;i++) {
mano+=puntuaCarta(cartas[i]);
}
return mano;
}
console.log('Puntuación',puntua(cartas));