Best Query Approach

Question

I have a scenario to display data from 2 tables combined. First table Named 'DayTable' consists of daily plan and actual. Second table named 'MonthTable' consists of Monthly plan and actual. I need to display last 6months data and the current month daily
data.
My output is like
For this i wrote the query in 2 methods. One is using temp tables and another with out temp table. Please suggest the best apporach for this. Details are below
Approach 1: Using Temp Table
Declare @startdate date = CONVERT(DATE, DATEADD(dd, -DAY(DATEADD(MONTH, 0, GETDATE())) + 1, DATEADD(MONTH, 0, GETDATE())))
Declare @endDate date = DATEADD(DAY, -DAY(DATEADD(MONTH, 1, GETDATE())), DATEADD(MONTH, 1, GETDATE()))
CREATE TABLE #TEMP
PlanDate NVARCHAR(100),
[PastTrend - Plan] INT,
[PastTrend - Actual] INT,
[Current - Plan] INT,
[Current - Actual] INT,
;With cte
as
Select @startdate sDate
Union All
Select DATEADD(day,1,sDate) From cte where DATEADD(day,1,sDate) <= @endDate
INSERT INTO #TEMP
SELECT
REPLACE(CONVERT(CHAR(6), A.sDate, 106),' ',' - ') PlanDate
,NULL AS [PastTrend - Plan]
,NULL AS [PastTrend - Actual]
,SUM(B.PlanQuantity) AS [Current - Plan]
,SUM(B.Actual) AS [Current - Actual]
FROM cte A
LEFT OUTER JOIN DayTable B
ON A.sDate = CONVERT(DATE,B.PlanDate)
GROUP BY A.sDate
SELECT
FROM
SELECT
CONVERT(CHAR(3), datename(month,PlanMonth)) + ' ' + RIGHT(CONVERT(VARCHAR(4), YEAR(PlanMonth)), 2) AS PlanDate
,SUM(PlanQuantity) AS [PastTrend - Plan]
,SUM(Actual) AS [PastTrend - Actual]
,NULL AS [Current - Plan]
,NULL AS [Current - Actual]
FROM
MonthTable
WHERE CONVERT(DATE, PlanMonth) >= CONVERT(DATE, DATEADD(dd, -DAY(DATEADD(MONTH, 0, GETDATE())) + 1, DATEADD(MONTH, -6, GETDATE())))
group by PlanMonth
UNION ALL
SELECT
PlanDate
,[PastTrend - Plan]
,[PastTrend - Actual]
,[Current - Plan]
,[Current - Actual]
FROM
#TEMP
) T1
DROP TABLE #TEMP
When i use Actual execution plan, Query Cost (relative to the batch) : 90%
Approach 2: With out temp Table
Declare @startdate date = CONVERT(DATE, DATEADD(dd, -DAY(DATEADD(MONTH, 0, GETDATE())) + 1, DATEADD(MONTH, 0, GETDATE())))
Declare @endDate date = DATEADD(DAY, -DAY(DATEADD(MONTH, 1, GETDATE())), DATEADD(MONTH, 1, GETDATE()))
;With cte
as
Select @startdate sDate
Union All
Select DATEADD(day,1,sDate) From cte where DATEADD(day,1,sDate) <= @endDate
SELECT
T1.PlanDate
,T1.[PastTrend - Plan]
,T1.[PastTrend - Actual]
,T1.[Current - Plan]
,T1.[Current - Actual]
FROM
SELECT
A.sDate AS OriginalDate
,REPLACE(CONVERT(CHAR(6), A.sDate, 106),' ',' - ') PlanDate
,NULL AS [PastTrend - Plan]
,NULL AS [PastTrend - Actual]
,SUM(B.PlanQuantity) AS [Current - Plan]
,SUM(B.Actual) AS [Current - Actual]
FROM cte A
LEFT OUTER JOIN DayTable B
ON A.sDate = CONVERT(DATE,B.PlanDate)
GROUP BY A.sDate
UNION ALL
SELECT
PlanMonth AS OriginalDate
,CONVERT(CHAR(3), datename(month,PlanMonth)) + ' ' + RIGHT(CONVERT(VARCHAR(4), YEAR(PlanMonth)), 2) AS PlanDate
,SUM(PlanQuantity) AS [PastTrend - Plan]
,SUM(Actual) AS [PastTrend - Actual]
,NULL AS [Current - Plan]
,NULL AS [Current - Actual]
FROM
MonthTable
WHERE CONVERT(DATE, PlanMonth) >= CONVERT(DATE, DATEADD(dd, -DAY(DATEADD(MONTH, 0, GETDATE())) + 1, DATEADD(MONTH, -6, GETDATE())))
group by PlanMonth
) T1
ORDER BY T1.OriginalDate
Here Query Cost (relative to the batch) : 100%
So can you suggest the best one. Actually i wrote the second method to avoid temp tables. If any failed in the query after create temp table, it will deleted. To avoid like problems and for simplicty i prefer the second. one. But now i am confused
which is the best one in the performance wise also. 

Answer 1

p>I will try to chnage as per your suggestions. Actually here the presentation is fully dynamic and the header is always used from the column name. So only i try to give the name with hyphen.
I added the schema design of the table. The application uses multilingual so we use nvarchar. The tables are aggregated tables. The data populated daily or monthly using sql job
CREATE TABLE [dbo].[MonthPlan](
[IPCMonthlyVsActualId] [int] IDENTITY(1,1) NOT NULL,
[IPCPlanMonth] [datetime] NOT NULL,
[ModelId] [int] NOT NULL,
[ModelName] [nvarchar](50) NOT NULL,
[PlanQuantity] [int] NOT NULL,
[Actual] [int] NOT NULL,
[DifferenceQuantity] [int] NOT NULL,
[ReasonCode] [nvarchar](max) NULL,
[Remarks] [nvarchar](max) NULL,
[Createdby] [int] NOT NULL,
[CreatedDate] [datetime] NOT NULL,
[CreatedIP] [nvarchar](30) NOT NULL,
[Approval] [int] NULL,
[UnApproval] [int] NULL,
[ModelTypeName] [varchar](100) NULL,
[ModelCategoryName] [varchar](100) NULL,
[ModelColorName] [varchar](100) NULL,
[ModelGroupName] [varchar](100) NULL,
[ModelLocationName] [varchar](100) NULL,
[ModelTypeId] [int] NULL,
[ModelCategory] [int] NULL,
[ModelColor] [int] NULL,
[ModelGroup] [int] NULL,
[ModelLocation] [int] NULL
) ON [PRIMARY]
GO
CREATE TABLE [dbo].[DayTable](
[IPCDailyVsActualId] [int] IDENTITY(1,1) NOT NULL,
[IPCPlanDate] [datetime] NOT NULL,
[ModelID] [int] NOT NULL,
[ModelName] [nvarchar](50) NOT NULL,
[PlanQuantity] [int] NOT NULL,
[Actual] [int] NOT NULL,
[DifferenceQuantity] [int] NOT NULL,
[ValuesetparameterId] [int] NULL,
[ReasonCode] [nvarchar](20) NULL,
[Remarks] [nvarchar](200) NULL,
[CreatedBy] [int] NOT NULL,
[CreatedDate] [datetime] NOT NULL,
[CreatedIP] [nvarchar](30) NOT NULL,
[Cell1Cell2Map] [int] NULL,
[Cell3Actual] [int] NULL,
[FB] [int] NULL,
[Remarks2] [nvarchar](max) NULL,
[Remarks3] [nvarchar](max) NULL,
[Remarks4] [nvarchar](max) NULL,
[Remarks5] [nvarchar](max) NULL,
[Remarks6] [nvarchar](max) NULL,
[Remarks7] [nvarchar](max) NULL,
[Remarks8] [nvarchar](max) NULL,
[Remarks9] [nvarchar](max) NULL,
[Remarks10] [nvarchar](max) NULL,
[Remarks11] [nvarchar](max) NULL,
[Remarks12] [nvarchar](max) NULL,
[UpdatedBy] [int] NULL,
[UpdatedDate] [datetime] NULL,
[UpdatedIP] [nvarchar](30) NULL,
[ClosureFlag] [bit] NULL,
[UnApproval] [int] NULL,
[ModelTypeName] [varchar](100) NULL,
[ModelCategoryName] [varchar](100) NULL,
[ModelColorName] [varchar](100) NULL,
[ModelGroupName] [varchar](100) NULL,
[ModelLocationName] [varchar](100) NULL,
[ModelTypeId] [int] NULL,
[ModelCategory] [int] NULL,
[ModelColor] [int] NULL,
[ModelGroup] [int] NULL,
[ModelLocation] [int] NULL
) ON [PRIMARY]