why are we checking for empty string also?