ZSH for loop array variable issue

Question

I'm working in zsh, but I'm sure that bash instructions will also be helpful.

I need to have a for loop that goes through the values stored in the array lw and then launches a shell script, based on the name value stored in the array.

So far, this is what I've come up with:

$lw=('plugin1' 'plugin2' 'plugin3')

for i in $lw;
  do . ~/Library/Rogall/plugins/$lw[$i]/lw.prg end;
done

Running this gives me an error saying that it can't find ~/Library/Rogall/plugins//lw.prg. It appears as if it's ignoring my variable all together.

Any ideas where I've messed up?

Try echo $i to see what it contains.

ott--
– ott--

2012-06-04 20:08:57 +00:00
Commented Jun 4, 2012 at 20:08 — ott--
– ott--, Commented Jun 4, 2012 at 20:08

ian · Accepted Answer · 2019-04-03 13:36:08Z

69

It's actually much simpler than that:

lw=('plugin1' 'plugin2' 'plugin3')

for i in $lw; do
  . ~/Library/Rogall/plugins/$i/lw.prg end
done

In summary:

Assign to foo, not $foo (the shell would try to expand $foo and assign to whatever it expands to; typically not useful)
Use the loop variable directly; it contains the array value rather than the index

edited Apr 3, 2019 at 13:36

ian

12.3k9 gold badges55 silver badges112 bronze badges

answered Jun 4, 2012 at 20:12

Jan Krüger

18.6k3 gold badges64 silver badges51 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

user1296965 Over a year ago

That makes so much sense! Thank you!

Stephane Chazelas Over a year ago

Or just for i ($lw) . ~/Library/Rogall/plugins/$i/lw.prg end

Stephane Chazelas Over a year ago

Not that in the general case, for i in $array would skip the empty elements. You'd need for i in "$array[@]" or for i in "${(@)array}" to preserve the empty ones (or the ksh/bash-compatible syntax: for i in "${array[@]}").

Jan Krüger Over a year ago

@theonlygusti you mean $lw? Simply put in the ('plugin1' 'plugin2' 'plugin3') directly in place of $lw.

William Pursell · Accepted Answer · 2012-06-04 20:31:17Z

30

Why bother using the array? This can be done in portable sh very easily:

lw='plugin1 plugin2 plugin3'

for i in $lw;
  do . ~/Library/Rogall/plugins/$i/lw.prg end
done

Note that for this to work in zsh, you need to make zsh do the right thing with: set -o shwordsplit

answered Jun 4, 2012 at 20:31

William Pursell

214k49 gold badges279 silver badges317 bronze badges

8 Comments

user507077 Over a year ago

Not everything is about being as portable as can be, especially if it's to be run on a Mac (judging by the directory names). Also your solution fails for the generic case in which there might be spaces in each word.

William Pursell Over a year ago

@Moritz This is not a generic case, but a specific case in which there are no spaces in the filenames. Portability may not always be necessary, but it is always a desirable goal. And this solution works just fine on a Mac.

William Pursell Over a year ago

@Moritz And spaces in filenames are an abomination! Also, Jan's solution fails if there is any whitespace in the filename.

user507077 Over a year ago

How do you know that the plugin directories don't contain spaces? The names presented in the original post are so generic that it's safe to assume that they're not the actual names. Jan's solution will work as it is on zsh with certain settings. Even if not it can be made with two trivial changes. Your solution cannot be used with white spaces easily. But alas, things like "spaces in file names are an abomination" is so incredibly up for debate that it's not even funny. But let's stop here and agree to disagree.

nik Over a year ago

It isn't portable if it relies on the ZSH defaults being changed when the context is likely a .zshrc file. Array is a better solution.

|

user3310334 · Accepted Answer · 2020-03-31 11:59:55Z

13

I had a problem where a loop like this one

for i in (1 2 3 4); do echo $i; done

was repeatedly not working as I wanted. I would get errors like zsh: unknown file attribute: 1, or outputs like

1 2 3 4

no matter how I re-contorted it, instead of my desired

To get my desired behaviour I had to remove the in keyword from the loop definition

for i (1 2 3 4); do echo $i; done
1
2
3
4

answered Mar 31, 2020 at 11:59

user3310334

Comments

altimes · Accepted Answer · 2021-10-14 23:48:26Z

For others who end up here wrestling with shell scripts, arrays and spaces in filenames...... like my file list below;

$ ls *.meta
20160324 1850 - ABC - Clarke And Dawe.ts.meta
20160706 1515 - 9Gem - Agatha Christie's Poirot.ts.meta
20181213 0155 - 10 BOLD - Star Trek_ The Next Generation.ts.meta
20210424 1927 - ABCTV HD - The Durrells.ts.meta
20210501 1927 - ABCTV HD - The Durrells.ts.meta
20210818 1930 - SBS ONE HD - Tony Robinson's World By Rail.ts.meta

After a lot of searching and trying different approaches this worked out to be the least problematic and worked on MacOS 10.15

eval "list=( $(find . -name "*.ts" -exec echo \"{}\" \;) )"
for name in "${list[@]}"
do
res=`mediainfo --Output=''Video\;\%Width\%:%Height\%'' "$name"`
echo ${name} :: $res
sed -i .bak -e 5s/\;[[:digit:]]*:[[:digit:]]*// -e 5s/$/\;$res/ "$name.meta" 
done

Ignoring the internals of what I was trying to achieve, the critical aspects were using eval and wrapping double quotes around the filename in the echo within the find. (What was I doing? I was pulling the resolution out of Enigma2 PVR recordings and writing it into line 5 of the related meta data file)

Thanks to all those who posted their own knowledge on the various issues that shell scripting generates.

Collectives™ on Stack Overflow

ZSH for loop array variable issue

4 Answers 4

4 Comments

8 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

4 Comments

8 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related