3

我有一个执行 SQL SELECT 并使用输出发送一些电子邮件的脚本，但是脚本存在问题导致错误消息：

第 34 行：johne@mail.com：语法错误：算术运算符无效（错误标记为“@mail.com”）

第 34 行是：

if (( ${msgData[$email]:-0} < $(LC_TIME=C date -d yesterday +%s) )); then

这是脚本：

#!/bin/bash
${BASH_VERSION+shopt -s extglob lastpipe} 2>/dev/null

# Full path to a file to store our date offset data. Will be overwritten.
datefile=date.log

# Assign your vars or insert whatever code does so here.
user=user pass=pass db=db

function doit {
    typeset x
    for x; do
        if [[ -z $x ]]; then
            echo 'doit: missing an argument.' >&2
            return 1
        fi
    done

    typeset template=$(</dev/fd/4)
    exec 4<&-
    sqlplus "${1}/${2}@${3}" <&3 | {
        if [[ -f $datefile ]]; then
            . "$datefile" || return 1
        else
            # An associative array that maps from num -> timestamp
            typeset -A msgData
        fi

        # If we've successfully read the file containing timestamps, then overwrite with new data on RETURN
        ${msgData+trap 'trap RETURN; typeset -p msgData >"$datefile"' RETURN}

        while IFS=, read -r num email limit orders; do
            ${email:+:} continue
            if (( ${msgData[$email]:-0} < $(LC_TIME=C date -d yesterday +%s) )); then
                printf -- "$template" "email" "$num" "$limit" "$orders" |
                    /usr/sbin/sendmail -f sender@example.com -oi -t

                msgData[$email]=$(LC_TIME=C date +%s)
            else
                printf 'Mail already sent to %s within the last 24 hours... skipping.\n' "$email" >&2
            fi
        done
    }
} 5<&0 <<\SQL 3<&0 <<\TEMPLATE 4<&0 <&5-
set pagesize 0
set feedback 0

SELECT kred_lim.kunr ||','|| kust_adr.ku_email ||','|| kred_lim.kred_limit ||','|| kred_lim.kred_zu_zahlen
FROM kred_lim, kust_adr
WHERE kred_lim.kred_zu_zahlen > kred_lim.kred_limit
AND kred_lim.kunr = kust_adr.ku_nr;
SQL
Subject: Credit limit
To: %s

Customer number: %s
Credit limit: %s
Current orders: %s
TEMPLATE

if ! doit "$user" "$pass" "$db"; then
    echo 'we failed :(' >&2
    exit 1
fi
4

2 回答 2

3

问题是您试图在算术表达式中执行字符串比较。切换到复合表达式：

if [[ ${msgData[$email]:-0} < $(LC_TIME=C date -d yesterday +%s) ]]; then

或检查您是否正在访问正确的元素msgData。

于 2013-07-08T12:38:40.793 回答
2

在我看来，这是在 bash 中实现关联数组的一个限制（从这里的 v4.3.11 开始）。

文档说：

关联数组是使用创建的

declare -A name.

数组被分配给使用形式的复合赋值

name=(value1 value2 … )

其中每个值的形式为[subscript]=string。

然后你subscript之前查了几行：

下标被视为必须计算为数字的算术表达式。

所以它是说字符串“foo@bar”不能计算为数字，也不能用作数组键？但这并不能解释为什么没有“@”的字符串可以。

令人困惑的部分还在于，当您手动进行分配时，例如

declare -A emails_to_accounts=( \
    ["foo@bar.com"]="baz" \
    ["qux@bar.com"]="quux" \
)

for email in "${!emails_to_accounts[@]}"; do
    name=${emails_to_accounts[${email}]}
    : [...whatever...]
done

这很好用。打扰。

事实证明，对此的解决方案是重复declare -A分配中的部分，如下所示：

declare -A "hash_name+=([$email]=$whatever)"

在上面的特定示例中，很可能${msgData[$email]:-0}会在整数上下文中进行评估，因为您使用了<比较。

所以我打赌你想做一些事情，比如引用它来首先逃避那个上下文。

或者简单地用更少的速记来做，没有:-修饰符，当与该行上的所有其他标点符号结合时，这确实有助于产生线条噪音的印象。例如：

msgTime=${msgData[$email]}
test -n "$msgTime" || msgTime=0
if [ $msgTime -lt $(LC_TIME=C date -d yesterday +%s) ]; then
于 2016-04-05T09:41:22.177 回答