Как говорили другие, это привычка, которую выучили у C, избегать опечаток - хотя даже в C я ожидал бы, что достойные компиляторы при достаточно высоких уровнях предупреждений выдадут предупреждение. Как говорит Чандру, сравнение с null в Java таким способом вызовет проблемы только в том случае, если вы используете переменную типа Boolean
(которой нет в примере кода). Я бы сказал, что это довольно редкая ситуация, и не та, для которой стоит менять способ написания кода в любом другом месте. (Я бы не стал обращать внимание на операнды даже в этом случае; если я достаточно ясно мыслю, чтобы рассмотреть возможность их изменения, я уверен, что могу посчитать знаки равенства.)
Что не было упомянуто, так это то, что многие люди (в том числе и я, конечно) находят if (variable == constant)
форму более читаемой - это более естественный способ самовыражения. Это причина не копировать слепо соглашение из C. Вы всегда должны подвергать сомнению практики (как вы делаете здесь :), прежде чем предполагать, что то, что может быть полезно в одной среде, полезно в другой.