home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1993 #3 / NN_1993_3.iso / spool / comp / unix / shell / 5517 < prev    next >
Encoding:
Internet Message Format  |  1993-01-28  |  2.2 KB
  1. Path: sparky!uunet!spool.mu.edu!yale.edu!ira.uka.de!Germany.EU.net!sbsvax!mpi-sb.mpg.de!uwe
  2. From: uwe@mpi-sb.mpg.de (Uwe Waldmann)
  3. Newsgroups: comp.unix.shell
  4. Subject: Re: for loop termination
  5. Message-ID: <24206@sbsvax.cs.uni-sb.de>
  6. Date: 28 Jan 93 18:27:48 GMT
  7. References: <1993Jan27.190516.29466@relay.nswc.navy.mil>
  8. Sender: news@sbsvax.cs.uni-sb.de
  9. Reply-To: uwe@mpi-sb.mpg.de
  10. Organization: Max-Planck-Institut fuer Informatik
  11. Lines: 56
  12.  
  13. In article <1993Jan27.190516.29466@relay.nswc.navy.mil>,
  14. rweisbe@starfleet.nswc.navy.mil (Bob Weisbeck) writes:
  15. >    I'm not sure why this happens but here goes.  I have a script that 
  16. >    I'm executing but it does not terminate the loop properly and I'm
  17. >    not sure what is wrong with it. The script is as follows:
  18. > 
  19. > [ I've shortened the script a bit.  --uwe]
  20. > 
  21. > #!/bin/sh
  22. > FILES=`cat datafile`
  23. > set $FILES 
  24. > for file in $*
  25. > do
  26. >   WC=`wc -l $1  | tr -s " " | cut -f2   -d" "`
  27. >   shift 2
  28. > done
  29. > 
  30. > [ script performs ok, but at the end the user has to type ctrl-c to
  31. >   terminate it.  ]
  32.  
  33. Remember that the for-list (i.e. $*) is evaluated _only_once_, at the
  34. beginning of the loop.  If $* contains six words, then the loop is
  35. executed six times, no matter whether you shift once, twice, or not at
  36. all during the loop.  (To see that $file and $1 are not necessarily
  37. equal insert "echo $file $1" after the "do".)  Now, what happens after
  38. three loops?  The "shift 2" statement has been executed three times,
  39. thus the positional parameter list is empty, and so is $1 during the
  40. fourth loop.  Because "wc" is called with only one argument, namely
  41. "-l", it reads from stdin.  As there is no input from stdin, the
  42. program hangs.
  43.  
  44. The solution is, of course, to use a "while" loop.  Either
  45.  
  46.   while test $# -gt 0
  47.   do
  48.     WC=`wc -l $1  | tr -s " " | cut -f2   -d" "`
  49.     shift 2
  50.   done
  51.  
  52. or (this should be slightly faster if "test" isn't built in)
  53.  
  54.   while :
  55.   do
  56.     case $# in
  57.       0) break ;;
  58.       *) ;;
  59.     esac
  60.     WC=`wc -l $1  | tr -s " " | cut -f2   -d" "`
  61.     shift 2
  62.   done
  63.  
  64.  
  65. -- 
  66. Uwe Waldmann, Max-Planck-Institut fuer Informatik
  67. Im Stadtwald, D-W6600 Saarbruecken 1, Germany
  68. Phone: +49 681 302-5431, Fax: +49 681 302-5401, E-Mail: uwe@mpi-sb.mpg.de
  69.