Conditional Where Clause

SQLServerCentral is supported by Red Gate Software Ltd.

Popular Topics

Home

Members

Calendar

Who's On

Home » SQL Server 2005 » SQL Server 2005 General Discussion » Conditional Where Clause

Conditional Where Clause

Rate Topic

Display Mode

Topic Options

Author

Message

al_anderson

Posted Thursday, July 24, 2008 8:33 AM

Grasshopper

Group: General Forum Members
Last Login: Tuesday, January 19, 2010 7:20 AM
Points: 14, Visits: 90

I have 2 formats of date fields in 2 seperate tables. Given the selection of P or F
I am attempting to do use the appropriate date range for the where clause:

WHERE 1=1
AND CASE
WHEN FPCH.ProcessedDateKey >= @Beg_Date AND FPCH.ProcessedDateKey <= @End_Date THEN 'P'
WHEN AR.RunCycleDate>= @pBeg_Date AND AR.RunCycleDate<= @pEnd_Date THEN 'F'
END = @DateSolution
AND FPCH.InventoryStatusCode in (97,98)

The P seems to work, but the F doesn't. I'm self taught, and have been programming in other languages for a long time, but I'm not sure what I'm missing here. Any help would be greatly appreciated. Thanks in advance.

Post #540220

GSquared

Posted Thursday, July 24, 2008 9:47 AM

SSCoach

Group: General Forum Members
Last Login: Monday, May 06, 2013 1:09 PM
Points: 15,439, Visits: 9,569

Not sure from just this little bit, but a place to start is that it will never get to F if the test for P comes up true. Case statements stop as soon as they find a true condition.

Also, why have the 1=1 in your Where clause?

I'd most likely rewrite it as:

WHERE
   (FPCH.ProcessedDateKey >= @Beg_Date  AND FPCH.ProcessedDateKey <= @End_Date and 'P' = @DateSolution
   or
   AR.RunCycleDate>= @pBeg_Date AND AR.RunCycleDate<= @pEnd_Date and 'F' = @DateSolution)
AND FPCH.InventoryStatusCode in (97,98)

- Gus "GSquared", RSVP, OODA, MAP, NMVP, FAQ, SAT, SQL, DNA, RNA, UOI, IOU, AM, PM, AD, BC, BCE, USA, UN, CF, ROFL, LOL, ETC
Property of The Thread

"Nobody knows the age of the human race, but everyone agrees it's old enough to know better." - Anon

Post #540309

al_anderson

Posted Thursday, July 24, 2008 9:55 AM

Grasshopper

Group: General Forum Members
Last Login: Tuesday, January 19, 2010 7:20 AM
Points: 14, Visits: 90

wow, was I over analyzing things! Thanks so much.

Post #540315

GSquared

Posted Thursday, July 24, 2008 9:56 AM

SSCoach

Group: General Forum Members
Last Login: Monday, May 06, 2013 1:09 PM
Points: 15,439, Visits: 9,569

You're welcome.

- Gus "GSquared", RSVP, OODA, MAP, NMVP, FAQ, SAT, SQL, DNA, RNA, UOI, IOU, AM, PM, AD, BC, BCE, USA, UN, CF, ROFL, LOL, ETC
Property of The Thread

"Nobody knows the age of the human race, but everyone agrees it's old enough to know better." - Anon

Post #540319

« Prev Topic | Next Topic »

Permissions