Problem while using UNION in SQL

Question

1.00/5 (1 vote)

See more:

SQL-Server-2008R2

MY Sql Query

SQL

select CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120) as 'monthyear',
		SUM(amount) as 'Amount1',0 as 'Amount2' 
from MYtable 
where accountid=4 
group by CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120)
union
select	CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120) as 'monthyear',
		0 as 'Amount1',
		SUM(amount) as 'Amount2'
from MYtable
where accountid=3 
group by CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120)

but when i execute query i am getting data in this way..

SQL

MonthYear     Amount1     Amount2
Jan 2015      0           100
Jan 2015      100         0
Feb 2015      200         0
Feb 2015      0           300
Mar 2015      400         0
Mar 2015      0           500

But i want Data in this format

SQL

MonthYear     Amount1     Amount2
Jan 2015      100         100
Feb 2015      200         300
Mar 2015      400         500

Thx in advance...

What I have tried:

SQL

select CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120) as 'monthyear',
		SUM(amount) as 'Amount1',0 as 'Amount2' 
from MYtable 
where accountid=4 
group by CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120)
union
select	CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120) as 'monthyear',
		0 as 'Amount1',
		SUM(amount) as 'Amount2'
from MYtable
where accountid=3 
group by CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120)

Posted 22-Jun-16 1:14am

Ravi Sargam

Updated 22-Jun-16 1:53am

Magic Wonder

v2

Add a Solution

Comments

Kornfeld Eliyahu Peter 22-Jun-16 7:18am

UNION works as excepted, it is just the wrong tool...Some JOIN and/or CTE would be better...

Ravi Sargam 22-Jun-16 7:21am

how can i use my query with CTE? i am new to this.

Kornfeld Eliyahu Peter 22-Jun-16 7:22am

If you are new start with JOIN...

Ravi Sargam 22-Jun-16 7:23am

new to CTC not JOINs...

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Maciej Los · Accepted Answer · 2016-06-22T01:51:00

Solution 2

You don't need UNION's! You have to use CASE WHEN ... END.

So, the simplest way is:

SQL

SELECT CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120) as 'monthyear',
		SUM(CASE WHEN accountid=4 THEN amount ELSE 0 END) as 'Amount1',
		SUM(CASE WHEN accountid=3 THEN amount ELSE 0 END) as 'Amount2' 
FROM MYtable 
GROUP BY CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120)

Posted 22-Jun-16 1:51am

Maciej Los

Updated 22-Jun-16 1:56am

v3

Comments

ZurdoDev 22-Jun-16 8:01am

Good catch. I didn't go the extra mile to see why the union was there. +5.

Maciej Los 22-Jun-16 8:02am

Thank you, Ryan.

Ravi Sargam 22-Jun-16 8:26am

This one is short and simpl thank you.. maciej Los

Maciej Los 22-Jun-16 8:37am

You're very welcome.

CHill60 22-Jun-16 8:50am

5'd

Maciej Los 22-Jun-16 8:51am

Thank you, Caroline.

ZurdoDev · Accepted Answer · 2016-06-22T01:23:00

As Peter said, UNION is doing exactly what it is supposed to do. An easy way around this is to wrap what you have inside another SELECT statement. Something like:

SQL

SELECT x.monthyear, SUM(x.amount1) AS amount1, SUM(x.amount2) as amount2
FROM (
  select CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120) as   'monthyear',
  SUM(amount) as 'Amount1',0 as 'Amount2' 
  from MYtable 
  where accountid=4 
  group by CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120)
  union
  select	CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120)   as 'monthyear',
  0 as 'Amount1',
  SUM(amount) as 'Amount2'
  from MYtable
  where accountid=3 
  group by CONVERT(CHAR(4), dateofentry, 100) + CONVERT(CHAR(4), dateofentry, 120)
) x
GROUP BY x.monthyear