Объекты в циклах и возвращаемые значения - PullRequest
0 голосов
/ 14 декабря 2011

У меня есть функция, которая похожа на эту:

public sub TestFunction() As Boolean

On Error GoTo NewError:

Dim testObject As New TestObject

For TestVaiable 0 to 1000
  TestObject.TestMethod(TestVariable)

Next

TestFunction = True
Exit function
NewError:
TestFunction = False
End Function

У меня есть два вопроса: 1) Это плохая практика - повторное использование объекта в цикле?Я не думаю, что это 2) Является ли плохой практикой возвращать логическое значение (false), если есть ошибка?

1 Ответ

1 голос
/ 14 декабря 2011

Хорошей практикой является повторное использование переменной в цикле.

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

Плохо практиковать повторное использование объекта таким образом, чтобы было труднее увидеть, что с ним происходит.

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

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

Плохая практика - не добавлять в этот комментарий менее 200 символов, объясняющих, почему можно просто возвращаться, когда произошла эта ошибка.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...