Well, tachyon gave you the answer that worked, so I really should butt out and let him answer. Nonetheless....

This line of code: @computers = join("\n",@computers);is creating a single array element with everything glued together (including the funky $). I'm not sure what its purpose is in your code, since you then try to traverse the array in the foreach loop that follows. You should either leave out that statement, or use it after you've chopped off the $.

HTH