Hi All,

I have been fighting with a syntax error for the last 2 days, still haven't got the solution. Could you please help me, your help will be greatly appreciated.

In my script I am getting a error in a for loop, its similar to the one as is below.

for v_id in v1 v2 v3 v4
do
v_flag=`sqlplus -s orauser/passwd@oracle_sid << oratest
set feedback off
set heading off
select 'Y' from mytable where myid = $v_id;
exit 5
oratest`
if [ $v_flag ]
then
update the o/p file
fi
done

If the condition fails in the query the v_flag is null, then its giving the error "for not matched"

Can't I use the if condition which will be null inside the for loop?


Thanks in Advance
Raju

I can's see any reason why not - are you sure that's the problem?

I'd echo out the value of $v_flag for each iteration and see what it actually contains. It may be that it has some unexpected value as a result of the query (such as apostrophe's or quotes) that are throwing the script.

when the query returns nothing then only its giving the error

Well I'm still not convinced it's because the query's returning nothing. It sounds more like it's returning something the query can't handle, but why not try putting quotes around the variable in the test, ie:-

if [ "$v_flag" ]

You really should add a proper test flag to the if [ $v_flag ] .. as well as putting quotes round the variable as grasper suggests.

Without any arguments the test just checks if the string contains any characters or not. If this is the test you want then write it like:

if [ -n "$v_flag" ]

The reason you have to put quotes round it is because if the string doesn't contain any characters then the command that gets run is

if [ ]

which is illegal. As opposed to

if [ "" ]

Which is what you want to be running.

Thanks for your immediate response.

But its still giving the same error. If I comment the for loop and executing the for a single variable then its fine, but for the "for loop" its giving the error, am checking for the alternatives if any.

Regards,
Raju

Ok, run the script for each possible value of v_id and echo out the values of $v_flag.

Or alternatively, add the loop back, but edit it each time to run only one of the potential values of v_id and again see what the values of $v_flag are. Cut and paste them in your reply if you can.