how to: Expand the array example with a loop

how_to_do_things_safely_in_bash.md

@@ -115,24 +115,34 @@ The syntax is verbose, but get over it. This bashism is reason alone to drop pos
 
 Good:
 
-    array=(
+    files=(
         a
         b
     )
-    array+=(c)
+    duplicates=()
-    if [ ${#array[@]} -gt 0 ]; then
+    for f in "${files[@]}"; do
-        rm -- "${array[@]}"
+        if cmp "$f" other/"$f"; then
+            duplicates+=("$f")
+        fi
+    done
+    if [ "${#duplicates[@]}" -gt 0 ]; then
+        rm -- "${duplicates[@]}"
     fi
 
 Bad:
 
-    pseudoarray=" \
+    files=" \
         a \
         b \
     "
-    pseudoarray="$pseudoarray c"
+    duplicates=
-    if ! [ "$pseudoarray" = '' ]; then
+    for f in $files; do
-        rm -- $pseudoarray
+        if cmp "$f" other/"$f"; then
+            duplicates+=" $f"
+        fi
+    done
+    if ! [ "$duplicates" = '' ]; then
+        rm -- $duplicates
     fi
 
 Here is why arrays are such a basic feature for a shell: [Command arguments are fundamentally arrays](http://manpag.es/RHEL6/3p+exec)