Можно ли так использовать Task.FromResult и имеет ли это вообще смысл?

Question

[Maxim Maxim]

Попался мне на глаза вот такой код. Как я понимаю, человек хочет использовать синхронную операцию в асинхронном методе (первый вопрос.. зачем если есть другие методы с асинхронной поддержкой, но ладно, возможно это не ошибка), поэтому он использует Task.FromResult. Но зачем использовать await? FromResult же должен использоваться как раз для того, чтобы избежать создания машины состояний, которая появляется при async/await. Не будет ли эта связка await+TaskResult очень тяжёлой по производительности и бесполезной в контексте данной задачи метода репозитория?

Код

public async Task> GetUsers(string id)
{
return await Task.FromResult(Users.Where(x => x.Id == id));
}

Comments

[freeExec]

А что в ней тяжёлого?

[Роман]

Человек видать не в курсе о асинхронных методах linq, вот и написал так.

[Петр]

Тут обычное не понимание как правильно писать асинхронный методы. Человек пытался оставить метод с результатом Task<>, но асинхронных вызовов в его коде нет, вот и выкрутился.
Тут правильно либо делать метод без Task, либо убрать в определении слово async и отдавать в результат Task.FromResult без await

Answer 1

[Владимир Коротенко]

Не будет ли эта связка await+TaskResult очень тяжёлой по производительности

Проверьте 2 версии кода на тестах, что быстрее то и используйте :)

Comments

[Maxim Maxim]

Но каков смысл использования такой конструкции.. Я не нашёл в интернете причины
совместного использования await и Task.FromResult

[Maxim Maxim]

Мне кажется что a = await Task FromResult(x) это просто неэффективный способ написать a = x, не знаю

Answer 2

[Василий Банников]

По производительности будет +/- одно и то же, но вариант с await чуть памяти сожрёт.
В целом похоже больше на ошибку.