как верно тут заметили главное придерживаться единого стиля...
лично я когда то давно на хабре увидел такую мысль:
если и условие и решение простое то пишем в одну строчку
if (view == null) continue;
если условие простое, а решение сложное то
if (isFreeze) {
setPrevEnabledView(null);
prevBlock = Block.EMPTY;
isRun = false;
if (timer != null) {
timer.cancel();
}
}
если и условие и решение сложные то
if ((objectEnabled = mapSettings.get(ATTRIBUTE_COLOR_BACKGROUND_ENABLED)) != null
| (objectDefault = mapSettings.get(ATTRIBUTE_COLOR_BACKGROUND_DEFAULT)) != null)
{
//body
}