Since you are doing this to get rid of blanks, I will assume that filename contains blanks: "no such file", for example. Thus,
Code:
mv $filename $newfilename
becomes
Code:
mv no such file no_such_file
Which won't work. You need to say
Code:
mv "$filename" $newfilename