Fetch owinięty w new Promise

Question

Cześć,
Mam pytanie dotyczące fetch. Przerabiam pewien kurs i koleś zrobił coś takiego: 

class EasyHTTP {
    get(url) {
        return new Promise((resolve, reject) => {
            fetch(url)
                .then(response => response.json())
                .then(data => resolve(data))
                .catch(error => reject(error));
        });
    }
}

const http = new EasyHTTP;

http.get('https://jsonplaceholder.typicode.com/users')
     .then(data => console.log(data))
     .catch(reason => console.log(Error(reason)));

Czy istnieje jakiś powód, dla którego powinno się owijać fetch w new Promise? Przecież fetch sam w sobie zwraca promise. Przerobiłem to na coś takiego i w praktyce też działa:

class EasyHTTP {
    get(url) {
        return fetch(url)
            .then(response => response.json())
            .then(data => data)
            .catch(reason => Error(reason));
    }
}

const http = new EasyHTTP;

http.get('https://jsonplaceholder.typicode.com/users')
    .then(data => console.log(data))
    .catch(reason => console.log(Error(reason)));

 

À propos programowania obiektowego. Czy nie lepszym rozwiązaniem byłoby tutaj wykorzystanie statycznych metod skoro klasa i tak nie ma konstruktora?

Comments

Ten 1 fragment kodu to deferred antipattern

Answer 1

Czy istnieje jakiś powód, dla którego powinno się owijać fetch w new Promise?Przecież fetch sam w sobie zwraca promise. 

Raczej nie. Jeszcze fajniej byłoby to przepisać z async/await.

 Czy nie lepszym rozwiązaniem byłoby tutaj wykorzystanie statycznych metod skoro klasa i tak nie ma konstruktora?

Być może jeszcze lepszym byłyby zwykłe wolne funkcje ;)

Comments

Dzięki za odpowiedź  Czy coś takiego jest bardziej poprawne?

class EasyHTTP {
    static async get(url) {
        try {
            const response = await fetch(url);
            const json = await response.json();
            return json;
        } 
        catch(reason) {
            return Error(reason);
        }
    }
}

Co do wolnych funkcji to zawsze mam mindfuck. Teoretycznie dużo rzeczy można robić wolnymi funkcjami bez używania OOP. Tak naprawdę kiedy piszę coś sam z siebie to rzadko odczuwam potrzebę używania konstruktorów i innych cudów. Może za mało jeszcze zakodowałem, żeby używać ich w odpowiednich sytuacjach 

Answer 2

Cześć! Nie. Pozdrawiam!!