Ваш первый случай, если If (null != Amount || !ObjectUtils.isEmpty(Amount))
должен быть
if (null != Amount && null != Amount.getAmountToBePaid() && !ObjectUtils.isEmpty(Amount))
, так как вы будете использовать их позже, это гарантирует, что вы не получите null pointer exception
, пока ObjectUtils.isEmpty(...)
обрабатывает null
соответственно.
Если метод возвращает логическое значение, основанное на этих условных выражениях, я бы return true;
, если бы он прошел через все эти операторы if, return false;
в конце, чтобы выполнить ваш boolean
return
.