RequireThis背后的动机是什么？

Question

Checkstyle warns about RequireThis

Personally, I find it more readable (especially when reading not from IDE, but from code review tools, for example), but are there other reasons for this check beside readability?

Answer 1

Consider the following:

int b,c;

public MyConstructor(int a, int b){
  b = a;
  c = b;
}

It's legal. It compiles. It probably doesn't do what you wanted it to. Adding this makes sure that you don't accidentally shadow something you didn't mean to.

Answer 2

It's just convention with some pros and cons.

This check is most useful if you do not use IDE. It may help for code review to disambiguate a code reference, improve readability and prevent confusion with local variable or parameter accesses.

On the other hand, most coding styles do not recommend using this everywhere, as this may be considered as unnecessary, redundant and too verbose.