Смотрите.
Для каждой переменной вы должны знать, она имеет право быть null
в вашей программе или нет. Это центральное соображение.
Теперь, для переменных, которые находятся внутри вашего модуля и не доступны снаружи, и которые, по вашему замыслу, не могут иметь значение null
, вы должны обеспечить выполнение этого при каждом присваивании. То есть, если вы присваиваете значение из другой переменной, которая потенциально может быть null
, вы должны проверять.
Да, и если значение приходит извне вашего модуля — например, это входной параметр, который не должен быть равен null
— вы всё равно должны проверить «на входе», один раз, и в случае чего бросить исключение, потому что вызывающий код может и ошибиться.
Если к вам приходит не параметр, а вы вызываете чужой метод и берёте его возвращаемое значение, и в документации указывается, что этот метод не возвращает null
, обычно дополнительные проверки не устраивают, т. к. возвращённый null
— баг на стороне чужого модуля. Впрочем, если вы не доверяете коду чужого модуля, то можно добавить проверки тоже.
До остальных переменных, наоборот, вы должны проверять только при доступе «внутрь», на чтение, при попытке получить значение роля или вызвать метод.
Что считать здесь «модулем», зависит от вас. Часто это один или несколько тесно связанных классов. Но вы можете сделать модуль и крупнее (вам придётся больше держать в голове, но меньше писать обвязочного кода), или мельче, как вам удобно.