janharders - 5:55 pm on Apr 7, 2011 (gmt 0)
What does / doesn't work?
Generally, I suggest you use CGI.pm to read values passed via CGI. Yes, it is a litle overhead, but you won't even notice it on todays' servers.
Also, a few comments in your code would be great. Just add a line like
# open file $user.ext to store comment
when you begin significant actions, so it's clear what your code should do. It might just be a typo or one line that's messing up.