Converting a for loop function results into an array

Question

I am looping through an array of selected index comparing each value to a database of machine pricing, and returning the price of each selected index. the problem is, the result repData1 return individual results, I want those resuls to displayed in an array for I can manipulate the array. I have tried push, concat.... string results is displayed for each item rather than a whole.

for (let a = 0; a < selectedindex.length; a++) {
  wixData
    .query('MachinePricing')
    .contains('title', selectedindex[a])
    .find()
    .then(async (results) => {
      if (results.items.length > 0) {
        let repData = results.items;
        let repData1 = repData.map(({ prices }) => prices);
        console.log(repData1);
      }
    });
}

mplungjan · Accepted Answer · 2023-01-29 11:40:59Z

1

Don't loop async calls using iterators

Instead do this

const a = 0
const repData = [];

function getData = () => {
  if (a >= selectedindex) {
    processRepData();
    return;
  }
  wixData
    .query('MachinePricing')
    .contains('title', selectedindex[a])
    .find()
    .then(results => {
      if (results.items.length > 0) {
        repData.concat(results.items.map(({prices}) => prices));
      }
      a++;
      getData()
    });
}    
getData()

answered Jan 29, 2023 at 11:40

mplungjan

180k29 gold badges183 silver badges246 bronze badges

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

Comments

Moritz Ringler · Accepted Answer · 2023-01-29 12:32:30Z

1

I think what you are doing is this (run a query for each selected index and extract the returned prices into an array):

const queries = selectedindex.map(ix =>  wixData
    .query('MachinePricing')
    .contains('title', ix)
    .find())
const results = await Promise.all(queries)
const prices = results.flatMap(r => r.items.map(i => i.prices))

answered Jan 29, 2023 at 12:32

Moritz Ringler

16.4k10 gold badges29 silver badges45 bronze badges

3 Comments

Abdallah Mushi Nourdin Over a year ago

Interestingly it worked! 100% thanks!

mplungjan Over a year ago

This is a more elegant solution using await that my simpler version

mplungjan Over a year ago

This is a more elegant solution using await than my simpler version that does not use await at all

Collectives™ on Stack Overflow

Converting a for loop function results into an array

2 Answers 2

Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related