复制时将文件夹名追加至文件名：含空格的文件夹名致循环失效

Hey there, I’ve run into this exact issue with finicky enterprise folder naming before—spaces and special characters like parentheses always trip up shell scripts that don’t account for them. Let’s get your copy loop working again.

The Root Problem #

Your original loop was failing because the shell splits arguments on whitespace by default. So when it hits a folder like Q3 Sales Data (2), it treats Q3, Sales, Data, (2) as separate items instead of a single folder path. The fix boils down to always wrapping path/filename variables in double quotes to preserve spaces and special characters.

Working Script with Space Handling #

Here’s an updated version of your loop that handles folders with spaces, (2) suffixes, and even empty folders (no .gz files):

# Traverse all folders in the external team's directory (including those with spaces)
for folder in ../external_team/*/; do
    # Extract the folder name, keeping spaces and special characters intact
    folder_name=$(basename "$folder")
    
    # Loop through each .gz file in the current folder
    for gz_file in "$folder"/*.gz; do
        # Skip if no .gz files exist in the folder (avoids "no such file" errors)
        [ -e "$gz_file" ] || continue
        
        # Get the original filename without the path
        file_name=$(basename "$gz_file")
        
        # Copy the file to your working directory, prepending the folder name
        cp "$gz_file" "./${folder_name}_${file_name}"
    done
done

Key Fixes Explained #

• Double quotes everywhere: Wrapping variables like "$folder" and "$gz_file" tells the shell to treat the entire string as a single path, even with spaces or parentheses.
• ../external_team/*/: Adding the trailing slash ensures we only target folders, not any stray files in the external team’s root directory.
• [ -e "$gz_file" ] || continue: Prevents errors if a folder has no .gz files—this checks if the file exists before trying to copy it.
• ${folder_name}_${file_name}: Using curly braces around variable names makes sure the shell doesn’t mix up the folder name and filename (avoids weird parsing issues).

Quick Test Tip #

If you want to verify what the script will do without actually copying files, replace the cp line with an echo command:

echo "Would copy: $gz_file to ./${folder_name}_${file_name}"

This lets you double-check that the folder names and filenames are being combined correctly before running the real copy.

内容的提问来源于stack exchange，提问作者thermophile