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

CodePudding user response：

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