Что делать, если реализация интерфейса не обеспечивает асинхронность - PullRequest
Новая
       7

Что делать, если реализация интерфейса не обеспечивает асинхронность

0 голосов
/ 30 августа 2018

Допустим, у меня есть интерфейс: 

interface ILoader()
{
    Task<Data> LoadAsync(int id);
}

У меня есть две реализации этого интерфейса: 

class NiceDataBase : ILoader 
{
    public async Task<Data> LoadAsync(int id)
    {
        //this database has async way of fetching the data
        return await NiceDataBaseDriver.LoadDataAsync(id);
    }
}

class NotNiceDataBase : ILoader 
{
    public async Task<Data> LoadAsync(int id)
    {
        //this database's driver does not have aysnc methods, because it's old or done poorly.
        return await Task.Run(() => NotNiceDataBaseDriver.LoadData(id));
    }
}

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

Какой оптимальный способ справиться с такой ситуацией? Я думаю, что клиент NotNiceDataBase должен быть полностью осведомлен о том, что он делает, иначе он не сможет хорошо контролировать производительность приложения hiw.

Мой интерфейс может иметь дополнительный метод Load. Но в таком случае, какова реальная выгода здесь? Тем не менее, LoadAsync NotNiceDataBase должен иметь некоторую реализацию. Я думаю, что выбрасывание NotImplementedException никогда не является хорошим решением.

Ответы [ 2 ]

0 голосов
/ 30 августа 2018

Как вы знаете, async - это деталь реализации, и вы не можете определить async на интерфейсах. Так что все, что вам нужно сделать, это 

public class NotNiceDataBase : ILoader
{
    public Task<Data> LoadAsync(int id)
    {
        //this database's driver does not have aysnc methods, because it's old or done poorly.    
        var result = NotNiceDataBaseDriver.LoadData(id);
        return Task.FromResult(result);
     }
}
0 голосов
/ 30 августа 2018

Просто запустите метод синхронно и верните выполненное задание например, 

class NotNiceDataBase : ILoader 
{
    public Task<Data> LoadAsync(int id)
    {
        var data = NotNiceDataBaseDriver.LoadData(id);
        return Task.From(data);
    }
}

Вы говорите

Я думаю, что клиент NotNiceDataBase должен быть полностью осведомлен что он делает, иначе он не может контролировать производительность hi (s) приложение хорошо.

Я не согласен, , если производительность является проблемой и , если они определили, что это вызов NotNiceDataBase , тогда они могут что-то с этим сделать.

разработка

Время, волнующее о том, как клиент использует интерфейс, почти наверняка 1 время потрачено впустую, и не является событием "преждевременной оптимизации", но является опасностью существования по сравнению с кем-то, кто не использует неизвестную оптимизацию необходимо.

1 Если вы не являетесь вашим собственным клиентом, тогда все равно , вероятно, время потеряно.

...