add null value check