Destructuring Objects and default parameters

Question

I have a problem to solve involving default parameters and object destructuring. I have an object 'product' with this shape:

{
     name: "Slip Dress",
     priceInCents: 8800,
     availableSizes: [ 0, 2, 4, 6, 10, 12, 16 ]
   }

Here is my code so far, but I am receiving an error that 'availableSizes' is not iterable. Can someone help me correct this code?

I have tried adjusting the default parameters in my function and I have moved my return statements to no avail.

function checkIfSizeIsAvailable(product = {availableSizes:[]}, size = 0) {
  // let availableSizes = product;
  let foundSize = "";
  for (let sizeCheck of product.availableSizes) {
    if (sizeCheck === size) {
      foundSize = size;
    }
  }
  if (foundSize === ""){
    return false;
  } else {
    return true;
  }
  //for (let i = 0; i < sizes.length; i++) {
  // return false;
}

There is no destructuring in your code. You just have a default parameter. Presumably you pass an actual value for product which doesn't have availableSizes — VLAZ
– VLAZ, Commented Jul 29, 2021 at 17:46

Kelvin Schoofs · Accepted Answer · 2021-07-29 17:49:43Z

1

As VLAZ mentioned in a comment, you can pass an object without an availableSizes field and it'll cause that error.

Destructuring happens when your variables together form an object/array on the left-hand size of the assignment:

function checkIfSizeIsAvailable(product = {availableSizes:[]}, size = 0) {
  // ^ Because of the default parameter, product always exists
  // Now we actually destructure with a default value for missing fields
  const { availableSizes = [] } = product;
}

or more compact:

function checkIfSizeIsAvailable({ availableSizes = [] } = {availableSizes:[]}, size = 0) {

}

Mind that this does not defend against non-array values, or even falsy values. A user can pass { availableSizes: false } and your availableSizes would also be false.

answered Jul 29, 2021 at 17:49

Kelvin Schoofs

8,7482 gold badges17 silver badges34 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

VLAZ Over a year ago

Or even more compact: function checkIfSizeIsAvailable({ availableSizes = [] } = {}, size = 0)

VLAZ Over a year ago

Also, the body of the function can be compacted to return availableSizes.includes(size)

master__mind Over a year ago

That worked! Thanks for helping me solve this one.

Collectives™ on Stack Overflow

Destructuring Objects and default parameters

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related