Должны ли все асинхронные методы вызываться с ключом async?

Question

[Вадим]

Пытаюсь лучше понять как работает async/awai и Task в дотнете

Есть два следующих простых метода

public async Task<int> CalculateAsync(int x, int y)
{
    return await Task.Run(() => Calculate(x, y));
}

private int Calculate(int x, int y)
{
    // Simuliruyem dolgiy process
    Thread.Sleep(5000);
    return x + y;
}

Несколько вопросов по этому коду:
1) Нормально ли что у вызываемого Calculate отсутствует слово async, не приведет ли это к синхронному выполнению?
2) Стоит ли всегда использовать ключ Task при обьявлении асинхронного метода?

Answer 1

[Петр]

Calculate не является асинхронной функцией поэтому async не нужен. И очень плохо вызывать в асинхронном коде Thread.Sleep, для асинхронного кода есть Task.Delay.

async преобразует код в конечный автомат (State Machine)

Comments

[freeExec]

Тут это скорей создать видимость "нагрузки"

[Вадим]

@petermzg

так ведь вы сами сказали, что Calculate не является асинхронной функцией - значит в нем можно Thread.Sleep

[Петр]

Вадим, Да, не является. Но применяется внутри Task.Run, который и создает асинхронную функцию.

Answer 2

[rPman]

Асинхронность - это механизм многозадачности в пределах одного потока (.net умеет это делать и в пределах нескольких потоков), когда существует некий цикл, в пределах которого код (пользовательские функции) выполняются по очереди.

Чтобы асинхронность эффективно работала (как альтернатива threading), нужно чтобы эти кусочки кода исполнялись максимально короткое время. Если у тебя есть код, исполняющийся длительное время, то либо дели его на части (например есть цикл, то меняешь логику кода так чтобы кусками кода было содержимое цикла) либо используй threading.

async/await это синитаксический сахар, позволяющий красиво использовать асинхронный код в обычном стиле программирования. Когда компилятор видит async он сохраняет состояние (стек) и передает выполнение к очереди для других кусков кода (помеченный async)

Comments

[Вадим]

Ну и что можете обьяснить вышепревиденный код? Спасибо, но это все общие фразы, а здесь конкретный пример !

[rPman]

этот вопрос показывает что скорее всего ты не понял что такое асинхронность

твои вопросы не имеют смысла в контексте Thread.Sleep(5000); потому что этот кусок кода заблокирует весь поток на 5 секунд и никакие асинхронные операторы не помогут

[Петр]

rPman "Когда компилятор видит async он сохраняет состояние (стек) и передает выполнение к очереди для других кусков кода (помеченный async)"
Не делается это, async это директива для компилятора, чтобы из функции сделать State Machine

[rPman]

Петр, AsyncStateMachine сохраняет состояние, а именно все локальные переменные, исторически они на стеке выделяются.
p.s. я понимаю что есть ситуации когда переменные выделяются в куче, но поведение и отношение к ним в данном случае одинаковое

[Петр]

rPman, async/await не делает переключения контекста потока.
И все замыкаемые переменные сохраняются в обьект State Machine, который создается в куче, а не на стеке.

[rPman]

я говорил не про "где" сохраняет а про "что"

await должен привести к переключению контекста, как бы это не оформлено было в c#/clr но в конечном счете, он должен переключить к следующему контексту по очереди, который тригерится по событиям.

и я не понимаю что мы тут обсуждаем, автор вопроса совсем про другое спрашивал

p.s. по русски красиво в одном месте (первое что нагуглилось)