Как очистить обработку ошибок в функции?

Question

Я смотрю код, написанный кем-то другим.

Некоторые из больших функций - беспорядок, с изменениями в обработке ошибок во многих точках всей функции. Существует множество On Error Goto ErrHandler, On Error Goto 0, On Error Resume Next, когда вы проходите через эту функцию.

Что было бы лучшим способом пройти и очистить эту обработку ошибок, чтобы в верхней части функции было всего лишь On Error Goto ErrHandler?

Answer 1

Я не думаю, что наличие одного обработчика ошибок обязательно является лучшим решением.

Если у вас есть огромная функция с одним обработчиком ошибок, и что-то не так, вы понятия не имеете, в какой момент это не удалось.

Звучит так, как будто функции, которые вы описываете, имеют правильную обработку ошибок, но это не значит, что их легко читать.

Возможно, лучшим решением является re-factor большие функции в несколько более мелких, которые могут иметь свой собственный обработчик ошибок таким образом, когда возникает ошибка, у вас есть только несколько строк для отслеживания ошибки?

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

Answer 2

Трудно сказать, не глядя на код, о котором идет речь, но бывают случаи, когда несколько разных типов обработки ошибок в одной функции являются законными. В общем, один On Error GoTo ErrHandler в верхней части функции выполняет ту же функцию, что и закрытый блок try/catch на языке, который поддерживает исключения - он должен поймать неожиданные ошибки, которые возникают.

Однако в пределах функции могут быть места, где ошибка является нормальным ожидаемым появлением, которое вы хотите проверить для встроенного. Например, скажем, что вы переходите к сетевому диску, где вам приходится учитывать, что удаленный сервер находится в автономном режиме, сеть выключена и т. Д. В этом случае, хотя в верхней строке есть инструкция On Error GoTo ErrHandler, вы можете отключить это локально с On Error Resume Next и попытайтесь получить доступ к диску. Если возникла ошибка, VB6 теперь просто перейдет к следующему оператору, где вы можете проверить Err.Number на что угодно, кроме 0. Это аналогично вложенному try/catch, чтобы уловить ожидаемое условие ошибки, которое вы можете обрабатывать локально. После того, как выполнение перешло за пределы рискованного кода, вы должны пометить конец раздела, в котором вы собираетесь «вручную» проверить ошибки таким образом, с помощью другогоOn Error GoTo ErrHandler инструкции для повторного включения обработчика уровня функции.

Таким образом, есть случай, когда более одного счета On Error.... Аргумент может заключаться в том, что рисковая функция должна быть реорганизована и иметь собственный обработчик, но вам все равно придется иметь дело с конкретными (ожидаемыми) значениями возврата ошибки из этой отдельной функции, поскольку функция , называемая, вероятно, t знать, что делать перед лицом этих ошибок в контексте функции , вызывающей. Вы по-прежнему будете обрабатывать это с помощью краткосрочной техники On Error Resume Next.

On Error GoTo 0 disable все обработка ошибок, позволяющая VB6 освобождаться от вашей функции всякий раз, когда возникает непредвиденная ошибка.Мне трудно найти подходящую ситуацию, когда это было бы неплохо.