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

テクノロジーカテゴリーの変更を依頼記事元:

412 usersがブックマークコメント

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

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

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

シェルスクリプト

2024/08/21 リンク

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

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

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

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

2024/08/28 リンク

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

2024/08/24 リンク

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

2024/08/23 リンク

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

2024/08/22 リンク

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

プログラミング

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

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

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