Что должен удалить deleteItem?

Question

Документация для deleteItem гласит:

Если удаление асинхронное, метод должен вернуть обещание jQuery, которое будет разрешено после завершения удаления.

Что это действительно означает? Обещание должно вернуть какие-либо данные?

Будет ли работать следующее?

deleteItem: function(item) {
   return $.ajax("/api/members/", {
             method: "DELETE",
             data: { item: item},
             error: (err) => { console.log(err);},
             success: (result, status, jqXHR) => {},
         })
     }

Вышеописанное работает для меня при тестировании, но я не уверен, правильно ли просто игнорировать, если вызов Ajax успешен.

Answer 1

Если ваша бизнес-логика терпит неудачу при удалении элемента, тогда нормально не обрабатывать его.

В большинстве случаев это не тот случай, вам следует обработать сбой удаления элемента и правильно сообщить пользователю об этом сбое, возможно, с элегантным сообщением, чтобы попросить его попробовать позже.

О том, "что должен удалить deleteItem?" :

Он должен вернуть код состояния 200 в случае успеха с пустым телом. И он должен вернуть соответствующий код состояния http с телом ошибки, которое соответствует причине сбоя. например, если элемент, который вы пытаетесь удалить, не существует, то ожидается, что будет возвращена ошибка кода состояния 422 (Необработанный объект).

например, если вы не авторизованы для удаления этого элемента, возвращается код ошибки 403.

В целом, да, вы должны обработать ошибку и сообщить об этом пользователю.

Ваш код должен выглядеть примерно так:

deleteItem: function(item) {
   return $.ajax("/api/members/", {
             method: "DELETE",
             data: { item: item},
             error: (jqXHR , err) => { 
                if(jqXHR.status == 403){
                    console.log("handle forbidden error code);
                    alert("You are not authorized to delete this item, check 
                    with your manager...");
                }
              },
             success: (result, status, jqXHR) => {},
         })
     }