ouch...Thanks.

Thanks all!  I tend to go with what is recommended industry wide.  On with the brackets!

what if you want to check for zero or one, like it's boolean.  Or check for an empty string?

the reason I need to perform an IF on that @TermID is because the user may have not selected a TErm in the Search UI.  Thus, if that param is...

Why are you using dynamic sql or this ?

because, I have a bunch of params incoming from a search interface where the user is able to either select or not...

I know about the need for it with reserver words, but, for example our dba always puts it around everything.  Also, thew new SQL Prompt by RedGate does also.

here, someone else found a way for it to work

FROM

[dbo]

.[Product] P WITH (NOLOCK)

WHERE

...

this is one reason I hate SQL.  we have to come up with crazy ass ways to get it to do what we want cause it's not an OOP language....

thanks for your pointers though!

regards

Ok, figured it out on my own:

Declare @Keyword varchar(30),

            @KeywordLen tinyint,

            @ProductID int,

            @Pos smallint,

            @StartPos smallint,

            @EndPos smallint,

            @Result varchar(1000),

            @ResultToReturn...

oK, looks like I'm slowy working my way through this one myself.  I found out that no matter what, after the IF statements, @valid is never being set to true

Ok, I tested this here

Declare @Keyword varchar(30),

            @KeywordLen tinyint,

            @ProductID int,

            @Pos smallint,

            @StartPos smallint,

            @EndPos smallint,

            @Result varchar(1000),

            @valid int,

           ...

so going back to my example again now (updated):

Declare @Keyword varchar(30),

            @KeywordLen tinyint,

            @ProductID int,

            @Pos smallint,

            @StartPos smallint,

            @EndPos smallint,

            @Result varchar(1000),

           ...

I am thinking this will also not work:

SELECT

@MP3Pos = CHARINDEX('MP3', @ProductDescription,...

I don't understand why you're doing this

@WavPos

< @MP3Pos OR @MP3Pos = 0

The whole idea is to check whether...