Code style, single line statements и автоматический рефакторинг - Public forum of MSU united student networks

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

Имеется код, который некоторыми джавистами может считаться плохим:

code:if (disposed)
	throw new ObjectDisposedException("cannot invoke methods on a disposed service proxy");
if (method == AUTHORIZE_METHOD)
	throw new InvalidOperationException("the Authorize() method must not be called explicitly");

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

Так вот, если ошибки, связанные с {} возникают у нормального человека, то это как правило причина рефакторинга, причем автоматического. Человек может разбить выражение на два, и т.п. Для всего этого существуют автоматизированые средства.

Часто в code style, даже Джавовских, пишут, что при выкидывании исключений, возвратах из функций и простых однострочных if-ах скобки ставить вовсе необязательно. К сожалению, не всегда удается поместить подходящий под эти условия код на одну строку. Поэтому я считаю, что приведенный мною блок кода не имеет никаких проблем.

Еще скобки ставят для того, чтобы облегчить задачи code review. Когда человек будет просматривать код, ему может быть проще оценить такую характеристику, как cyclomatic complexity.

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

Редактировал DarkGray (30.01.2009 14:55)