Question

我在bash脚本中编写了2个日志记录方法：

# Function to log information
function print_info() {
  format="$1"
  shift
    printf "$(date +'%F %T') ${SCRIPTNAME}: INFO: $format\n" "$@" | tee -a ${LOGFILE};
}

# Function to log error
function print_error() {
  format="$1"
  shift
    printf "$(date +'%F %T') ${SCRIPTNAME}: ERROR: $format\n" "$@" | tee -a ${LOGFILE} >&2;
}

并将几条消息定义为：

BACKUP_DIR_INFO="All the contents of directory %s are compressed to %s."
BACKUP_FILE_INFO="File %s is compressed at path %s."
ERROR_FILE_NOT_EXIST="File or directory %s does not exist."

从我的脚本中，我使用上面显示的2种方法：

function tarIt() {
  FILE_NAME=$1
  TAR_FILE_NAME=$2
  if ! ([ -f $FILE_NAME ] || [ -d $FILE_NAME ]);
  then
    print_error $ERROR_FILE_NOT_EXIST $FILE_NAME
  else
    if [[ -f $FILE_NAME ]]
    then
      print_info "$BACKUP_FILE_INFO" "$FILE_NAME" "$BACKUP_DIR"
    else
      print_info "$BACKUP_DIR_INFO" "$FILE_NAME" "$TAR_FILE_NAME"
    fi
  fi
}

我用两个文件名调用函数tarIt两次，其中一个存在而另一个不存在，但得到的输出如下所示：

2015-03-15 09:42:46 : ERROR: File
2015-03-15 09:42:46 : INFO: All the contents of directory /opt/mgtservices/relay/logs are compressed to /root/backup/relay_logs.tar.gz.

错误字符串未完全打印。

没有理由。甚至我试图将print_error方法用于其他消息，但只打印了字符串的第一个字。后来它忽略了。

Answer 1

正如@Biffen在评论中所说，在所有变量引用周围加上双引号。运行此命令时：

print_error $ERROR_FILE_NOT_EXIST $FILE_NAME

shell将其扩展为：

print_error File %s is compressed at path %s. /path/to/nonexistent/file

...因此print_error函数接收＆＃34;文件＆＃34;作为$ 1，使用它作为格式字符串，你得到无意义的输出。顺便说一句，将数据（日期和脚本名称）嵌入格式字符串也是一个坏主意;使用%s并将其添加到字段部分：

printf "%s %s: INFO: $format\n" "$(date +'%F %T')" "$SCRIPTNAME" "$@" | tee -a "$LOGFILE"

请注意，我还冒昧地引用了$ LOGFILE的引用。请养成将变量引用放在双引号中的习惯，除非有特殊原因不这样做。

printf bash奇怪的行为

1 个答案: