Go Premium for a chance to win a PS4. Enter to Win

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 300
  • Last Modified:

Sum - Case when

The results shown in the Excel file : text.xls

Have a table tbl_testingfreight :
The result is the first tab of the Excel sheet (Actual Data)
After running the below script I get the results showed in the second tab (Result based on the query)

I am trying to do a SQL case when , but not able to see the result as needed
The final result needs tto be as shown in the tab (Results required)

The conditions are:
when the VndrNum = 359958 and  OpCo = 177 for that SUVCVndrNum, ItemNum --------  need to sum up the [Freight Cost / Case ($)] jus t as highlighted.

How can i apply this condion in the below query:

select TOP 100 PERCENT
	d.OpCo
	,d.SUVCVndrNum
	,d.VndrNum	
	,ItemNum
	,sum(Cases) [Case Volume]	
	,sum(FOB) [COGS ($)]
	,sum(isnull(FrtPerCase, 0)*Cases) [Freight Cost ($)]
	,sum(isnull(LEIPerCase, 0)*Cases) [LEI ($)]
	,max(isnull(FrtPerCase, 0)) [Freight Cost / Case ($)]
	,max(isnull(LEIPerCase, 0)) [LEI ($) / Case]
	
from
	tbl_testingfreight d
	left outer join test.dbo.[OpCo Locations] o
		on d.OpCo  =  o.OpCo
	cross join (select distinct sourcinginitiative 
			from [freightPO-OPCO]) si
where
	SourcedCases  >  0
 
group by
	d.OpCo
	,d.SUVCVndrNum
	,d.VndrNum
	,ItemNum
order by
	d.OpCo
	,ItemNum

Open in new window

test.xls
0
mercybthomas74
Asked:
mercybthomas74
  • 5
  • 3
  • 3
1 Solution
 
David ToddSenior DBACommented:
Hi,

I'm having difficulty following your logic.

But think that my attached query might be close.

Plese apply the table alias's consistently. They are applied to only half the columns. It helps with code review and maintenance if they are all applied.

Cheers
  David
select TOP 100 PERCENT
	d.OpCo
	,d.SUVCVndrNum
	,d.VndrNum	
	,ItemNum
	,sum(Cases) [Case Volume]	
	,sum(FOB) [COGS ($)]
	,sum(isnull(FrtPerCase, 0)*Cases) [Freight Cost ($)]
	,sum(isnull(LEIPerCase, 0)*Cases) [LEI ($)]
	,max(isnull(FrtPerCase, 0)) [Freight Cost / Case ($)]
	,max(isnull(LEIPerCase, 0)) [LEI ($) / Case]
	, case
	   when VndrNum = 359958 and  OpCo = 177  then 
	       (
	       select sum(isnull(FrtPerCase, 0)*Cases)
	       from
            	tbl_testingfreight di
            	left outer join test.dbo.[OpCo Locations] o
            		on d.OpCo  =  o.OpCo
            	cross join (select distinct sourcinginitiative
            			from [freightPO-OPCO]) si
            where
            	SourcedCases  >  0
            	and di.SUVCVndrNum = d.SUVCVndrNum
            	and di.ItemNum = d.ItemNum
	       )
	   else null
	end
	
from
	tbl_testingfreight d
	left outer join test.dbo.[OpCo Locations] o
		on d.OpCo  =  o.OpCo
	cross join (select distinct sourcinginitiative 
			from [freightPO-OPCO]) si
where
	SourcedCases  >  0
 
group by
	d.OpCo
	,d.SUVCVndrNum
	,d.VndrNum
	,ItemNum
order by
	d.OpCo
	,ItemNum

Open in new window

0
 
Kevin CrossChief Technology OfficerCommented:
Based on your XLS, it seems like you would be better off making a view out of your original query and then do something like this:

SELECT *,
CASE VndrNum WHEN '359958' THEN ISNULL((SELECT [Freight Cost / Case ($)] FROM yourView v2 WHERE v2.VndrNum = v1.SUVCVndrNum AND v2.ItemNum = v1.ItemNum AND v2.OpCo = 177), 0.0)
ELSE 0.0 END AS [Addtl Freight Cost / Case ($)]
FROM yourView v1
WHERE OpCo <> 177

Then add up [Addtl Freight Cost / Case ($)] AND [Freight Cost / Case ($)] in spreadsheet or in query.  Just simplified here, but if you type out columns in select you can just place this case statement after the original freight/case column as a + and add right then and there.
0
 
mercybthomas74Author Commented:
i have the code working but i see duplicates , something were i did the grouping, please help...
I took the d.SUVCVndrNum  d.VndrNum off
select TOP 100 PERCENT
	d.OpCo
	,max(o.OpCoName) OpCoName
	,ItemNum
	,max(Description) Description
	,sum(Cases) [Case Volume]
	,sum(Weight) [Weight Volume]		
	,sum(FOB) [COGS ($)]
	,sum(isnull(FrtPerCase, 0)*Cases) [Freight Cost ($)]
	,sum(isnull(LEIPerCase, 0)*Cases) [LEI ($)]	
	, (max(isnull(FrtPerCase, 0)) ) + (case when VndrNum = '359958'  then 
	       		isnull((select max(FrtPerCase)
	       			from SourcingFreightDetail_VndrNum di
            		      	where
            				di.SourcedCases  >  0
            				and di.SUVCVndrNum = d.SUVCVndrNum
            				and di.ItemNum = d.ItemNum  
					and di.opco='177'  
			      	
				group by di.SUVCVndrNum,di.ItemNum ,di.opco),0)
	else 0
	end )[Freight Cost / Case ($)]
	
	,max(isnull(LEIPerCase, 0)) [LEI ($) / Case]
	,max(si.SourcingInitiative) SourcingInitiative
 
from
	SourcingFreightDetail_VndrNum d
	left outer join test.dbo.[OpCo Locations] o
		on d.OpCo  =  o.OpCo
	cross join (select distinct sourcinginitiative 
			from [freightPO-OPCO]) si
where
	d.SourcedCases  >  0 and 
	d.opco <>'177' 
	
 
group by
	d.OpCo
	,ItemNum
	,d.SUVCVndrNum 
	,d.VndrNum
 
order by
	d.OpCo
	,ItemNum

Open in new window

test.xls
0
Concerto's Cloud Advisory Services

Want to avoid the missteps to gaining all the benefits of the cloud? Learn more about the different assessment options from our Cloud Advisory team.

 
David ToddSenior DBACommented:
Hi,

You have removed d.SUVCVndrNum and d.VndrNum from the column list, but not from the group by. Suggest that you take them out of the group by and that should remove your duplicates.

Cheers
  David
0
 
mercybthomas74Author Commented:
i tried that , since it is used the subquery its asking for those 2 fields to be grouped
0
 
David ToddSenior DBACommented:
Hi,

Okay, then you are going to see apparent duplicates. Since they are in the group by, removing them from the field list like you have shouldn't alter the results as such.

That is, I don't understand how removing them suddenly makes the query work.

Cheers
  David
0
 
mercybthomas74Author Commented:
the error i get is:

Column 'd.SUVCVndrNum' is invalid in the select list because it is not contained in either an aggregate function or the GROUP BY clause.
Server: Msg 8120, Level 16, State 1, Line 1
Column 'd.VndrNum' is invalid in the select list because it is not contained in either an aggregate function or the GROUP BY clause.
0
 
mercybthomas74Author Commented:
how can i show one record, without having to add those 2 fields with total values,
I tried all ways either showing error or duplicates
0
 
Kevin CrossChief Technology OfficerCommented:
Have you completely tried my suggestion -- http:22782044?

It appears you are using pieces of it, but an important part of what I suggested was to make your original query into a view.  This is important as the subquery must exactly match the original query and you need to do the calculations / joins based on the already grouped and aliased data like it appears in your spreadsheet.  

If you are unwilling/able to use that approach, please advise and will go on from there but will probably need to be slightly adjusted as my solution was based on that fact.

Regards,
Kevin
0
 
mercybthomas74Author Commented:
mwvisa1: Actually I tried your suggention ,
I took off the d.SUVCVndrNum and d.VndrNum and thats when i get the error.
So i created the given script into a view and then sumed up the values in another view...
I guess that could be the best approach
0
 
Kevin CrossChief Technology OfficerCommented:
Yes, OR pull all the original data and extra column into Excel and SUM into Excel.  That way you only have the one view and your query to pull data in Excel does the additional logic.
0

Featured Post

Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

  • 5
  • 3
  • 3
Tackle projects and never again get stuck behind a technical roadblock.
Join Now