add null value checking