Business
Jobs
  • About Us
  • Solutions
    • Job Postings
      Post your job and receive qualified candidates in 48h.
    • Candidate Assessments
      500+ technical and psychological tests, plus anti-fraud.
    • Headhunting
      Tailor-made executive search from start to finish.
    • Payroll + EOR
      Payroll dispersal and EOR across 15+ LATAM countries.
  • Pricing
  • Jobs

0

111
Views
How to make async wait for result

I have a class like this (simplified):

export class Foo {

    constructor(el, menuItems ) {
        this.id = this.el.dataset.id;
        this.menuItems = menuItems;

        this.fetchResult = async function (uri) {

            fetch(uri, {
                method: 'get'
            })
                .then(response => response.json())
                .then(data => {
                    return data.saved === true;
                })
                .catch(function (err) {
                    console.log('Failed ', err);
                    return false;
                });
        }

    }

    doAction(action) {

        const uri  = 'foo/'+ action +'?id=' + this.id;

        const map = {
            'close': async () => {
                const success = await this.fetchResult(action);
                console.log(success); // says undefined? 
            }
            /* some other actions */
        }
        
        return map[action];
    }

    attachDialogs() {
        this.menuItems.addEventListener('click', (element) => {
            this.doAction( element.dataset.action )();
        });
    }    
}

For some reason the const success = await this.fetchResult(action); does not wait for the result. It immediately proceeds, resulting in and 'undefined' value.

In the background the fetch is being done. How can I make it wait for the result?

about 4 years ago · Juan Pablo Isaza
2 answers
Answer question

0

I believe the issue is that you are not resolving the response from your request, try returning a promise and resolving only when you have your response (or error)

this.fetchResult = function(uri) {
  return new Promise((resolve, reject) => {
    fetch(uri, {
      method: 'get'
    })
    .then(response => response.ok
      ? response.json()
      : reject(new Error(response.statusText)))
    .then(result => resolve(result))
  }
}


const map = {
  'close': async () => {
     const success = await this.fetchResult(action);
   }
}

Remove the async from fetchResult there is no reason for it. You're awaiting the result of the Promise in fetchResult, you only need the async keyword if fetch result is written in async await as well:

await res = fetch(url)
about 4 years ago · Juan Pablo Isaza Report

0

Thanks @Toto Nabendo and @ggorlen!

Working solution:

export class Foo {

    constructor(el, menuItems ) {
        this.id = this.el.dataset.id;
        this.menuItems = menuItems;

        this.fetchResult = async function (uri) {

            return fetch(uri, {
                method: 'get'
            })
                .then(response => response.json())
                .then(data => {
                    return data.saved === true;
                })
                .catch(function (err) {
                    console.log('Failed ', err);
                    return false;
                });
        }

    }

    async doAction(action) {

        const uri  = 'foo/'+ action +'?id=' + this.id;

        const map = {
            'close': async () => {
                const success = await this.fetchResult(action);
                console.log(success);  
            }
            /* some other actions */
        }
        
        return map[action];
    }

    attachDialogs() {
        this.menuItems.addEventListener('click', (element) => {
            this.doAction( element.dataset.action )();
        });
    }    
}
about 4 years ago · Juan Pablo Isaza Report
Answer question
Find remote jobs

Discover the new way to find a job!

Top jobs
Top job categories
Business
Post vacancy Pricing Sales
Legal
Terms and conditions Privacy policy
© 2026 PeakU Inc. All Rights Reserved.
Andres GPT
Show me some job opportunities
There's an error!