シェルスクリプト総合 その29

[1 デフォルトの名無しさん 2018/09/21(金) 23:34:47.98 ID:/eJt2TVD]

シェルスクリプトに関する総合スレッドです。

全般
・荒しは無視しましょう。
・丁寧な姿勢を心掛けましょう。
・ネチケット（死語）を意識しましょう。

前スレ:
シェルスクリプト総合 その28
http://mevius.5ch.net/test/read.cgi/tech/1532397676/

[984 デフォルトの名無しさん 2019/07/02(火) 04:35:32.93 ID:Atdu9zIi]

>>983
正しく動くし何も問題ない。

お前こそ、何も言い返せないのに
無理してレスしなくていいぞｗ

[985 デフォルトの名無しさん 2019/07/02(火) 04:37:28.40 ID:uO4UDEG+]

いやあ、ああいうのが普通な人には何も言い返せないよ。てか「言い返す」って。喧嘩かなんかなのか？w

[986 デフォルトの名無しさん 2019/07/02(火) 04:41:11.93 ID:Atdu9zIi]

喧嘩じゃなくて、お前の意見を言えって。

正しく動くだろ？
なにか問題が有るのか？

[987 デフォルトの名無しさん 2019/07/02(火) 04:45:47.25 ID:Atdu9zIi]

シェルスクリプト初心者です。 { } の使い方がわかりません
とかいうなら、関数にでもすればいい

foo() {
　なにか前処理

　hoge=0

　while IFS= read -r line; do
　　[ なにか ] && hoge=1
　　ほかにいろいろ
　done

　if [ hoge = 0 ]; then
　　前処理の調整
　else
　　前処理の調整
　fi
}

cat | foo

お前の苦手な { } が無くてすんだぞｗ
サブシェルうんぬんの前に、関数の中で使う変数は、関数の中にとどめておけよ。
わかりやすいコーディングというのはこういうものだ

[988 デフォルトの名無しさん 2019/07/02(火) 04:47:32.87 ID:uO4UDEG+]

そのサブシェルの中で後は全部やるんでしょ？必要な部分を。それで問題無いと思うなら、それでいいんじゃないの
「なんか無理してんな」ってのがわからないんだったらそれはそれでいいよ。元ネタと同じく、好みの問題。好みがわからないでしょ

[989 デフォルトの名無しさん 2019/07/02(火) 04:50:12.51 ID:Atdu9zIi]

だから問題ないし、お前を含めて誰も問題があると言ってない。

お前さ、なんで自分で問題点を指摘できないのに
さも問題があるかのように装ってるわけ？
ばれないとでも思った？

[990 デフォルトの名無しさん 2019/07/02(火) 04:50:38.17 ID:uO4UDEG+]

なにか頑張ってるな。そんな話じゃないよ
知識自慢臭しかもうしませんよ？そんなの自慢げというか馬鹿にしたいだけなのか知らんけど

[991 デフォルトの名無しさん 2019/07/02(火) 04:53:12.30 ID:Atdu9zIi]

そんな話じゃないよ → じゃあどんな話なのか → 答えずに逃げる（笑）

[992 デフォルトの名無しさん 2019/07/02(火) 04:53:48.95 ID:uO4UDEG+]

問題がある問題がないの話じゃないよ。好みだよ好み
そう次々あれこれ考えなくてもw bash(など)の拡張使えばスッキリだろうにという

[993 デフォルトの名無しさん 2019/07/02(火) 04:54:45.55 ID:Atdu9zIi]

>>987のどこがスッキリとしてないというのか？
お前ならbashでどう書き直す？ｗ

[994 デフォルトの名無しさん 2019/07/02(火) 04:57:00.91 ID:uO4UDEG+]

hoge=0
while IFS= read -r line; do
[ なにか ] && hoge=1
ほかにいろいろ
done < <(cat)

if [ hoge = 0 ]; then
前処理の調整
else
前処理の調整
fi

そもそもやりたいことがレベルが変わらずにそのままでスッキリだなwあとあとでなにか追加もとかもね

[995 デフォルトの名無しさん 2019/07/02(火) 04:58:47.96 ID:Atdu9zIi]

>>994
それ、俺が書いたfoo関数の中身を書いただけじゃねーかｗ

hoge=0
while IFS= read -r line; do
　[ なにか ] && hoge=1
　ほかにいろいろ
done

if [ hoge = 0 ]; then
　前処理の調整
else
　前処理の調整
fi

これで動くよ。
foo() {
　上のコード
}
cat | foo

ってやればいいだけ

[996 デフォルトの名無しさん 2019/07/02(火) 05:01:09.48 ID:Atdu9zIi]

Q.これをPOSIX準拠で書きたいのですが？

hoge=0
while IFS= read -r line; do
[ なにか ] && hoge=1
ほかにいろいろ
done < <(cat)

if [ hoge = 0 ]; then
前処理の調整
else
前処理の調整
fi

A. まず1. 関数にしましょう

foo() {
hoge=0
while IFS= read -r line; do
[ なにか ] && hoge=1
ほかにいろいろ
done < <(cat)

if [ hoge = 0 ]; then
前処理の調整
else
前処理の調整
fi
}

foo

[997 デフォルトの名無しさん 2019/07/02(火) 05:01:36.77 ID:uO4UDEG+]

いや、だから、お題に対してだけじゃなくな。普遍的にって言ってるでしょ
それが普遍的なのか？

なんで否定されたのをそんなに拘るねん。否定してんのは俺の好みじゃないってだけだぞ？w

[998 デフォルトの名無しさん 2019/07/02(火) 05:01:53.99 ID:Atdu9zIi]

2. bash依存の命令をなくしましょう。

foo() {
hoge=0
while IFS= read -r line; do
[ なにか ] && hoge=1
ほかにいろいろ
done

if [ hoge = 0 ]; then
前処理の調整
else
前処理の調整
fi
}

cat | foo

はい。完成です（笑）

[999 デフォルトの名無しさん 2019/07/02(火) 05:02:22.44 ID:Atdu9zIi]

>>997
だから < <(cat) を使う場合の普遍的な解答

[1000 デフォルトの名無しさん 2019/07/02(火) 05:02:50.90 ID:Atdu9zIi]

やっぱり書き方を知らんだけだったな（笑）