在线时间:8:00-16:00
迪恩网络APP
随时随地掌握行业动态
扫描二维码
关注迪恩网络微信公众号
这是我写BASH程序的招式。这里本没有什么新的内容,但是从我的经验来看,人们爱滥用BASH。他们忽略了计算机科学,而从他们的程序中创造的是“大泥球”(译注:指架构不清晰的软件系统)。 在此我告诉你方法,以保护你的程序免于障碍,并保持代码的整洁。 一、不可改变的全局变量 1.尽量少用全局变量 在我的程序中常使用的全局变量: 复制代码 代码如下: readonly PROGNAME=$(basename $0) readonly PROGDIR=$(readlink -m $(dirname $0)) readonly ARGS="$@" 二、一切皆是局部的 所有变量都应为局部的。 复制代码 代码如下: change_owner_of_file() { local filename=$1 local user=$2 local group=$3 chown $user:$group $filename for i in $files 1.自注释(self documenting)的参数 复制代码 代码如下: kfir@goofy ~ $ local a bash: local: can only be used in a function 三、main() 1.有助于保持所有变量的局部性 复制代码 代码如下: main() { local files="/tmp/a /tmp/b" local i for i in $files 四、一切皆是函数 唯一全局性运行的代码是: 1.保持代码整洁 复制代码 代码如下: main() { local files=$(ls /tmp | grep pid | grep -v daemon) } 复制代码 代码如下: temporary_files() { local dir=$1 ls $dir \ main() { 1.第二个例子好得多。查找文件是temporary_files()的问题而非main()的。这段代码用temporary_files()的单元测试也是可测试的。 2.如果你一定要尝试第一个例子,你会得到查找临时文件以和main算法的大杂烩。 复制代码 代码如下: test_temporary_files() { local dir=/tmp touch $dir/a-pid1232.tmp returns "$dir/a-pid1232.tmp" temporary_files $dir touch $dir/b-pid1534.tmp returns "$dir/a-pid1232.tmp $dir/b-pid1534.tmp" temporary_files $dir 如你所见,这个测试不关心main()。 五、调试函数 带-x标志运行程序: 复制代码 代码如下: bash -x my_prog.sh 只调试一小段代码,使用set-x和set+x,会只对被set -x和set +x包含的当前代码打印调试信息。 复制代码 代码如下: temporary_files() { local dir=$1 set -x 打印函数名和它的参数: 复制代码 代码如下: temporary_files() { echo $FUNCNAME $@ local dir=$1 ls $dir \ 调用函数: 复制代码 代码如下: temporary_files /tmp 会打印到标准输出: 复制代码 代码如下: temporary_files /tmp
六、代码的清晰度 这段代码做了什么? 复制代码 代码如下: main() { local dir=/tmp [[ -z $dir ]] \ [[ -n $dir ]] \ [[ -f $dir ]] \ [[ -d $dir ]] \ 让你的代码说话: 复制代码 代码如下: is_empty() { local var=$1 [[ -z $var ]] is_not_empty() { [[ -n $var ]] is_file() { [[ -f $file ]] is_dir() { [[ -d $dir ]] main() { is_empty $dir \ is_not_empty $dir \ is_file $dir \ is_dir $dir \ 七、每一行只做一件事 用反斜杠\来作分隔符。例如: 复制代码 代码如下: temporary_files() { local dir=$1 ls $dir | grep pid | grep -v daemon 可以写得简洁得多: 复制代码 代码如下: temporary_files() { local dir=$1 ls $dir \ 符号在缩进行的开始 符号在行末的坏例子:(译注:原文在此例中用了temporary_files()代码段,疑似是贴错了。结合上下文,应为print_dir_if_not_empty()) 复制代码 代码如下: print_dir_if_not_empty() { local dir=$1 is_empty $dir && \ echo "dir is empty" || \ echo "dir=$dir" } 好的例子:我们可以清晰看到行和连接符号之间的联系。 复制代码 代码如下: print_dir_if_not_empty() { local dir=$1 is_empty $dir \ 八、打印用法 不要这样做: 复制代码 代码如下: echo "this prog does:..." echo "flags:" echo "-h print help" 它应该是个函数: 复制代码 代码如下: usage() { echo "this prog does:..." echo "flags:" echo "-h print help" } echo在每一行重复。因此我们得到了这个文档: 复制代码 代码如下: usage() { cat <<- EOF usage: $PROGNAME options Program deletes files from filesystems to release space. It gets config file that define fileystem paths to work on, and whitelist rules to keep certain files. OPTIONS: Run specific test: Run: Just show what you are going to do: 注意在每一行的行首应该有一个真正的制表符‘\t'。 在vim里,如果你的tab是4个空格,你可以用这个替换命令: 复制代码 代码如下: :s/^ /\t/ 九、命令行参数 这里是一个例子,完成了上面usage函数的用法。我从http://kirk.webfinish.com/2009/10/bash-shell-script-to-use-getopts-with-gnu-style-long-positional-parameters/ 得到这段代码 复制代码 代码如下: cmdline() { # got this idea from here: # http://kirk.webfinish.com/2009/10/bash-shell-script-to-use-getopts-with-gnu-style-long-positional-parameters/ local arg= for arg do local delim="" case "$arg" in #translate --gnu-long-options to -g (short options) --config) args="${args}-c ";; --pretend) args="${args}-n ";; --test) args="${args}-t ";; --help-config) usage_config && exit 0;; --help) args="${args}-h ";; --verbose) args="${args}-v ";; --debug) args="${args}-x ";; #pass through anything else *) [[ "${arg:0:1}" == "-" ]] || delim="\"" args="${args}${delim}${arg}${delim} ";; esac done #Reset the positional parameters to the short options while getopts "nvhxt:c:" OPTION if [[ $recursive_testing || -z $RUN_TESTS ]]; then 你像这样,使用我们在头上定义的不可变的ARGS变量: 复制代码 代码如下: main() { cmdline $ARGS } main 十、单元测试 1.在更高级的语言中很重要。 复制代码 代码如下: test_config_line_paths() { local s='partition cpm-all, 80-90,' returns "/a" "config_line_paths '$s /a, '" config_line_paths() { echo $partition_line \ source /usr/bin/shunit2 这里是另一个使用df命令的例子: 复制代码 代码如下: DF=df mock_df_with_eols() { test_disk_size() { DF=mock_df_with_eols df_column() { $DF $disk_device \ disk_size() { df_column $disk_device 2 这里我有个例外,为了测试,我在全局域中声明了DF为非只读。这是因为shunit2不允许改变全局域函数。 |
请发表评论