Skip to main content
se añadieron 1154 caracteres en el cuerpo
Origen Enlace
Pablo Lozano
  • 49.9k
  • 8
  • 55
  • 92

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));

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 quieres comprobar el valor dos veces, la segunda vez volverá a multiplicar por dos el valor de las cartas rojas.

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)),0)
}

console.log('Puntuación',puntua(cartas));

Aparte de que no te funcione bien, tienes otro problema: estás modificando el valor de las cartas, lo que deberías evitar: 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){
  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));

Origen Enlace
Pablo Lozano
  • 49.9k
  • 8
  • 55
  • 92

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 quieres comprobar el valor dos veces, la segunda vez volverá a multiplicar por dos el valor de las cartas rojas.

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)),0)
}

console.log('Puntuación',puntua(cartas));