Рассмотрим эту строку:
if (object.getAttribute("someAttr").equals("true")) { // ....
Очевидно, что эта строка является потенциальной ошибкой, атрибут может быть null
, и мы получим NullPointerException
. Поэтому нам нужно реорганизовать его на один из двух вариантов:
Первый вариант:
if ("true".equals(object.getAttribute("someAttr"))) { // ....
Второй вариант:
String attr = object.getAttribute("someAttr");
if (attr != null) {
if (attr.equals("true")) { // ....
Первый вариант неудобен для чтения, но более кратким, в то время как второй четкий в намерении, но подробный.
Какой вариант вы предпочитаете с точки зрения удобочитаемости?