1

Helo everybody,

I have build this function to convert the fetch data of an Api to an array of objects, but linters are requesting me to use array destructuring.

can jou please help me? im just a begginner so please be nice!

export const organiceData = async () => {
  const data = await getData();
  const toArr = Object.entries(data);
  let newBook;
  const arr = [];
  toArr.forEach((book) => {
    newBook = book;
    newBook[1][0].id = book[0];
    arr.push(newBook[1][0]);
  });
  return arr;
};

8
  • What exactly did your linter tell you? Commented Jul 13, 2022 at 20:36
  • This is all it is telling me "error Use array destructuring prefer-destructuring" in this case it would be on line 8 of my code "newBook[1][0].id = book[0];" Commented Jul 13, 2022 at 20:37
  • 1
    It looks like it's asking you to replace newBook[1][0].id = book[0] with [newBook[1][0].id] = book. Commented Jul 13, 2022 at 20:42
  • no. it is still showing that error Commented Jul 13, 2022 at 21:01
  • I don't believe you. I tried it at validatejavascript.com and your lint error went away. foo = bar[0] can be replaced with [ foo ] = bar. Commented Jul 13, 2022 at 21:10

1 Answer 1

1

Ignore that particular warning message. But still you should fix some things in your code:

  • use map instead of forEach+push
  • get rid of the useless newBook alias - assignment is not creating a copy, and certainly not a deep copy of the object
  • book is not actually a book object, it's a tuple (two-element array) representing a key-value pair. The book object is the value - or rather, it appears to be contained in the first element of the value which is itself an array.
export async function organiceData() {
  const data = await getData();
  return Object.entries(data).map(pair => {
    pair[1][0].id = pair[0];
    return pair[1][0];
  });
}

Now we could also use destructuring instead of the pair variable, same as doing const key = pair[0], value = pair[1];:

export async function organiceData() {
  const data = await getData();
  return Object.entries(data).map(([key, value]) => {
    value[0].id = key;
    return value[0];
  });
}

If you want to create a copy of the object before putting an id property on it, as to not mutate it, you can use spread syntax in an object literal. We may also further use destructuring on the value (like const book = value[0];):

export async function organiceData() {
  const data = await getData();
  return Object.entries(data).map(([key, [book]]) => {
    return {...book, id: key};
  });
}
Sign up to request clarification or add additional context in comments.

Comments

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.