Bashで単純なループを試すときの問題

Question

スクリプトに構文エラーがあります。以下を使用して、シェルスクリプトに問題のある構造があるかどうかを確認できます。住宅検査オンライン。

これはあなたに言うでしょう

Line 3:
while["$a -lt 50"]
^-- SC1009: The mentioned parser error was in this while loop.
     ^-- SC1035: You need a space after the [ and before the ].
     ^-- SC1069: You need a space before the [.
     ^-- SC1073: Couldn't parse this test expression.
                  ^-- SC1020: You need a space before the ].
                  ^-- SC1072: Missing space before ]. Fix any mentioned problems and try again.

変更によるスペースのトラブルシューティング

while["$a -lt 50"]

入力する

while [ "$a -lt 50" ]

代わりに、以下が提供されます。

Line 3:
while [ "$a -lt 50" ]
           ^-- SC2157: Argument to implicit -n is always true due to literal strings.

Line 6:
a='expr $a+1'
  ^-- SC2016: Expressions don't expand in single quotes, use double quotes for that.

報告された最初の問題は文字列に関するものです"$a -lt 50"。実際、ここではそのような文字列が欲しくありません"$a" -lt 50。ところで、文字列は常に「true」なのでこれがループが無限の理由です。（構文エラーが修正された場合）

2番目の問題は、チェッカーが$a一重引用符で囲まれた文字列内の変数を検出するために発生します。これはその値には拡張されません（そのため、印刷された文字列は次のようになります。expr $a+1）。解決策は二重引用符に変更しないことです。これは同じ文字列を提供しますが、値は拡張されます。コマンドを実行したいですexpr。

一重引用符をバックティックに変更するだけです。

これでスクリプトは次のようになります。

#!bin/sh
a=0
while [ "$a" -lt 50 ]
do
echo $a
a=`expr $a+1`
done

...そしてShellCheckはまだ満足していません。

Line 6:
a=`expr $a+1`
  ^-- SC2006: Use $(..) instead of legacy `..`.
   ^-- SC2003: expr is antiquated. Consider rewriting this using $((..)), ${} or [[ ]].

$( ... )新しいシェルコードは実際にはバックティックの代わりにバックティックを使用する必要があります。また、expr古い使用法に関する警告も表示されます。

この行は次のように書き直すことができます。

a="$(( a + 1 ))"

最終バージョン（インデントと行の修正を含む#!）：

#!/bin/sh

a=0
while [ "$a" -lt 50 ]; do
  echo $a
  a="$(( a + 1 ))"
done

bashまたは、ksh93算術評価用のバージョンを使用して(( ... ))コードをさらに短縮します。

#!/bin/bash

a=0
while (( a < 50 )); do
  echo "$(( a++ ))"
done

Answer 1