# Delphinum (10.11.2013 / 18:00)Главный принцип должен быть такой: код максимально чистый, никаких паразитных переменных.
По поводу условия я частично согласен. По правде говоря есть очень много подобных правил, которые я (возможно еще) не включил в стандарт из за их сомнительности. Данное правило я намеренно указал след
---
Давай еще раз разберем кусок кода:
if(elementIndex < 0 || elementIndex > MAX_ELEMENTS || elementIndex == lastElementIndex){
ЪДля улучшения читаемости его при желании можно сделать многострочным (если условий много), но вот в последующем примере разбивка по переменным была сделана логически неверно.
Тут выражение простейшее, отлично читается и не нуждается даже в скобках.
Чтоб сразу же разобраться, достаточно помнить принцип действия логического орператора ||
А именно, следующая часть проверяется только тогда, когда предыдущая часть FALSE
И если мы вдруг разобьем все по переменным, визуально это намного ухудшится, ибо кроме паразитных переменных, теряется (визуально) логическая цепочка и мне приходится бегать по лишним выражениям, чтоб понять. откуда берется данная переменная.