4

我有这个 bash 脚本，它可以粘贴一个文本文件，我想让它变得更简单和更短以提高效率，有人对我如何做到这一点有任何想法吗？

$vi

function displayHelp
{
        echo "Use '-f' to set the file to be used "
        echo "Use '-s' to sort the data bya column"
        echo "Use '-m' to output the rows which match this expression"
}

function displayColumn
{
    columnnumber="$2"
    awk '{print $'$columnnumber'}' $1

}

function displayParameter
{   
    parameter="$3"
    columnnumber="$2"
    awk -v s=$3 -v c=$2 '$c ~ s { print $0 }' $1
}

while getopts f:s:m:h opt
do
  case "$opt" in
    h) displayHelp;;
    f) filepath="$OPTARG";;
    s) column="$OPTARG"  
    displayColumn $filepath $column
    ;;
    m) searchParam="$OPTARG"
    displayParameter $filepath $column $searchParam
    ;;

  esac
done
4

1 回答 1

2

在displayColumn()中，您没有使用-v变量传递，但在displayParameter()您中。您应该始终使用-v而不是嵌入 shell 变量。

同样在displayParameter()你分配两个变量并且从不使用它们。

你应该总是在扩展 shell 变量时引用它们。当它们单独出现时，不必在作业的右侧引用它们。

-f如果这是您的意图，则不需要强制执行。如果不提供它（及其所需的参数），您的脚本将尝试从中读取stdin（这可能是您想要的）。

没有强制执行-m并且-s是必需的。如果两者都不存在，AWK 命令将输出错误消息。

通常，必需的参数应作为位置参数处理，选项标志应用于选项（及其参数）。

您的问题可能更适合Code Review。

于 2012-05-01T21:24:02.233 回答