Требуется ли вызов AsyncOperation.PostOperationCompleted?
Я использую класс AsyncOperation, чтобы не писать тонны методов «if (control.InvokeRequired) then / else» (в отличие от его традиционной роли в асинхронном шаблоне, основанном на событиях). В некоторых случаях меня действительно не волнует получение уведомления о завершении рабочего потока. Из-за этого я бы не хотел вызывать метод PostOperationCompleted на моем AsyncOperation, поскольку для этого мне потребовалось бы написать обратный вызов, который ничего не делает.
Мне любопытно, стоит ли опускать вызов PostOperationCompleted. В документации указано, что он завершает время существования операции и делает дальнейшие вызовы недействительными. При этом не уточняется, существует ли внутреннее поведение, связанное с этим вызовом, которое является жизненно важным. Это несколько намекает на то, что пропуск вызова может иметь внутренние последствия, но, в конце концов, я не вижу каких-либо серьезных рекомендаций, чтобы всегда его вызывать.
Итак, следует ли мне считать плохой практикой пропускать звонок, или в этом нет ничего страшного?
Ответы 2
В документации для PostOperationCompleted говорится: «Объект AsyncOperation гарантирует, что ваш делегат будет вызываться в потоке или контексте, подходящем для модели приложения». Это очень важно, если вашему потоку графического интерфейса нужно что-то сделать после завершения фоновой операции. Я думаю, что это, наверное, самая важная причина использовать этот класс в первую очередь!
Ваше здоровье, Дэн :)
В порядке. Что ж, я думаю, я говорю, что вам даже не нужно использовать этот класс, если вам все равно, когда рабочий поток завершится. Цель AsyncOperation - отслеживать время жизни фоновой операции. Если вам это не нужно, просто поставьте операцию в потоке пула потоков и забудьте об этом.
Имейте в виду ... Я действительно не знаю специфики вашего проекта, так что, возможно, мои комментарии действительно не применимы :)
... Еще один комментарий. Возможно, вы в своем коде не знаете, нужно ли вам публиковать сообщения, пока вы не выполните фоновую операцию? Вы можете просто вызвать OperationCompleted вместо PostOperationCompleted, если вам не нужно отправлять сообщения в поток графического интерфейса.
Оуэн,
Если вы знаете, что в вашей ситуации вам не требуется обратная маршрутизация в исходный поток вызывающих абонентов, вам следует просто вызвать OperationCompleted. Между вашей операцией и лежащим в ее основе SynchronizationContext существует неявный простой конечный автомат. Само по себе это не будет проблемой, если вы знаете, что после того, как ваша операция прекратилась, она больше не используется из остальной части вашего приложения. Хотя на самом деле это нормально, с точки зрения программирования необходимо учитывать следующие моменты:
Прежде всего, кто-то, кто приходит к коду с опытом использования этой конструкции, будет ожидать некоторого завершения асинхронной операции, поэтому это может сбивать с толку - вы должны четко указать это в коде.
Ключевым аспектом завершения операции является удаление объекта из списка финализатора. Для одного или двух экземпляров это может не быть проблемой, но вы наверняка получите снижение производительности, если количество этих операций будет большим, поскольку вы позволите всем этим экземплярам AsyncOperation получить повышение, и GC должен будет выполнять больше работы. Поэтому вам следует хотя бы вызвать OperationCompleted, чтобы избежать этого.
Вызывайте версию Post только в том случае, если вам это действительно нужно, потому что вы снова используете ресурсы, здесь в названии дополнительного задания, отправленного в пул потоков, что позже включает переключение контекста и обработку задания. Если не требуется, избегайте этого, особенно если пункт 2 действителен ... хотя для хорошей практики лучше всего вызвать соответствующую версию.
HTH
Ник.
Правильно, именно поэтому я прямо заявил, что моему пользовательскому интерфейсу все равно.