"test -f" checks to see if $j is a file. This is to avoid trying to copy "." and ".." from /etc/skel. If you have any directories in /etc/skel that you want copied, then this will be a problem. And as bzbee said, the fi is end if (if spelled backwards).
Before you change the echo's it is...