Wouldn't it be better to check empty instead of isnull?