This is the fourth time you've been told to use warnings; use strict; (1, 2, 3). use warnings; use strict; is also the first item on the Basic debugging checklist. The second item on the Basic debugging checklist, printing your values, would have shown you that your original assumption "the values are not empty" was wrong.