Как использовать функцию Promise.all () - PullRequest
Новая
       18

Как использовать функцию Promise.all ()

0 голосов
/ 12 января 2019

Я создаю университетский проект для отображения событий, происходящих в Лидсе. Я почти завершил эту страницу, которую я делаю, используя EventbriteAPI. Тем не менее, в данный момент моя страница не загружается, пока не выясняются все названия мест проведения мероприятия, что занимает довольно много секунд.

Я считаю, что для этого мне нужно использовать функцию Promise.all (), но я изо всех сил пытаюсь понять, как ее реализовать, поскольку я довольно новичок в этом.

eventbrite.js 

class EventBrite {
    constructor() {
        this.auth_token = 'Example';
        this.orderby = 'date';
        this.city = 'Leeds';
    }

    //Load Leeds events from API
    async loadAPI() {
        const initalEvents = await fetch(`https://www.eventbriteapi.com/v3/events/search/?sort_by=${this.orderby}&location.address=${this.city}&token=${this.auth_token}`);

        const events = await initalEvents.json();

        return {
            events
        }
    }

    async loadVenue(venueID) {
        const initalVenue = await fetch(`https://www.eventbriteapi.com/v3/venues/${venueID}/?token=${this.auth_token}`);

        const venue = await initalVenue.json();

        return venue;
    }
}

app.js 

eventbrite.loadAPI()
    .then(async events => {
        //Check for events
        const eventsList = events.events.events;

        //Print venue IDs
        for(let i = 0, l = eventsList.length; i < l; i++) {
            const venueID = eventsList[i].venue_id;

            let venue = await eventbrite.loadVenue(venueID);
            eventsList[i].venue = venue.name
        }
        ui.displayEvents(eventsList);
    }
)

ui.js 

class UI {
    constructor() {
        this.init();
    }
    init() {
        this.printCategories();

        this.result = document.getElementById('result');
    }

    //Display events
    displayEvents(events) {   
        //Build template
        let HTMLTemplate = '';
        events.forEach(eventInfo => {
            HTMLTemplate += `
                <div class="row">
                    <div class="column">
                        <p class="event">${eventInfo.name.text}</p>
                        <p class="event">${eventInfo.venue}</p>
                        <p class="event">${eventInfo.start.local}</p>
                    </div>
                </div>
            `;
        });
        this.result.innerHTML = HTMLTemplate;
    }
}

1 Ответ

0 голосов
/ 12 января 2019

Одна проблема состоит в том, что вы делаете все эти вызовы последовательно, что означает, что время для их завершения складывается.

Лучше запустить их в паралеле: 

eventbrite.loadAPI()
    .then(async events => {
        let promises = events.events.events.map(async (eventInfo) => {
            let venue = await eventbrite.loadVenue(eventInfo.venue_id);
            eventInfo.venue = venue.name;
            return eventInfo;
        });

        const eventsList = await Promise.all(promises);

        ui.displayEvents(eventsList);
    });

Если ответы все еще возвращаются медленно и с большой разницей между первым и последним, вы можете отобразить промежуточные результаты: 

eventbrite.loadAPI()
    .then(events => {
        const eventsList = events.events.events;

        eventsList.forEach(async (eventInfo) => {
            let venue = await eventbrite.loadVenue(eventInfo.venue_id);
            eventInfo.venue = venue.name;

            ui.displayEvents(eventsList.filter(item => item.venue));
        });
    });
...