0

我对以下脚本有疑问。由于我对 BASH 缺乏了解，我不能 100% 确定如何创建一个循环，该循环使用“for”来计算数组中变量的数量并循环遍历每个变量。基本上我想要做的是让脚本计算变量 $sqldumps 中的变量数量，然后为每个变量运行一个 MySQL Dump（也将它们记录到数据库中）。脚本的长版本发布在底部，但下面是一个短版本：

#! /usr/bin/env bash
echo -ne '\033]0;World of Clucky - Backup\007'
BINDIR="$(dirname "$(readlink -fn "$0")")"
cd "$BINDIR"
sqldumps=("lwc" "bans" "frisnuk_permissions" "jail" "imonies" "GriefP" "permissions" "showcase" "simpleclans")
#Copy MySQL Databases
echo "[`date '+%H:%M'`] Starting MySQL Backup" >> /home/clucky/MinecraftServers/backup.log
for i in ${#sqldumps[*]}
    do
        echo "     ${sqldumps[i]}" >> /home/clucky/MinecraftServers/backup.log;
        /opt/lampp/bin/mysqldump -u'root' -p'CENSORED' --skip-lock-tables ${sqldumps[i]} > /home/clucky/MinecraftServers/.backups/$timedate/MySQL/${sqldumps[i]}.sql;
    done
echo "     forum";
/opt/lampp/bin/mysqldump -u'root' -p'CENSORED' forum >> /home/clucky/MinecraftServers/.backups/$timedate/MySQL/forum.sql;
echo "[`date '+%H:%M'`] MySQL Backup Complete" >> /home/clucky/MinecraftServers/backup.log

我遇到的问题尤其是：

for i in ${#sqldumps[*]}

我假设我为此使用了错误的语法，所以如果有人能给我正确的语法，将不胜感激。谢谢！

#! /usr/bin/env bash
echo -ne '\033]0;World of Clucky - Backup\007'
BINDIR="$(dirname "$(readlink -fn "$0")")"
cd "$BINDIR"
timedate=`date '+%m.%d.%Y-%H:%M'`
sqldumps=("lwc" "bans" "frisnuk_permissions" "jail" "imonies" "GriefP" "permissions" "showcase" "simpleclans")
echo "-------------- `date '+%d-%B-%Y %H:%M'` --------------" >> /home/clucky/MinecraftServers/backup.log
echo "[`date '+%H:%M'`] Starting Minecraft Backup" >> /home/clucky/MinecraftServers/backup.log
mkdir -p /home/clucky/MinecraftServers/.backups/$timedate
mkdir -p /home/clucky/MinecraftServers/.backups/$timedate/MySQL
nice --adjustment=19 tar -zcpf /home/clucky/MinecraftServers/.backups/$timedate/Minecraft.tar.gz \
    --directory /home/clucky/MinecraftServers/ \
    --exclude=SkyBlockkit \
    --exclude=Tekkit \
    --exclude=Vanilla \
    --exclude=test \
    --exclude=Zflocco_LP \
    --exclude=Mortuus/plugins/dynmap/web --exclude=Mortuus/plugins/AutoSaveWorld/backups --exclude Mortuus/CraftBukkitVersion --exclude=Mortuus/orebfuscator_cache --exclude=Mortuus/plugins/#Removed --exclude=Mortuus/server.log \
    --exclude=Frisnuk/plugins/dynmap/web --exclude=Frisnuk/plugins/AutoSaveWorld/backups --exclude Frisnuk/CraftBukkitVersion --exclude=Frisnuk/orebfuscator_cache --exclude=Frisnuk/plugins/#Removed --exclude=Frisnuk/plugins/#AwaitingUpdate --exclude=Frisnuk/server.log --exclude=Frisnuk/Suwako_Moriya100 --exclude=Frisnuk/Uk_Shadow --exclude=Frisnuk/Whiteghost99 --exclude=Frisnuk/greenrangermatt \
    --exclude=dailybackup.log \
    --exclude=.backups \
    --exclude=backup.log \
    --exclude=backups.log \
    --exclude=backupscript.sh \
    --exclude=dailybackup.sh \
    --exclude=.keptbackups .
echo "[`date '+%H:%M'`] Minecraft Backup Complete" >> /home/clucky/MinecraftServers/backup.log
#Purge files 3 days old
echo "[`date '+%H:%M'`] Purging Old Backups" >> /home/clucky/MinecraftServers/backup.log
find /home/clucky/MinecraftServers/.backups* -mmin +4320 -exec rm --recursive {} \;
echo "[`date '+%H:%M'`] Purging Complete" >> /home/clucky/MinecraftServers/backup.log
#Copy MySQL Databases
echo "[`date '+%H:%M'`] Starting MySQL Backup" >> /home/clucky/MinecraftServers/backup.log
for i in ${#sqldumps[*]}
    do
        echo "     ${sqldumps[i]}" >> /home/clucky/MinecraftServers/backup.log;
        /opt/lampp/bin/mysqldump -u'root' -p'CENSORED' --skip-lock-tables ${sqldumps[i]} > /home/clucky/MinecraftServers/.backups/$timedate/MySQL/${sqldumps[i]}.sql;
    done
echo "     forum";
/opt/lampp/bin/mysqldump -u'root' -p'CENSORED' forum >> /home/clucky/MinecraftServers/.backups/$timedate/MySQL/forum.sql;
echo "[`date '+%H:%M'`] MySQL Backup Complete" >> /home/clucky/MinecraftServers/backup.log
echo "[`date '+%H:%M'`] Daily Backup Complete" >> /home/clucky/MinecraftServers/backup.log
#Read back file size
filesize=$(ls -lah /home/clucky/MinecraftServers/.backups/$timedate | awk '{ print $5}')
echo "     Total Compression Size: $filesize\n" >> /home/clucky/MinecraftServers/backup.log

谢谢您的帮助！

4

4 回答 4

2

不要索引数组，只需使用值：

for value in "${sqldumps[@]}"; do echo $value; done
于 2013-03-13T23:24:03.703 回答
2

该语法${#varname[*]}返回数组的大小，但for您使用的语法希望得到一个元素列表。所以，要么你想要：

for elem in "${varname[@]}"; do
    ... something with ${elem} ...
done

或者

for (( i = 0; i < ${#varname[@]}; i++ )); do
    ... something with ${varname[$i]} ...
done

请注意，我使用@而不是，*因为这通常可以很好地预防包含空格的元素（即使在这种情况下它可能不是绝对必要的）。

于 2013-03-13T23:24:16.483 回答
0

我冒昧地重新组织了您的脚本以使其更具可读性。

#!/usr/bin/env bash

# Script starts here.
main() {
    backup_dir=~clucky/MinecraftServers
    dbs=(lwc bans frisnuk_permissions jail imonies GriefP permissions showcase simpleclans forum)

    echo -ne '\033]0;World of Clucky - Backup\007'

    log 'Starting MySQL Backup'
    dump_dbs "${dbs[@]}"
    log 'MySQL Backup Complete'
}

# Write a timestamped message to the log file.
log() {
    echo "[$(date '+%H:%M')] $*" >> "$backup_dir/backup.log"
}

# Dump each of the databases passed as an argument. The list of databases is "$@".
dump_dbs() {
    for db in "$@"; do
        # Use --skip-lock-tables for every database except "forum".
        local options=(--skip-lock-tables)
        [[ $db = forum ]] && options=()

        log "     $db"
        /opt/lampp/bin/mysqldump -u root -p CENSORED "${options[@]}" "$db" \
            > "$backup_dir/.backups/$timedate/MySQL/$db.sql"
    done
}

main "$@"

以下是我所做的一些事情：

  1. 像这样的循环将直接遍历每个值，而不是遍历数组索引：

    for value in "${array[@]}"; do ...; done

  2. ~user用来指代用户的主目录会更短。它也更安全，因为用户的主目录不一定在/home.

  3. 将代码分解为单独的函数有助于消除一些冗余，例如backup.log重复编写完整路径。log() 函数将带时间戳的消息行放在一个位置。

  4. 该forum表没有--skip-lock-tables选项，所以我更明确地说明了这使得摆脱重复mysqldump行成为可能。

  5. 我从不需要的地方删除了引号。

于 2013-03-13T23:26:49.290 回答
0

$i将每个项目保存在一个数组中，直到它到达末尾。因此，您不必每次在for循环中需要它时都引用该数组。我也不喜欢硬编码echo，这是个人喜好，所以我把它换成了printf.

for i in ${#sqldumps[@]}
  do
    printf "     %s" "$i" >> /home/clucky/MinecraftServers/backup.log;
    /opt/lampp/bin/mysqldump -u'root' -p'CENSORED' --skip-lock-tables $i > /home/clucky/MinecraftServers/.backups/$timedate/MySQL/${sqldumps[i]}.sql;
done

如果您只想找到可以使用的数组的大小。

Length=${#sqldumps[@]}

然后$Length将等于数组中的位置数

于 2013-03-13T23:44:39.427 回答