Bin ich über mit dem folgenden Code versprechen?

Question

Der Code funktioniert, aber ich frage mich, ob ich über vielversprechende bin.

Ich habe diese Redux Aktion

import Promise from 'bluebird'; 
const uploadAsynch = Promise.promisify(api.upload); 

uploadFiles : function(data, dispatch){ 
    var data = { 
     ep:"EP_UPLOAD", 
     payload: { 
     files: data.files, 
     profile: data.profile 
     } 
    } 
    uploadAsynch(data).then((result)=>{ 
     dispatch({type: FILES_UPLOADED}) 
    }); 
    }, 

api.upload ist die folgende

import axios from 'axios'; 

upload : function(data, callback){ 
    var files = new FormData(); 
    for(var i=0; i<data.payload.files.length; i++){ 
     files.append('files', data.payload.files[i], data.payload.files[i].name); 
    } 
    axios.post(apiEndpoints[data.ep], files, { 
     headers: { 
     'accept': 'application/json', 
     'Accept-Language': 'en-US,en;q=0.8', 
     'Content-Type': `multipart/form-data; boundary=--*`, 
     } 
    }) 
    .then((response) => { 
     callback(null, response) 
    }).catch((error) => { 
     callback(error) 
    }); 
    }, 

So frage ich mich. Wenn Axios ein auf Versprechen basierender Anforderungs-Client ist, ist es richtig, ihn mit bluebird in der Aktion zu verpacken?

Answer 1

Ich würde sagen, Sie sind unterversprechend, weil Sie Versprechen nicht in vollem Umfang nutzen. Sie greifen jedoch auf Callbacks zurück und promi- tieren gleichzeitig diese Funktion erneut, was ziemlich sinnlos ist. Nur return das Versprechen, das Sie in erster Linie hatten.

// no promisification 
uploadFiles: function(data, dispatch){ 
    var data = { 
    ep:"EP_UPLOAD", 
    payload: { 
     files: data.files, 
     profile: data.profile 
    } 
    } 
    return api.upload(data).then(result => 
//^^^^^^ 
    dispatch({type: FILES_UPLOADED}) 
); 
} 

import axios from 'axios'; 

upload: function(data) { // no callback 
    var files = new FormData(); 
    for (var i=0; i<data.payload.files.length; i++){ 
    files.append('files', data.payload.files[i], data.payload.files[i].name); 
    } 
    return axios.post(apiEndpoints[data.ep], files, { 
//^^^^^^ 
    headers: { 
     'accept': 'application/json', 
     'Accept-Language': 'en-US,en;q=0.8', 
     'Content-Type': `multipart/form-data; boundary=--*`, 
    } 
    }); 
}