Skip to content

SC2321

Jump to bottom Edit New page
Joachim Ansorg edited this page Oct 31, 2022 · 2 revisions

Array indices are already arithmetic contexts. Prefer removing the $(( and )).

Problematic code:

values[$((i+1))]=1

Correct code:

values[i+1]=1

Rationale:

In indexed arrays (but not associative ones), the array index is already an arithmetic context. There is no point or value in wrapping it in an additional, explicit $((..)).

Exceptions:

If ShellCheck has failed to realize that your array is associative, or if you for stylistic reasons prefer the redundancy, you can ignore this message.

Related resources:

  • Help by adding links to BashFAQ, StackOverflow, man pages, POSIX, etc!
Add a custom footer
Add a custom sidebar

Clone this wiki locally