如何改进这个 FTP (shell) 功能？

Question

我有大量使用以下函数的脚本：

# Copies files over using FTP.
# Configurations set at the beggining of the script
# @param    $1 = FTP Host
#           $2 = FTP User
#           $3 = FTP User password
#           $4 = Source file name
#           $5 = destination directory
#           $6 = local directory
doftp() {
    log_message_file "INFO" "Starting FTP"

    ftp_hst=$1
    ftp_usr=$2
    ftp_pwd=$3
    sourcefile=$4
    destdir=$5
    locdir=$6

    ftp -nv $FTPH << EOF 2> ftp.err.$$
quote USER $ftp_usr
quote PASS $ftp_pwd
cd $destdir
lcd $locdir
bin

put $sourcefile

bye
EOF

    if [ "$(wc ftp.err.$$|cut -d" " -f8)" != 0 ] ; then
        log_message_file "ERROR" "Problem uploading files: $(cat  ftp.err.$$)"
    else
        log_message_file "INFO" "FTP finished"
    fi
    rm ftp.err.$$
}

它工作，它的工作，除非 ftp 失败。对我来说幸运的是，脚本非常精确，FTP 几乎从不失败。但这是难得的时刻之一，人们有机会（时间）返回并查看 TODO 列表中标记的代码。唯一的问题是我不太确定如何改进它......我会向你们提出关于在那里改变什么的建议。

一个明显的问题是从 ftp 解析错误，这完全是蹩脚的。但我也会考虑函数的其他部分:)

值得一提的是这是在 AIX 服务器上运行的吗？哦，不，我不能使用 SFTP :(

感谢您的任何意见！

ps.：log_message_file 只是一个基本的日志记录...对功能没有影响。

Answer 1

• 良好的文档
• 好的变量名
• 很好的缩进，你可能想了解
• 很好地使用带有 $$ 的 tmp 文件名（根据使用此函数的程度，您可能希望将父脚本名称附加为 tmp 名称的一部分，以进一步消除歧义，但优先级较低）

• 善用 $(cat ftp.err.$$) 即实际显示错误消息，而不仅仅是“发生错误”之类的消息（我一直看到这个，什么错误？什么是味精？！）

1234563检查您的 mput ${fileNames} 变量是否仍按预期工作。

• 可能要考虑改进的一个地方是使用 case 语句来解析 STDERR 输出，但同样，额外的好处可能不值得未来的维护成本。

  李>

errMsgs="$(cat ftp.err.$$)"

case "${errMsgs}" in
    *warningStrings* ) print "warning found, msg was ${errMsg} ;;
    *errorStrings* ) print "error found, msg was ${errMsg} ;;
    *fatalStrings* ) pring "fatal error found, can't continue, msg was ${errMsg} ;;
esac

我希望这会有所帮助。