Home/ Questions/Q 7915353
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-06-03T14:25:25+00:00

sometimes I found that a few PMD rules conflict with each other, therefore you

  • 0

sometimes I found that a few PMD rules conflict with each other, therefore you can not write code that satisfies all PMD rules.

For example, it seems the following two rules exclude with each other:
“Assigning an Object to null is a code smell. Consider refactoring.”
and
“A method should have only one exit point, and that should be the last statement in the method”

The following is my example code:

enter image description here

if I use get1(), I will violate the former rule, and if I use get2(), then I will violate the latter rule. I prefer that A method should have only one exit point, but I don’t want PMD reports that “Assigning an Object to null is a code smell”, does anybody have any good idea? Thanks a lot 🙂

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    In both of these cases, I’d use the conditional operator:

    return condition > 5 ? Integer.valueof(123) : null;

    Personally I don’t try to stick to one exit point though. There are far too many times when that makes the code less readable – often you can tell at the start of the method what the return value should be (e.g. due to it being a base case)… so why not just return it? Code which dogmatically sticks to “only one exit point” typically ends up with unnecessarily nested code which is then harder to scour through.

    • 0