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; };
Advertisement
Answer
Ignore that particular warning message. But still you should fix some things in your code:
- use
map
instead offorEach
+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}; }); }