いい加減シェルスクリプトで [ $? -eq 0 ] や [ $? -ne 0 ] なんて エラー処理を書くのはやめよう！ - Qiita

いい加減シェルスクリプトで [ $? -eq 0 ] や [ $? -ne 0 ] なんて エラー処理を書くのはやめよう！ShellScriptBashLinuxUNIXmacOS はじめに [ $? -eq 0 ] や [ $? -ne 0 ] は冗長でデメリットしかありません。非常に多く見かける書き方ですが、1979 年に Bourne シェルが広く公開された時からこのようなコードは必要ありませんでした。実際に当時はこのような書き方は使われておらず、このような書き方をしなければならなかった歴史的な経緯などはありません。これはなぜか広まってしまった良くない書き方です。 優れたコードとは無駄がないシンプルなコードです。丁寧なコードとは無駄な処理を書くことではありません。[ $? -eq 0 ] や [ $? -ne 0 ] は書かないほうが、簡単で読みやすくわかりやすくなります。優れた文法

[kkobayashi]

うーん言いたいことは分かるがコマンドを処理から切り離したい気持ちはある

[kobito19]

関数(ここではコマンドだけど)の実行と結果の評価を1行で行うな教徒なので…

[d0i]

そもそもshellscriptを書くのをやめたいです、 先生...

[rna]

知らんかったとおもったらcsh系ではそう書けないのか。自分がUnix触り始めた頃はcshかtcshが主流だったからなぁ。

[iww]

コマンドの実行を if文の中に書きたくないんだよ！ という話

[dorapon2000]

“[ $? -eq 0 ] や [ $? -ne 0 ] なんて書くのをやめようと言っているのは、私だけではありません。良いシェルスクリプトを書くのに必須の ShellCheck でも同じように指摘されます。”

[punychan]

もとから書かねーよ。終了コードが重要なケースならcaseで分けるだろうし。／さすがに || や && はワンライナーくらいでしか使わんが。

[iamamachine]

いい加減シェルスクリプトを書くのはやめよう！

[xlc]

私はifすら使わず&&と||で連結するかな。シェルになれると$?を使う機会は減る。set -eしたら使えないしね。ブコメ: コマンドの正常終了は0を返すことで統一されてます。

[shingo-sasaki-0529]

これ普通に書くな。なんでだろ。コマンドの実行と、その結果による分岐を分けて書いたほうがデバッグしやすいからかも。どういうときにどの終了ステータスを返すかがコマンドによってマチマチだからってのもある。

[y-kawaz]

単に趣味の問題。

[UME]

言いたいことはわかるつもりだけど、if [ $? -eq 0 ]; でもそんなに見にくくはないし、むしろなにをやってエラーなのかは行が分かれてるおかげでわかりやすくなってるように思います。

[tmatsuu]

ifの後がコマンド実行なのは知ってたけど、 if now=$(date) と書いても拾えることはしらなかった。へー。まぁ自分はついついret=$?を書いてしまうんですが。

[ku__ra__ge]

「if判定を通ると破壊的な操作が実行される」のは非直感的なのでは……と思ったが、「シェル言語の if はどちらかと言えば try ... catch に近い」か。なるほど。

[tybalt]

意図の読み取りやすさや修正のしやすさを考えて、敢えて冗長に書くことも多い

[mohri]

大学のときシェルスクリプトを書けって課題が出て、よくわからんままログインシェルの文法に即して提出したら「またcshだ」って言われたのおもいだした（当時はSunがまだBSDでログインシェルはcsh系だった）

[richmikan]

完全なる宗教。ShellCheck原理主義。コマンドをif文の中に入れずに可読性を追求する宗派だってあるだろうに。

[rryu]

コマンドを実行したいだけで分岐したい訳じゃないんだという意図を盛り込むとエラーの分岐を別に書きたくなる。分岐のためにコマンドを実行するならそう書くべきだと思う。

[honma200]

とりあえず途中のデータは一時変数に保持しておきたいデータMOTTAINAI教徒がいましてな

[tetsuya_iwatanai]

ついやっちゃうのは俺の記憶力が悪いためかな

[ducktoon]

シェルスクリプトでコマンドを並べる以上のことはするな

[poponponpon]

シェルスクリプトは全てを忘れた頃に使う場面が出てくるの繰り返しなので、毎回新鮮な気持ちでググってます。使いたくない。

[chiroruxx]

esacも書きたくないです

[Iridium]

シェルスクリプトはすぐに難読化するし、実行内容が追いにくいから可能な限り使いたくない。言語自体アップデートしてよ。あと開発環境もなんとかして

[als_uz]

"[" じゃなく "[[" を使えって話かと思ったら違った様子 / 冒頭で set -eu しちゃうんで…

[mjy]

ブコメの「ifすら使わず&&と||で連結する」のもありだと思うが、これは if else とは違うので要注意。if else ならば実行されるのはif節かelse節のどちらかだが、&& || は違う。例: 「true && ls doesnt-exist.txt || echo False」

[aobon700]

シェルスクリプトガチ勢ではないので、ちょっとでも困ったらGPT先生に聞いてます

[AmaiSaeta]

自分の書き方を考えてみると、 `grep` の様な検証・判定を行う場合は `if` の条件として書くが、環境を変える（ファイルコピー等）場合は分けて、無意識に使い分けてる。一般的なプログラミングでどうかに引き摺られてる

[taguch1]

いいかげんシェルスクリプトやめろ

[mas-higa]

場合によるけど something || error_handling って書くこともある

[kzm1760]

確かにそうすねというのと自分は使い分けているんだけど、記事が全体的に煽り口調なので肯定出来ないし読む気が失せてしまうのが勿体無い。煽り記事は別の意図が透けるのでまともに読まないようにしてるから。

[napsucks]

このへん色々嫌い

[getcha]

へー。知らんかった（か過去に見ていても目が滑っていたか）。コマンドが長い時とかは、使いにくいから、使える時に使えば良いでしょう。ぐらいのものじゃないかな。複数の処理系使ってる人もいるだろうし(私だ）

[ka-ka_xyz]

にょも〜〜〜〜〜〜〜〜〜〜

[ya--mada]

どうだろうなぁあ？

[n_vermillion]

if文の中に直接処理を埋め込むんじゃないって俺の中でwarningが出てしまう。そのif処理はcpのリターンコードを判定する処理であってcpを実行するところと分けないとメンテナンスが云々。まず$?を変数に格納しちゃうわ…。

[craftone]

リターンコード0がtrueと評価されるのが気持ち悪いというのはある

[hevohevo]

良いシェルスクリプトを書こうというモチベーションがそもそもない。複雑で見づらくなりそうなら別のスクリプト言語を使うから。

[hjmk]

込み入った処理はRubyやPythonで書くようにしてる。シェルスクリプトはメンテが辛い。