И почему лучше в каких-то местах проверять на null и не проверять на NullTaxId?

Сколько видел кода - любое введение своих собственных обозначений для "налл", "нуль" и прочего заставляет проверять кроме стандартных обозначений ещё и свои собственные. Т.е. просто прибавляется работы.

Т.е. вместо привычных условных

if (null)

else if (0)

становится

if (null)

else if (0)

else if (empty)

else if (whateverAdditionalDesignation1)

else if (whateverAdditionalDesignation2)

...

Причём все варианты, кроме двух первых, обычно сводятся к двум первым. Т.е. все последующие проверки, кроме двух первых, избыточны.

По-моему, функция возврата айди должна возвращать айди. Если он может быть налловым по типу данных - пусть будет налловым. Функция возврата айди не знает контекста своего вызова (мы же все такие ООП, солиды, банды четырёх и прочие дяди Бобы?), значит контекст должен привноситься где-то снаружи неё - там, где эта функция вызывается. Вот по месту вызова и обрабатывайте этот налл - бросайте исключение (может даже кастомного типа), пишите содержательное сообщение, логируйте и прочее.