シェルスクリプトのread lineにおいて、最終行だけ処理を変える方法

Question

read lineにおいて、最終行だけ処理を変える方法をご教授下さい。

下記の通り、各行に数値が羅列されている
msg_id.txtがあります。

1
2
：
10

msg_id.txtの各行をカンマ(,)で結合した文字列：TARGET_MSG_IDを作りたいです。

下記のような処理では、1,2,…,10,となってしまいますが、
1,2,…,10と、文字列の最後にはカンマがつかないようにしたいです。

while分の中で、最終行だけ処理内容を変えるような
記述は出来るのでしょうか。

========================================
TARGET_MSG_ID=""
while read line
do
TARGET_MSG_ID="$TARGET_MSG_ID${line},"
done < msg_id.txt

Answer 1

readline は「今が最終行である」ことを検知できません。検知できるのは「もう取る行がない」だけです。
ですから最終行を判定するようなことをしようとしてはいけません。

, がついてほしくないならば、逆に考えて

• 最初に読み込んだ行ならばそのまま
• それ以外なら、',' を付けてから読み込んだ内容を付け加える

でもいいのです。
なのでこういう手が使えます。

TARGET_MSG_ID=""
SEPARATOR=""
while read line
do
  TARGET_MSG_ID="${TARGET_MSG_ID}${SEPARATOR}${line}" 
  if [ "${SEPARATOR}" = "" ]
    SEPARATOR=","
  fi
done

最初の行のときは SEPARATOR が "" ですから、何もつきません。
その直後の if 文で SEPARATOR が "," に変更されるので、2行目では [1行目の内容],${line} とくっつくようになります。

Answer 2

最後にカンマを削るのは？
TARGET_MSG_ID=${TARGET_MSG_ID:0:${#TARGET_MSG_ID}-1}

Comments

[otn]

TARGET_MSG_ID=${TARGET_MSG_ID%,}
で良いです。

[A.Ichi]

さすがですね。/でないので思い出せませんでした。

[otn]

# と % で、どっちがどっちか覚えにくかったのですが、キーボードの左の方にある # が先頭の削除、右の方にある % が末尾の削除ということで覚えました。

[A.Ichi]

ありがとうございます、いつもその度にechoしてました。頂きます。

[DAIKAN]

迅速はご回答ありがとうございました。
ひとまずは、ループ処理終了後にTARGET_MSG_ID=${TARGET_MSG_ID%,}とすることで、
対応するようにしようと思います。

Answer 3

「各行をカンマで繋ぐ」であれば、t_obaraさんの paste -s で十分ですが、
「最終行だけ別の処理をしたい」の一般化された場合にも出来るように考えてみました。
ループで一回ずつ遅らせて処理します。

TARGET_MSG_ID=
last=
while read line
do if test -n "$last"
   then TARGET_MSG_ID="$TARGET_MSG_ID$last," #最後以外の処理
   fi
   last="$line"
done < msg_id.txt
TARGET_MSG_ID="$TARGET_MSG_ID$last" #最後の処理

echo "$TARGET_MSG_ID"

Comments

[DAIKAN]

ありがとうございます！
「最終行だけ別の処理をしたい」の、一般化された考えも知りたかったため、大変助かりました。

Answer 4

パッと思いつく方法としては、全部","をつけてしまった後に、最後の行だけ","を抜く。という方法でしょうか。

しかし、個人的にはcatとsedでワンラインでなんとかできちゃうんじゃないかと勝手に思ってしまうのですが、やっぱりダメですかね。

cat msg_id.txt | sed -e s/([0-9]+)\$/\1,/g | sed -e s/,\$//g > done.txt

こんな感じでどうでしょう。間違っていたら申し訳ございません。バックアップを取ってから実行して見てください。

Comments

[t_obara]

> ワンラインでなんとか
確かに。
この場合に限っては、pasteコマンドで一発っぽいです。
paste -s -d ',' msg_id.txt > new_msg_id.txt
* あまりサイズが大きいと怒られそうですけど。