add card object values together-CodePudding

Im new to programming and Javascript and im a bit stuck on a school assignment atm. My code below works when i log one object like this:
score([{ suit: 'HEARTS', value: 1 }]) This logs 11 but if i want to log multiple objects at the same time and add their values together, how could i do that? For example, if i call these objects below with the values 2, 3 and 4 it should return 9.

score([{ suit: 'HEARTS', value: 2 }, { suit: 'HEARTS', value: 3 }, { suit: 'HEARTS', value: 4 }])

Here is my code:

let score = function (cardObject) {
  let getScore = 0;

  for (let i = 0; i < cardObject.length; i  ) {
   getScore  = cardObject[i].value
   
    if (getScore === 1) {
      return 11;
    } else if (getScore === 2) {
      return 2;
    } else if (getScore === 3) {
      return 3;
    } else if (getScore === 4) {
      return 4;
    } else if (getScore === 5) {
      return 5;
    } else if (getScore === 6) {
      return 6;
    } else if (getScore === 7) {
      return 7;
    } else if (getScore === 8) {
      return 8;
    } else if (getScore === 9) {
      return 9;
    } else {
      return 10;
    }
  }
}

CodePudding user response：

Since your if statements are covering every number, you don't need an if statement for all of them.

Here I'm using a ternary function to see if the value is 1, which I will add 11, if not it adds the card's value.

Then I simply return the getStore variable.

let score = function (cardObject) {
  let getScore = 0;

  for (let i = 0; i < cardObject.length; i  ) {
    let cardValue = cardObject[i].value;
    getScore  = (cardValue === 1) ? 11:cardValue;
   }
   
  return getScore;
}

console.log(score([{ suit: 'HEARTS', value: 2 }, { suit: 'HEARTS', value: 3 }, { suit: 'HEARTS', value: 4 }]));

CodePudding user response：

You can use Array#reduce.

function score(arr) {
   return arr.reduce((acc, {value}) => acc   (value === 1 ? 11 : value), 0);
}
let res = score([{ suit: 'HEARTS', value: 2 }, { suit: 'HEARTS', value: 3 }, { suit: 'HEARTS', value: 4 }]);
console.log(res);

CodePudding user response：

For a simpler approach, that should do de trick:

function score(cardObject) {
  let getScore = 0

  for (let i = 0; i < cardObject.length; i  ) {
    switch (cardObject[i].value) {
      case 1: {
        getScore  = 11
        break
      }
      default: {
        getScore  = cardObject[i].value
        break
      }
    }
  }

  return getScore
}

score([{suit:"HEARTS",value:1}]) // 11
score([{suit:"HEARTS",value:2},{suit:"HEARTS",value:3},{suit:"HEARTS",value:4}]) // 9

CodePudding user response：

You actually don't need to do if num return num to get the number. But what's actually happening here is the use of the return function. Basically, it completely exits the function and gives the value back. So what you would really need to do instead is something like this

let score = function (cardObject) {
  let getScore = 0;

  for (let i = 0; i < cardObject.length; i  ) {
    //Instead of making it so that we return a bunch of numbers, we check to see if that
    //value from the cardObject array is a 1, and if it is add an 11 to getScore,
    //otherwise, add that number to getScore
    if (cardObject[i].value == 1){
      getScore  = 11;
    } else {
      getScore  = cardObject[i].value;
    }
  }
  //And then finally we return getScore instead
  return getScore;
}