foreach (<LIST>) ==> this is a bad idea (and bad practice)...

This is what happens if you use it: first a temporary list is created, and that list will be passed to foreach. This means you are first reading the entire file, it would be much better to read it line by line...

(Which you can do by using something like while (<LIST>)...)

(Update, typo)