簡化bash代碼

Question

我在shell腳本中有以下代碼。這一切工作正常。

#!/bin/bash 

baseDirPath = '/stackoverflow/question' 
newDir = '/stackoverflow/question/answers' 


# first check some business condition 
if [[ some condition here ]] 
then 
    # check if base Directory path already exit 
    if [ -d $baseDirPath ];then 

     # check if new Directory exits or not, if not create one 
     if [ ! -d $newDir ];then 

      mkdir $newDir 
      if [ $? -ne 0 ] ; then 
      echo "error occurred while creating directory " 
      fi 
     fi 
    else 
     exit 1; 
    fi 
fi 

這是非常混亂，不是很乾淨的代碼，我覺得。 我對編程非常陌生，所以不確定它是如何清潔的。

我很好奇，如果它可以變得更簡單或有一些其他方式來做到這一點。

（完整的shell腳本並不如上圖所示，只是複雜的if-else部分所示）。

Answer 1

#!/bin/bash 

die(){ >&2 printf '%s\n' "$@"; exit 1; } 

#Can't have spaces here 
baseDirPath='/stackoverflow/question' 
newDir='/stackoverflow/question/answers' 


# first check some business condition 
if [ some condition here ]; then 
    # check if base Directory path already exit 
    mkdir -p "$newDir" || die 'error occured while creating directory' 
fi 

這會稍微改變語義 - 它退出如果newDirs創造不管是什麼原因失敗 - baseDirPath不是目錄或baseDirPath是不能創建的目錄和newDir。

你也可以擺脫那個錯誤信息。 mkdir就已經給你在stderr上一個錯誤，如果它由於某種原因失敗：

mkdir -p "$newDir" || exit 1 

如果你的大多數命令應該像這樣工作（即成功或搞垮整個腳本），那麼你很可能最好設置set -e（〜退出時具有非零狀態的命令返回），然後就做：

mkdir -p "$newDir" 

Answer 2

這可以很徹底簡化：

#!/bin/bash 
baseDirPath='/stackoverflow/question' 
newDir='/stackoverflow/question/answers' 

# first check some business condition 
if [[ some condition here ]]; then 
    if ! mkdir -p "${newDir}"; then 
     echo "Unable to create directory ${newDir}. Aborting." 
     exit 1 
    fi 
    # Proceed as normal 
fi 

如果你真的需要爲baseDirPath存在，那麼肯定可以添加一個條件：

if [[ ! -d "${baseDirPath}" ]] || ! mkdir -p "${newDir}"; then