topic Re: Looping in a script in Operating System - HP-UX

Looping in a script

Jeeshan — Tue, 19 May 2009 11:13:15 GMT

I have a script which is ran to get a file from ftp site and manipulate it. but the file is overwritten and match with the pattern. if pattern found then do some jobs. here is that
#!/usr/bin/sh
function ftps {
sleep 20
USER=anonymous
PW=xxxxx
ftp -v -i -n << EOF
user $USER $PW
ascii
mget notification.log
bye
EOF
}
ftps
MAIL_RECEIVER=""
a=`cat notification.log |head -n 5|sed 's/^.............................................//' > file.not.tmp`
b=`cat file.not.tmp |awk '{print $4}'`
if [[ $b = "" ]]
then
mailx -s "CDR backup is completed in blbilstg server" $MAIL_RECEIVER < file.not.tmp
fi

The scripts seems ok if the pattern is matched. But my concern is, I want to run the script until the pattern is not matched.

Re: Looping in a script

Doug O'Leary — Tue, 19 May 2009 11:25:45 GMT

Hey;

If I'm understanding you correctly, you should be able to change the if statement at the end to

while [ "${b}" = "" ]
do
...

done

I'd caution against this implemenetation, though. You're setting yourself up for problems if the file gets overwritten while it's being processed, if the ftp fails for any reason, etc.

HTH;

Doug O'Leary

Re: Looping in a script

James R. Ferguson — Tue, 19 May 2009 11:27:32 GMT

Hi Ahsan:

There are several ways to do this. One way is to use a 'true' loop and 'break' out of it when the condition you want to see is satisfied:

#!/usr/bin/sh
function ftps {
sleep 20
USER=anonymous
PW=xxxxx
ftp -v -i -n server_name << EOF
user $USER $PW
ascii
mget notification.log
bye
EOF
}
ftps
MAIL_RECEIVER=""
a=`cat notification.log |head -n 5|sed 's/^.............................................//' > file.not.tmp`
while true
do
b=`cat file.not.tmp |awk '{print $4}'`
if [[ $b = "" ]]
then
mailx -s "CDR backup is completed in blbilstg server" $MAIL_RECEIVER < file.not.tmp
break
fi
done

...Now, let's make some improvements to your general script. (1) Eliminate useless 'cat' processes! (2) the 'sed' match can be made much more readable with a repeat part; (3) eliminate the backticks and use the clearer POSIX shell $() notation for command execution:

#!/usr/bin/sh
function ftps {
sleep 20
USER=anonymous
PW=xxxxx
ftp -v -i -n server_name << EOF
user $USER $PW
ascii
mget notification.log
bye
EOF
}
ftps
MAIL_RECEIVER=""
a=$(cat notification.log |head -n 5|sed 's/^.{45}//' > file.not.tmp)
while true
do
b=$(awk '{print $4}' file.not.tmp)
if [[ $b = "" ]]
then
mailx -s "CDR backup is completed in blbilstg server" $MAIL_RECEIVER < file.not.tmp
break
fi
done

Regards!

...JRF...

Re: Looping in a script

Jeeshan — Tue, 19 May 2009 11:38:24 GMT

thanks for your inputs.

My concern is notification.log is overwritten and i do want to do ftp again until the pattern is matched.

Re: Looping in a script

James R. Ferguson — Tue, 19 May 2009 11:51:35 GMT

Hi (again) Ahsan:

> My concern is notification.log is overwritten and i do want to do ftp again until the pattern is matched.

Since you are getting (with FTP) and then parsing this file, I would remove the file before performing the FTP. This also serves as a success-or-failure test:

...
rm -f notification.log
ftps
...

By the way, since you reference this file name multiple times, you should consider making it the value of a variable:

MYLOG=notification.log
...
rm -r ${MYLOG}
ftps
...

Too, I fail to see why you are using 'mget' instead of a simple 'get'. You aren't globbing for multiple files.

Regards!

...JRF...

Re: Looping in a script

Jeeshan — Tue, 19 May 2009 12:02:57 GMT

Yeah JRF, you are right, I should not suppose to mget instead of get.

BTW, where i can put this rm then ftps lines?

Re: Looping in a script

James R. Ferguson — Tue, 19 May 2009 12:14:41 GMT

Hi (again) Ahsan:

> BTW, where i can put this rm then ftps lines?

As I showed, before the ftps() call.

#!/usr/bin/sh
set -u
typeset MYLOG=notification.log
typeset MYNOT=file.not.tmp
typeset MAILTO="you@your.net"
function ftps {
sleep 20
USER=anonymous
PW=xxxxx
ftp -v -i -n server_name << EOF
user ${USER} ${PW}
ascii
get ${MYLOG}
bye
EOF
}
rm -f ${MYLOG}
ftps
a=$(cat ${MYLOG}|head -n 5|sed 's/^.{45}//' > ${MYNOT})
while true
do
b=$(awk '{print $4}' ${MYNOT})
if [[ $b = "" ]]
then
mailx -s "CDR backup is completed in blbilstg server" ${MAILTO} < ${MYNOT}
break
fi
done

...

Regards!

...JRF...

Re: Looping in a script

Jeeshan — Tue, 19 May 2009 12:52:23 GMT

thanks again.

But i want if the pattern is not matched it again ftp the file and check accordingly until the matched found. If matched found than the program will terminate.

Re: Looping in a script

James R. Ferguson — Tue, 19 May 2009 13:18:47 GMT

Hi (again):

OK, your loop could look like this:

...
while true
do
rm -f ${MYLOG}
ftps
a=$(cat ${MYLOG}|head -n 5|sed 's/^.{45}//' > ${MYNOT})
b=$(awk '{print $4}' ${MYNOT})
if [[ $b = "" ]]
then
mailx -s "CDR backup is completed in blbilstg server" ${MAILTO} < ${MYNOT}
continue
else
break
fi
done
...

Regards!

...JRF...

Re: Looping in a script

Dennis Handly — Tue, 19 May 2009 16:06:52 GMT

>Doug: while [ "${b}" = "" ]

If you want a pattern instead of a simple string compare, you must use [[ ]]. And if you have a pattern, you must not use "".

>JRF: (1) Eliminate useless 'cat' processes!

Any reason you left this one? ;-)
a=$(cat ${MYLOG}|head -n 5|sed 's/^.{45}//' > ${MYNOT})

You could reverse the logic and save an indent for the main logic:
if [[ $b != ]]; then
break
fi
mailx -s "CDR backup is completed in blbilstg server" ${MAILTO} < ${MYNOT}

Re: Looping in a script

James R. Ferguson — Tue, 19 May 2009 16:11:29 GMT

HI:

> Dennis: >JRF: (1) Eliminate useless 'cat' processes! Any reason you left this one? ;-)

Yeah, I was focused on the 'sed' and on the original backticks :-}}

Regards!

...JRF...

Re: Looping in a script

Jeeshan — Wed, 20 May 2009 05:46:18 GMT

JRF, just a modification and it works.