当找不到物品时，带有计数器的bash循环的数量为1。为什么？

Question

在下面的功能中，只要在$ dt_files中找到一个项目，就可以正常工作。如果该文件夹的发现为空，则计数器的计数为1而不是0。我不确定我缺少什么。

脚本在这里所做的是1）制作一个包含所有父文件夹的变量。 2）循环浏览每个文件夹，每个文件夹在每个文件夹中，并列出包含字符串“ -dt-”的所有文件的列表。 3）如果它找到一个不以“ .tif”结尾的文件，则它将复制DT文件并将.TIF扩展程序放置在其上。非常简单。

我计算循环确实创建一个带有“ .tif”扩展名的新文件的次数。

因此，我不确定为什么有时会得到1个。

function create_tifs()
{
    IFS=$'\n'

    # create list of main folders
    LIST=$( find . -maxdepth 1 -mindepth 1 -type d )

    for f in $LIST
    do
        echo -e "\n${OG}>>> Folder processed: ${f} ${NONE}"
        cd ${f}

            DT_FILES=$(find . -type f -name '*-DT-*' | grep -v '.jpg')
            if (( ${#DT_FILES} ))
            then
                count=0
                for b in ${DT_FILES}
                do
                    if  [[ "${b}" != *".tif" ]]
                    then
#                        cp -n "${b}" "${b}.tif"
                        echo -e "TIF created ${b} as ${b}.tif"
                        echo
                       ((count++))
                    else
                        echo -e "TIF already done ${b}"

                    fi
                done
            fi

            echo -e "\nCount = ${count}"
}

Answer 1

我无法再现您的问题，但是您的代码包含几个可疑构造。以下是重构，可能还可以消除您遇到的任何问题。

#!/bin/bash

# Don't use non-portable function definition syntax
create_tifs() {
    # Don't pollute global namespace; don't attempt to parse find output
    # See also https://mywiki.wooledge.org/BashFAQ/020
    local f
    for f in ./*/; do
        # prefer printf over echo -e
        # print diagnostic messages to standard error >&2
        # XXX What are these undeclared global variables?
        printf "\n%s>>> Folder processed: %s %s" "$OG" "$f" "$NONE" >&2

        # Again, avoid parsing find output
        find "$f" -name '*-DT-*' -not -name '*.jpg' -exec sh -c '
            for b; do
                if  [[ "${b}" != *".tif" ]]
                then
#                   cp -n "${b}" "${b}.tif"
                    printf "TIF created %s as %s.tif\n" "$b" "$b" >&2
                    # print one line for wc
                    printf ".\n"
                else
                    # XXX No newline, really??
                    printf "TIF already done %s" "$b" >&2
                fi
            done
        fi' _ {} +
    # Missing done!
    done |
    # Count lines produced by printf inside tif creation
    wc -l |
    sed 's/.*/Count = &/'
}

可以通过使用查找./*/而不是通过f来进一步简化这。同样，您可以添加-not -name'*.tif'，但是随后您无法为这些打印“ TIF已经完成”。

切线也许也可以参见正确的bash and shell script脚本可变资本化;为私人变量使用较低的外壳。

在您的实际消息（例如第一个printf）之前打印newline 是一个奇怪的对抗图案，尤其是当您不这样做时。通常的安排是在​​每个发出的消息的末尾放置一个新线。

Answer 2

如果您有bash 4.0或以后，则可以使用globstar而不是（容易出错）查找。尝试此 shellCheck -clean Code：

#! /bin/bash -p

shopt -s dotglob extglob nullglob globstar

function create_tifs
{
    local dir dtfile
    local -i count
    for dir in */; do
        printf '\nFolder processed: %s\n' "$dir" >&2

        count=0
        for dtfile in "$dir"**/*-DT-!(*.jpg); do
            if [[ $dtfile == *.tif ]]; then
                printf 'TIF already done %s\n' "$dtfile" >&2
            else
                cp -v -n -- "$dtfile" "$dtfile".tif
                count+=1
            fi
        done
        printf 'Count = %d\n' "$count" >&2
    done

    return 0
}

• shopt -s shopt -s ...启用一些bash代码要求的设置：
  • dotglob使Globs能够匹配以。开头的文件和目录。 查找默认情况下显示此类文件。
  • extglob启用“扩展地球布”（包括！（*。jpg）等模式）。参见 extglob glob“ rel =“ nofollow noreferrer”> glob -Greg的Wiki 。
  • nullglob使Globs在没有匹配的情况下将其扩展到无与伦比（否则它们扩展到Glod模式本身，这几乎在程序中从来都不是有用的）。
  • globstar启用**可以通过目录树递归匹配路径。
• 请注意，globstar在4.3之前的bash版本中可能是危险的，因为它遵循符号链接，可能导致多次处理相同的文件或目录，或者陷入周期。
• -v带有cp的选项使其打印其所做的详细信息。您可能更喜欢丢弃该选项，而是打印另一种消息格式。
• 请参阅被接受的且极好的答案，对为什么printf胜于echo？ 为什么要解释为什么我使用<<代码> printf 而不是echo。
• 我没有使用CD，因为它通常会导致程序中的问题。