Что служба должна возвращать контроллеру в операциях CRUD?

Один из подходов состоит в том, чтобы сделать идемпотентными как метод службы, так и метод контроллера. В данном случае это означает, что «удаление» объекта возвращает один и тот же результат независимо от того, существует ли объект.

Во-первых, что возвращает сервис картошки? Не нужно ничего возвращать. Предполагая, что это асинхронно, это может быть

async Task Delete(potatoId, ct);

Возможны два исхода.

1. Метод не генерирует исключение. Либо элемент существовал и был удален, либо его не существовало. Результат одинаков в любом случае.
2. Метод выдает исключение.

Что возвращает контроллер? Он может вернуть 204 в случае успеха или 500 в случае сбоя удаления.

Думайте об этом как об обновлении. Предположим, вы меняли название картошки на «Отличная картошка». В случае успеха вы возвращаете 200 или 204.

Но что, если у картофеля уже было это имя? Вы бы вернули ошибку? Нет, вы бы вернули один и тот же результат, потому что независимо от того, меняется ли имя или нет, результат остается тем же. Теперь у него такое название.

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

Вам не нужно ничего возвращать из вашей службы в ваш контроллер, но для обработки различных сценариев с ошибками вы можете настроить свой контроллер с помощью блока try/catch и создать собственные исключения для различных сценариев в вашей службе. В этом случае у вас будет ответ 200 для успешных операций и ответ 400 с пользовательскими сообщениями для неудачных операций. Вот пример:

[HttpDelete("{potatoId:long}")]
public async Task<IActionResult> DeletePotato([FromRoute] long potatoId, CancellationToken ct)
{

    try
    {
        var result = _potatoService.Delete(PotatoId, ct);
        return Ok();
    {
    catch(Exception ex)
    {
        return BadRequest(ex.Message);
    }
}

В вашем сервисе (также может быть асинхронным):

public void Delete (potatoId, ct)
{

    // check if record exists

    if (recordExists) 
    {
    //proceed to delete
    }
    else
    {
       throw new Exception("Record does not exist");
    }
}

вы можете вернуться в соответствии с вашим результатом. но все же вы хотите вернуться как ОК, только тогда вы можете создать общую модель ответа с полями ниже

Public class ResponseModel
{
    public dynamic Result { get;set;} // to return any dataset optional
    public bool Status { get;set;}    // based on delete operation
    public string Message { get;set;} // Return your reason may of exception
}