tr/\W/_/

tr doesn't work that way... You'll have to stick with s/// if you want to use character classes. Also your logic will add a '.' on to the end of a file w/o an extension. i.e. "xyzfile" => "xyzfile."

-Blake