use nil? and empty?