while (<XLSFILE>) already reads the file line by line, no point in splitting on newlines again (there will be just one line, ending in a newline). If you want all lines in an array, just do: @accounts = <XLSFILE>;, without an explicite while.