0 
for k in {794..800}
if [  -f    "vesta_$k.xyz"    ]
then
      do
         mkdir $k
         mv vesta_$k.xyz $k/vesta.xyz
         cd $k/
                 <<COMMANDS>>
         cd ..
      done
fi

PS vesta_$k.xyz is a sequence of files. The problem is at the first line of the IF command.

Improve this question
3
  • 1
    Put a ; at the end of if statement: if [ -f "vesta_$k.xyz" ]; Commented Feb 5, 2016 at 9:26
  • 1
    @bangal You don't need the ; when there is a line break. Commented Feb 5, 2016 at 10:33
  • I'm not sure this question is on-topic for Server Fault but in any case, shellcheck.net is a great resource for diagnosing shell scripting problems (such as syntax errors). Commented Feb 5, 2016 at 11:07

1 Answer 1

Reset to default
2

if.. then.. fi should be fully inside for.. do.. done, you cannot cross them

for        k in {794..800}
do
    if     [ -f "vesta_$k.xyz" ]
    then
           mkdir "$k"
           mv "vesta_$k.xyz" "$k/vesta.xyz"
           cd "$k" || exit 3
           echo '<<COMMANDS>>'
           cd - "$OLDPWD"
    fi
done
Improve this answer
2
  • 2
    I would change a few other things in the script: 1: The #! line is missing. 2: Inconsistency in style for and do are on the same line, but if and then are not. Pick one style and stick with it. 3: Body of the for loop did not get indented. 3: Excessive whitespace within the [ ] construct. 4: Missing " around most expansions of $k (it is not strictly needed here, but I think it would be a good habbit to get into.) Commented Feb 5, 2016 at 10:40
  • user1700494 and kasperd , thanks for your comments. The script is good now. I will consider your suggestions for the next ones. Commented Feb 5, 2016 at 15:19

You must log in to answer this question.