I have an issue with the procedure below.
Error is:
Msg 402, Level 16, State 1, Procedure Get_FormattedBankStatement, Line
87. The data types nvarchar and nvarchar are incompatible in the subtract operator.
USE [K2_Objects]
GO
/****** Object: StoredProcedure [K2RestSrv].[Get_FormattedBankStatement] Script Date: 2/27/2019 5:00:12 PM ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
ALTER PROCEDURE [K2RestSrv].[Get_FormattedBankStatement]
--Declaring input parameter variable
#OpeningBalance Decimal(18,2),
#jsonValue nvarchar(max)
AS
BEGIN
DECLARE
#TotalCount nvarchar(150),
#MinRowNum int, #MaxRowNum int,
#DebitRecord nvarchar(150),
#CreditRecord nvarchar(150),
#NewBalance nvarchar(150),
#PreviousBalance nvarchar(150);
--Creating Temp Table #GetRowID
Create table #GetStatement
(ID int identity(1,1),
PostDate nvarchar(150),
TransDate nvarchar(150),
ValueDate nvarchar(150),
TransID nvarchar(150),
Narration nvarchar(max),
Debit nvarchar(150),
Credit nvarchar(150),
Balance nvarchar(150));
--Inserting into TempTable #GetStatement Temp Table, from the select statement
INSERT INTO #GetStatement
(PostDate,
TransDate,
ValueDate,
TransID,
Narration,
Debit,
Credit,
Balance)
SELECT
max(case when name='post_date' then convert(nvarchar(150),StringValue) else '' end) as [PostDate],
max(case when name='tran_date' then convert(nvarchar(150),StringValue) else '' end) as [TranDate],
max(case when name='value_date' then convert(nvarchar(150),StringValue) else '' end) as [ValueDate],
max(case when name='tran_id' then convert(nvarchar(150),StringValue) else '' end) as [TransID],
max(case when name='narration' then convert(nvarchar(150),StringValue) else '' end) as [Narration],
max(case when name='debit' then convert(nvarchar(150),StringValue) else '' end) as [Debit],
max(case when name='credit' then convert(nvarchar(150),StringValue) else '' end) as [Credit],
max(case when name='balance' then convert(nvarchar(150),StringValue) else '' end) as [Balance]
FROM parseJSON
(
#jsonValue
)
WHERE ValueType IN ('string','real','int','object','array') and Object_ID is NULL
GROUP BY parent_ID;
--Selecting the first and Last RowNum from the TempTable
SET #MinRowNum = (SELECT Min(ID) FROM #GetStatement)
SET #MaxRowNum = (SELECT Max(ID) FROM #GetStatement)
SET #PreviousBalance = #OpeningBalance;
WHILE #MinRowNum < #MaxRowNum
BEGIN
SET #DebitRecord = (SELECT Debit FROM #GetStatement WHERE ID = #MinRowNum);
SET #CreditRecord = (SELECT Credit FROM #GetStatement WHERE ID = #MinRowNum);
SET #NewBalance = (#PreviousBalance - (#DebitRecord + #CreditRecord));
UPDATE #GetStatement SET Balance = #NewBalance WHERE ID = #MinRowNum;
SET #PreviousBalance = (Select Balance from #GetStatement where ID = #MinRowNum);
SET #MinRowNum = (#MinRowNum + 1);
END
Select ID, PostDate,TransDate,ValueDate,TransID,Narration,Debit,Credit,Balance from #GetStatement;
END
GO
Thanks for the support and insights shared.
I was able to overcome the issue, by changing the data type for all areas where I had Decimal(18,2) to money. The variable #OpeningBalance Decimal(18,2) was changed to #OpeningBalance money
Same was done for #DebitRecord, #CreditRecord, #NewBalance, #PreviousBalance
Likewise, 'Coalesce' was used for the calculation of the new balance, by coalescing the PreviousBalance, the CreditRecord, and the DebitRecord.
Change These DECLARES:
#NewBalance nvarchar(150),
#PreviousBalance nvarchar(150);
To
#NewBalance DECIMAL(10,2),
#PreviousBalance DECIMAL(10,2);
Related
How do I count the number of actual records returned by a group by query
For e.g
-- Exec SP_GET_ITEM_STOCK 0,10,NULL,'Charger'
ALTER PROCEDURE [dbo].[SP_GET_ITEM_STOCK]
#startRowIndex int ,
#pageSize int,
#ItemID bigint = null,
#ItemName varchar(250) = null
AS
BEGIN
DECLARE #SQL varchar(MAX)
DECLARE #SQLWHERE varchar(MAX)
SET #SQL = 'WITH DATA AS (
select
ROW_NUMBER() OVER (ORDER BY Item_ID) ''SNo'',
Item_ID,
Item_Name,
SUM(Inward) as Total_Purchase,
SUM(Outward) as Total_Sale,
(sum(Inward) - sum(outward))as Balance_Stock'
Set #SQLWHERE = ' from Item_Ledger_Details where Active = 1'
IF #ItemID IS NOT NULL and #ItemID <> ''
SET #SQLWHERE = #SQLWHERE + ' and Item_ID = ' + CONVERT(varchar,#ItemID) + ''
IF #ItemName IS NOT NULL and #ItemName <> ''
SET #SQLWHERE = #SQLWHERE + ' and Item_Name like ''%' + #ItemName + '%'''
SET #SQL = #SQL + #SQLWHERE + ' group by Item_ID,Item_Name) SELECT * FROM DATA WHERE SNo BETWEEN ' + CONVERT(Varchar,#startRowIndex) + ' AND ' + CONVERT(Varchar,#startRowIndex+#pageSize) + ' ORDER BY SNo'
EXEC(#SQL +';SELECT COUNT(*) ''Count'' '+ #SQLWHERE)
print(#SQL)
END
Which returns:
I need to count the above first result records to get 1 + 1 = 2 in second result where I get count = 48
Continue Anand Answer. I've just modify his query. Below query is solved my answer. But I think may be this query needs optimization.
-- SP_GET_ITEM_STOCK 0,10,NULL,NULL
ALTER PROCEDURE [dbo].[SP_GET_ITEM_STOCK]
#startRowIndex int,
#pageSize int,
#ItemID bigint = null,
#ItemName varchar(250) = null
AS
BEGIN
Declare #Temp Table (
SNo bigint,
Item_ID bigint,
Item_Name varchar(max),
Total_Purchase money,
Total_Sale money,
Balance_Stock money
);
WITH DATA AS (
select
ROW_NUMBER() OVER (ORDER BY Item_ID) as SNo,
Item_ID,
Item_Name,
SUM(Inward) as Total_Purchase,
SUM(Outward) as Total_Sale,
(sum(Inward) - sum(outward))as Balance_Stock
from Item_Ledger_Details
where Active = 1
and (coalesce(#ItemID, '') = '' or Item_ID = CONVERT(varchar,#ItemID))
and ( coalesce(#ItemName, '') = '' or Item_Name like '%' + #ItemName + '%')
group by Item_ID,
Item_Name
)
insert into #Temp
SELECT *
FROM DATA
WHERE SNo BETWEEN #startRowIndex AND #startRowIndex+#pageSize
ORDER BY SNo
select * from #temp
SELECT COUNT(*) as Count from #temp
END
There, I've cleaned it up:
ALTER PROCEDURE [dbo].[SP_GET_ITEM_STOCK]
#startRowIndex int,
#pageSize int,
#ItemID bigint = null,
#ItemName varchar(250) = null
AS
BEGIN
;WITH DATA AS (
select
ROW_NUMBER() OVER (ORDER BY Item_ID) as SNo,
Item_ID,
Item_Name,
SUM(Inward) as Total_Purchase,
SUM(Outward) as Total_Sale,
(sum(Inward) - sum(outward))as Balance_Stock
from Item_Ledger_Details
where Active = 1
and (coalesce(#ItemID, '') = '' or Item_ID = CONVERT(varchar,#ItemID))
and ( coalesce(#ItemName, '') = '' or Item_Name like '%' + #ItemName + '%')
group by Item_ID,
Item_Name
)
SELECT *
FROM DATA
WHERE SNo BETWEEN #startRowIndex AND #startRowIndex+#pageSize
ORDER BY SNo
SELECT COUNT(*) as Count
from DATA
END
Can I create a view or function using the below query? It uses temp tables. I am not able to create either a view or function.
--TEmp table--
IF OBJECT_ID('tempdb..#Enquiries') IS NOT NULL
DROP TABLE #Enquiries
GO
CREATE TABLE #Enquiries
(ID INT,PID INT,Name VARCHAR(50),Enquiries INT,EnquiryDate datetime)
INSERT INTO #Enquiries
SELECT ROW_NUMBER()
OVER (ORDER BY ProjectName) AS Row, (SELECT top 1 ITEM FROM DBO.split(e.InterestedProjects,',')) as Projects,P.ProjectName,COUNT(CAST(EnquiryDate AS DATE)) AS Enquiries,CAST(EnquiryDate AS DATE) AS EnquiryDate
FROM tbl_Enquiry e INNER JOIN
tbl_Projects AS P ON P.ProjectId = (SELECT TOP 1 ITEM FROM DBO.split(e.InterestedProjects,','))
WHERE e.IsActive=1 and e.isdeleted=0 AND p.IsActive=1 AND p.IsDeleted=0 AND P.ProjectId IN (SELECT TOP 1 ITEM FROM DBO.split(e.InterestedProjects,','))
GROUP BY e.InterestedProjects,P.ProjectName,CAST(EnquiryDate AS DATE)
--SiteVisits
IF OBJECT_ID('tempdb..#SiteVisit') IS NOT NULL
DROP TABLE #SiteVisit
GO
CREATE TABLE #SiteVisit
(ID INT,PID INT,Name VARCHAR(50),Sitevisits INT,PickUpDatetime datetime)
INSERT INTO #SiteVisit
SELECT ROW_NUMBER() OVER (ORDER BY ProjectName) AS Row,s.ProjectId,p.ProjectName As Name,count(sd.PickUpDatetime) AS Sitevisits,CAST(PickUpDatetime AS DATE) AS PickUpDatetime FROM tbl_SiteVisit s
INNER JOIN tbl_SiteVisitDetails sd ON s.SiteVisitId=sd.SiteVisitId
INNER JOIN tbl_Projects p ON p.ProjectId=s.ProjectId
WHERE s.IsActive=1 and s.isdeleted=0 AND sd.isactive=1 AND sd.isdeleted=0
GROUP BY s.ProjectId,sd.PickUpDatetime,p.ProjectName,CAST(PickUpDatetime AS DATE)
--Bookings
IF OBJECT_ID('tempdb..#Bookings') IS NOT NULL
DROP TABLE #Bookings
GO
CREATE TABLE #Bookings
(ID INT,PID INT,Name VARCHAR(50),Bookings INT,BookingDate datetime,Revenue money,Area float)
INSERT INTO #Bookings
SELECT ROW_NUMBER() OVER (ORDER BY ProjectName) AS Row, u.ProjectId AS ProjectId,p.ProjectName,count(u.ProjectId) AS Bookings,CAST(b.BookingDate AS DATE) AS BookingDate,SUM(b.TotalAmount) AS [Revenue],SUM(u.UnitArea) AS [Area] FROM tbl_Bookings b
INNER JOIN tbl_Unit u ON b.UnitId=u.UnitId
INNER JOIN tbl_Projects p on p.ProjectId=u.ProjectId
WHERE b.IsActive=1 AND b.IsDeleted=0 and u.IsActive=1 AND u.IsDeleted=0 AND u.Status = 'B'
GROUP BY u.ProjectId,p.ProjectName,CAST(b.BookingDate AS DATE),b.TotalAmount,u.UnitArea
--ORDER BY u.ProjectId
IF OBJECT_ID('tempdb..#T1') IS NOT NULL
DROP TABLE #T1
create TABLE #T1 (
PrimaryNo INT,
EnquiryDate Date,
Enquiries INT,
SiteVisits INT,
Bookings INT,
Revenue Money,
Area Float,
PID INT,
ProjectName nvarchar(max)
)
INSERT INTO #T1(PrimaryNo,EnquiryDate,Enquiries,PID,ProjectName)
SELECT ID,EnquiryDate,sum(enquiries) AS Enquiries,PID,Name FROM #Enquiries GROUP BY id,pid,Name,enquirydate
DECLARE #SVDate date
DECLARE #SV11 INT
DECLARE #BookingDate Date
DECLARE #Bookings11 INT
DECLARE #Revenue11 MONEY
DECLARE #Area11 FLOAT
DECLARE #intFlag_pw11 INT
SET #intFlag_pw11 = 1
DECLARE #TableCntw11 INT
DECLARE #Date Date
DECLARE Cur_SiteVisit CURSOR FAST_FORWARD FOR
SELECT PickUpDatetime FROM #SiteVisit
OPEN Cur_SiteVisit
FETCH NEXT FROM Cur_SiteVisit INTO #Date
DECLARE #ProjectId INT
DECLARE #Count INT = 1
WHILE ##FETCH_STATUS = 0
BEGIN
SET #ProjectId = (SELECT PID FROM #SiteVisit WHERE ID = #Count)
SET #SVDate = ISNULL((SELECT CAST(PickUpDatetime AS DATE) FROM #SiteVisit
WHERE CAST(PickUpDatetime AS DATE) = #Date AND PID = #ProjectId
GROUP BY PickUpDatetime),'-')
SET #SV11 = ISNULL((SELECT Sitevisits FROM #SiteVisit
WHERE CAST(PickUpDatetime AS DATE) = #Date AND PID = #ProjectId
GROUP BY Sitevisits),0)
EXEC ('UPDATE #T1 SET SiteVisits = ' + #SV11 + ' WHERE EnquiryDate = ' + ''''+ #SVDate +''' AND PID =' + #ProjectId)
FETCH NEXT FROM Cur_SiteVisit INTO #Date
SET #Count = #Count + 1
END
CLOSE Cur_SiteVisit
DEALLOCATE Cur_SiteVisit
--For Bookings
DECLARE #Date1 Date
DECLARE Cur_Bookings CURSOR FAST_FORWARD FOR
SELECT BookingDate FROM #Bookings
OPEN Cur_Bookings
FETCH NEXT FROM Cur_Bookings INTO #Date1
DECLARE #ProjectId1 INT
DECLARE #Count1 INT = 1
WHILE ##FETCH_STATUS = 0
BEGIN
SET #ProjectId1 = (SELECT PID FROM #Bookings WHERE ID = #Count1)
SET #Bookings11 = ISNULL((SELECT TOP 1 Bookings FROM #Bookings
WHERE CAST(BookingDate AS DATE) = #Date1 AND PID = #ProjectId1
GROUP BY Bookings),0)
SET #BookingDate = ISNULL((SELECT TOP 1 CAST(BookingDate AS DATE) FROM #Bookings
WHERE CAST(BookingDate AS DATE) = #Date1 AND PID = #ProjectId1
GROUP BY CAST(BookingDate AS DATE)),'-')
SET #Revenue11 = ISNULL((SELECT TOP 1 Revenue FROM #Bookings
WHERE CAST(BookingDate AS DATE) = #Date1 AND PID = #ProjectId1
GROUP BY Revenue),0)
SET #Area11 = ISNULL((SELECT TOP 1 Area FROM #Bookings
WHERE CAST(BookingDate AS DATE) = #Date1 AND PID = #ProjectId1
GROUP BY Area),0)
EXEC ('UPDATE #T1 SET Bookings = ' + #Bookings11 + ',Revenue=' + #Revenue11 + ',Area = ' + #Area11 + ' WHERE EnquiryDate = ' + '''' + #BookingDate + ''' AND PID =' + #ProjectId1)
FETCH NEXT FROM Cur_Bookings INTO #Date1
SET #Count1 = #Count1 + 1
END
CLOSE Cur_Bookings
DEALLOCATE Cur_Bookings
Select * from #T1
You can't create a view from that, if only because you're doing a number of inserts and other actions. A view is created from a single query.
With regard to user-defined functions: you cannot use dynamic SQL, which you are doing:
EXEC ('UPDATE #T1 SET Bookings = ' + #Bookings11 + ',Revenue=' + #Revenue11 +
',Area = ' + #Area11 + ' WHERE EnquiryDate = ' + '''' + #BookingDate +
''' AND PID =' + #ProjectId1)
You also can't use temp tables. Both of these limitations are documented here: http://msdn.microsoft.com/en-us/library/ms191320.aspx.
But, you can do all of these things in a stored procedure. It is also possible to direct the output of a stored procedure to a temp table or table variable, allowing you to use the output in another query:
INSERT INTO dbo.MyTable
(MyTableId, Column1, Column2) -- arbitrary-chosen column names
EXEC dbo.MyStoredProcedure
SELECT *
FROM dbo.MyTable
WHERE Column1 = 'Some Value'
Finally, you might be able to rework your SQL to work with table variables rather than temp tables, which are allowed. It also looked to me like your dynamic SQL didn't need to be dynamic, so you might be able to eliminate that as well.
Hope this helps.
Create table TmpEmp(Empcode int,Ename nvarchar(50),department nvarchar(50))
Insert into TmpEmp values(1001,'Scott','IT')
Insert into TmpEmp values(1002,'Peter','IT')
Insert into TmpEmp values(1003,'Ricky','HR')
select * from TmpEmp
Declare #Department nvarchar(50)
Set #Department = '''IT'',''HR'''
--Set #Department = 'ALL'
--print #Department
select * from TMPEMP where (1=1)
and Department in
(case #Department when 'ALL' then Department else #Department End )
First of all your #Department variable has no string ALL
Set #Department = '''IT'',''HR'''
Second, it's comma separated strings; so you just can't use = operator. rather you will have to use LIKE operator. If I just change your posted code as below; result will be returned
Declare #Department nvarchar(50);
Set #Department = '''IT'',''ALL''';
select * from TMPEMP where (1=1)
and Department in
(case when #Department like '%ALL%' then Department else #Department End )
Change your query to use IN operator. See this fiddle demo http://sqlfiddle.com/#!3/4d9c9/20
select * from TMPEMP
where Department in (case when #Department not in ('ALL')
then Department else #Department End )
From the looks of it I've wrapped everything in the proper BEGIN...END statements, however the code goes through and executes almost every single line of code. I've done print statements too to make sure that both #rows variables actually do contain values greater than 0. Can anyone help point me in the right direction?
IF #rows > '0'
--This agent's Tax ID number has been found to exist in AgentIdentification table
BEGIN
--Set UniqueAgentId according to mapped value from AgentIdentification table
SELECT #uniqueAgentId = UniqueAgentId
FROM AgentIdentification
WHERE AgentTaxId = #ssn
--Check to make sure this record exists in UniqueAgentIdToAgentId table
SELECT #rows = COUNT(*)
FROM UniqueAgentIdToAgentId
WHERE UniqueAgentId = #uniqueAgentId and AgentId = #agentId
PRINT #rows
IF #rows > 0
--Record exists in UniqueAgentIdToAgentId table
--Check to make sure correct UniqueAgentId is mapped to correct AgentId and vice versa
BEGIN
SELECT #agentIdRows = COUNT(AgentId)
FROM UniqueAgentIdToAgentId
WHERE UniqueAgentId = #uniqueAgentId and AgentId <> #agentId
SELECT #uniqueIdRows = COUNT(UniqueAgentId)
FROM UniqueAgentIdToAgentId
WHERE AgentId = #agentId and UniqueAgentId <> #uniqueAgentId
IF #uniqueIdRows = 0 AND #agentIdRows = 0
BEGIN
SET #returnValue = 1
END
ELSE IF #agentIdRows = 0 AND #uniqueIdRows > 0
BEGIN
SET #returnValue = 2
END
ELSE
BEGIN
SET #returnValue = 3
END
END
--Record does not exist in UniqueAgentIdToAgentId and will be added
ELSE
BEGIN
INSERT INTO UniqueAgentIdToAgentId (UniqueAgentId, AgentId, CompanyCode, LastChangeOperator, LastChangeDate)
VALUES (#uniqueAgentId, #agentId, #companyCode, #lastChangeOperator, #LastChangeDate)
SET #returnValue = 4
END
END
ELSE
BEGIN TRANSACTION
--This agent Tax ID number does not exist on AgentIdentification table
--Add record into Agent and AgentIdentification table
INSERT INTO Agent (EntityType, FirstName, LastName, NameSuffix, CorporateName, LastChangeOperator, LastChangeDate)
VALUES (#entityType, #firstName, #lastname, '', #corporateName, #lastChangeOperator, #LastChangeDate)
SELECT #uniqueAgentId = ##IDENTITY
SELECT UniqueAgentId
FROM Agent
INSERT INTO AgentIdentification (UniqueAgentId, TaxIdType, AgentTaxId, LastChangeOperator, LastChangeDate)
VALUES (#uniqueAgentId, #taxIdType, #ssn, #lastChangeOperator, #lastChangeDate)
--Check to make sure this record exists in UniqueAgentIdToAgentId table
SELECT #rows = COUNT(*)
FROM UniqueAgentIdToAgentId
WHERE UniqueAgentId = #uniqueAgentId and AgentId = #agentId
IF #rows > 0
--Record exists in UniqueAgentIdToAgentId table
--Check to make sure correct UniqueAgentId is mapped to correct AgentId and vice versa
BEGIN
SELECT #agentIdRows = COUNT(AgentId)
FROM UniqueAgentIdToAgentId
WHERE UniqueAgentId = #uniqueAgentId and AgentId <> #agentId
SELECT #uniqueIdRows = COUNT(UniqueAgentId)
FROM UniqueAgentIdToAgentId
WHERE AgentId = #agentId and UniqueAgentId <> #uniqueAgentId
IF #uniqueIdRows = 0 AND #agentIdRows = 0
BEGIN
SET #returnValue = 5
END
ELSE IF #agentIdRows = 0 AND #uniqueIdRows > 0
BEGIN
SET #returnValue = 6
END
ELSE
BEGIN
SET #returnValue = 7
END
END
--Record does not exist in UniqueAgentIdToAgentId and will be added
ELSE
BEGIN
INSERT INTO UniqueAgentIdToAgentId (UniqueAgentId, AgentId, CompanyCode, LastChangeOperator, LastChangeDate)
VALUES (#uniqueAgentId, #agentId, #companyCode, #lastChangeOperator, #LastChangeDate)
SET #returnValue = 8
END
COMMIT TRANSACTION
I think this:
ELSE
BEGIN TRANSACTION
Needs to be this:
ELSE
BEGIN
BEGIN TRANSACTION
And this:
COMMIT TRANSACTION
Needs to be this:
COMMIT TRANSACTION
END
I have this SQL query which is valid but always times out.
CREATE PROCEDURE Sibm_getstudentdivisionattendancereport2 (
#course_id INT,
--#Subject_Id int,
#academic_year INT,
#division_id INT,
#semister_id INT,
#fromdate1 DATETIME,
#todate1 DATETIME)
AS
BEGIN
DECLARE #fromdate VARCHAR(100)
DECLARE #todate VARCHAR(100)
SET #fromdate=CONVERT(VARCHAR(10), #fromdate1, 105)
SET #todate=CONVERT(VARCHAR(10), #todate1, 105)
CREATE TABLE #getstudentmarks
(
id INT NOT NULL,
roll_no VARCHAR(100),
stud_id INT,
stud_name VARCHAR(200),
stud_last_name VARCHAR(200),
subject_name VARCHAR(200),
marks INT,
student_id INT,
division_id INT,
division VARCHAR(50),
attended VARCHAR(12),
exempted VARCHAR(12),
total_attendance VARCHAR(12),
attendancepercentage VARCHAR(30),
subject_id INT
)
CREATE TABLE #getstudentmarks1
(
id INT IDENTITY(1, 1) NOT NULL,
roll_no VARCHAR(100),
stud_id INT,
stud_name VARCHAR(200),
stud_last_name VARCHAR(200),
subject_name VARCHAR(200),
marks INT,
student_id INT,
division_id INT,
division VARCHAR(50),
attended VARCHAR(12),
exempted VARCHAR(12),
total_attendance VARCHAR(12),
attendancepercentage VARCHAR(30),
subject_id INT
)
CREATE TABLE #divisionlist
(
division_id INT
)
CREATE TABLE #getstudent_attended
(
id INT IDENTITY(1, 1) NOT NULL,
student_id INT,
exempted VARCHAR(12),
attended VARCHAR(12)
)
DECLARE #Total_Lectures INT
DECLARE #Student_Id INT
DECLARE #TotalConductedLecture INT
DECLARE #TotalExempted INT
INSERT INTO #getstudentmarks1
(roll_no,
student_id,
stud_name,
stud_last_name)
SELECT DISTINCT( SA.roll_no ),
SA.student_id AS id,
SR.first_name,
SR.last_name
FROM tblstudent_academic_record SA,
tblstudent_record SR
WHERE SR.student_id = SA.student_id
AND SR.student_id IN (SELECT SAR.student_id
FROM tblstudent_academic_record SAR
INNER JOIN tblcoursesemester_subject_student
CSS
ON SAR.student_id = CSS.student_id
AND SAR.current_record = 1
AND SAR.course_id = #course_id
AND
SAR.semister_id = #semister_id
AND
SAR.division_id = #division_id
AND
SAR.academic_year = #academic_year)
CREATE TABLE #temp
(
id INT,
subject_name VARCHAR(100)
)
DECLARE #Master_Id INT
SET #Master_Id = 0
IF EXISTS(SELECT *
FROM tblcoursesemester_main
WHERE academic_year = #Academic_Year
AND course_id = #Course_Id
AND semester_id = #semister_id)
BEGIN
SET #Master_Id = (SELECT TOP 1 cs_id
FROM tblcoursesemester_main
WHERE academic_year = #Academic_Year
AND course_id = #Course_Id
AND semester_id = #semister_id)
END
INSERT INTO #temp
SELECT id,
subject_name
FROM tblsubject
WHERE id IN (SELECT subject_id
FROM tblcoursesemester_subject
WHERE is_archieve IS NULL
AND cs_id = #Master_Id)
AND isarchive IS NULL
ORDER BY subject_name
DECLARE #tempid INT
DECLARE #sub VARCHAR(max)
DECLARE c1 CURSOR FOR
SELECT student_id
FROM #getstudentmarks1
OPEN c1
FETCH next FROM c1 INTO #Student_Id
WHILE ##FETCH_STATUS = 0
BEGIN
DECLARE c2 CURSOR FOR
SELECT id
FROM #temp
OPEN c2
FETCH next FROM c2 INTO #tempid
WHILE ##FETCH_STATUS = 0
BEGIN
UPDATE #getstudentmarks1
SET subject_name = (SELECT subject_name
FROM #temp
WHERE id = #tempid),
subject_id = #tempid
WHERE student_id = #Student_Id
FETCH next FROM c2 INTO #tempid
END
CLOSE c2
DEALLOCATE c2
FETCH next FROM c1 INTO #Student_Id
END
CLOSE c1
DEALLOCATE c1
--select * from #TEMP
--SET IDENTITY_INSERT #GetStudentMarks1 ON
INSERT INTO #getstudentmarks
(id,
roll_no,
stud_id,
stud_name,
stud_last_name,
subject_name,
marks,
student_id,
division_id,
division,
attended,
exempted,
total_attendance,
attendancepercentage,
subject_id)
SELECT #getstudentmarks1.id,
roll_no,
stud_id,
stud_name,
stud_last_name,
#temp.subject_name,
marks,
student_id,
division_id,
division,
attended,
exempted,
total_attendance,
attendancepercentage,
#temp.id
FROM #getstudentmarks1
CROSS JOIN #temp
-- select * from #GetStudentMarks cross join #TEMP
--SET IDENTITY_INSERT #GetStudentMarks1 OFF
--select * from #GetStudentMarks
---------calculate percentage-------------------
INSERT INTO #divisionlist
SELECT divisionid
FROM tblschedulemaster
WHERE academicyear = #academic_year
AND courseid = #Course_Id
AND semesterid = #Semister_Id
DECLARE #subject_id INT
DECLARE c1 CURSOR FOR
SELECT student_id
FROM #getstudentmarks
OPEN c1
FETCH next FROM c1 INTO #Student_Id
WHILE ##FETCH_STATUS = 0
BEGIN
DECLARE c3 CURSOR FOR
SELECT subject_id
FROM #getstudentmarks
OPEN c3
FETCH next FROM c3 INTO #subject_id
WHILE ##FETCH_STATUS = 0
BEGIN
-- GET TOTAL LECTURES OF EACH STUDENT
SELECT #Total_Lectures = Count([id])
FROM tblfacultyschedule
WHERE subjectid = #subject_id
AND sch_mas_id IN (SELECT [id]
FROM tblschedulemaster
WHERE academicyear = #academic_year
AND courseid = #Course_Id
AND semesterid = #Semister_Id
AND divisionid IN (
-- Changed Divisions list Query Starts
SELECT
divisionid
FROM
tblschedulemaster
WHERE
id IN (SELECT DISTINCT
sch_mas_id
FROM
tblfacultyschedule
WHERE
[date] >= #fromdate
AND [date] <= #todate
AND
id IN (SELECT DISTINCT
schedule_id
FROM
tblstudentattendence
WHERE
student_id = #Student_Id
AND schedule_id IN (SELECT
id
FROM
tblfacultyschedule
WHERE
subjectid =
#subject_id
AND
sch_mas_id IN(SELECT
id
FROM
tblschedulemaster
WHERE
academicyear =
#academic_year
AND
courseid =
#Course_Id
AND
semesterid
=
#Semister_Id))
))
-- Changed Divisions list Query Ends
))
AND COALESCE(other_activity, '') NOT LIKE '%LUNCH%'
--print 'subject id'
--print #subject_id
--print 'totol att'
--print #Total_Lectures
-- Total Conducted Lectures
SET #TotalConductedLecture =
(SELECT
COALESCE(Count(is_attended), 0) AS Attended
FROM tblstudentattendence
WHERE student_id = #Student_id
AND is_attended = 'YES'
AND
schedule_id IN (SELECT [id]
FROM
tblfacultyschedule
WHERE
[date] >= #fromdate
AND [date] <= #todate
AND subjectid = #subject_id
AND
sch_mas_id IN (SELECT [id]
FROM
tblschedulemaster
WHERE
academicyear = #academic_year
AND courseid = #course_id
AND semesterid = #semister_id
AND divisionid IN (SELECT
division_id
FROM
#divisionlist))
AND
COALESCE(other_activity, '')
NOT
LIKE
'%LUNCH%'
AND
subjectid IN (SELECT DISTINCT
subject_id
FROM
tblcoursesemester_subject
WHERE
cs_subject_id IN (SELECT
cs_subject_id
FROM
tblcoursesemester_subject_student
WHERE
cs_subject_id IN (SELECT
cs_subject_id
FROM
tblcoursesemester_subject
WHERE
cs_id IN (SELECT cs_id
FROM
tblcoursesemester_main
WHERE
academic_year = #academic_year
AND course_id = #course_id
AND semester_id = #semister_id
)
AND is_archieve IS NULL)
AND student_id = #Student_id
AND is_archieve IS NULL)
AND is_archieve IS NULL)))
--print 'attended'
-- print #TotalConductedLecture
-- NOW COUNT Exempted LECTURES
SET #TotalExempted=
(SELECT COALESCE(Count(is_attended), 0) AS
Attended
FROM tblstudentattendence
WHERE student_id = #Student_id
AND is_attended = 'EXEMPTED'
AND schedule_id IN (SELECT [id]
FROM
tblfacultyschedule
WHERE
[date] >= #fromdate
AND [date] <= #todate
AND subjectid =
#subject_id
AND sch_mas_id IN (SELECT
[id]
FROM
tblschedulemaster
WHERE
academicyear =
#academic_year
AND
courseid =
#course_id
AND
semesterid =
#semister_id
AND
divisionid IN (
SELECT
division_id
FROM
#divisionlist))
AND
COALESCE(other_activity, '') NOT
LIKE
'%LUNCH%'
AND
subjectid IN (SELECT DISTINCT
subject_id
FROM
tblcoursesemester_subject
WHERE
cs_subject_id IN (SELECT
cs_subject_id
FROM
tblcoursesemester_subject_student
WHERE
cs_subject_id IN (SELECT
cs_subject_id
FROM
tblcoursesemester_subject
WHERE
cs_id IN (SELECT cs_id
FROM
tblcoursesemester_main
WHERE
academic_year = #academic_year
AND course_id = #course_id
AND semester_id = #semister_id
)
AND is_archieve IS NULL)
AND student_id = #Student_id
AND is_archieve IS NULL)
AND is_archieve IS NULL)))
-- print 'not attended'
-- print #TotalExempted
UPDATE #getstudentmarks
SET #getstudentmarks.total_attendance = #Total_Lectures,
#getstudentmarks.attended = #TotalConductedLecture,
#getstudentmarks.exempted = #TotalExempted
WHERE student_id = #Student_Id
AND subject_id = #subject_id
FETCH next FROM c3 INTO #subject_id
END
CLOSE c3
DEALLOCATE c3
FETCH next FROM c1 INTO #Student_Id
END
CLOSE c1
DEALLOCATE c1
-- CURSOR ENDS
--select * from #GetStudentMarks
UPDATE #getstudentmarks
SET #getstudentmarks.division = tbldivision.division
FROM #getstudentmarks,
tbldivision
WHERE #getstudentmarks.division_id = tbldivision.division_id
UPDATE #getstudentmarks
SET #getstudentmarks.attended = #getstudent_attended.attended
FROM #getstudentmarks,
#getstudent_attended
WHERE #getstudentmarks.student_id = #getstudent_attended.student_id
UPDATE #getstudentmarks
SET #getstudentmarks.exempted = #getstudent_attended.exempted
FROM #getstudentmarks,
#getstudent_attended
WHERE #getstudentmarks.student_id = #getstudent_attended.student_id
UPDATE #getstudentmarks
SET attended = Isnull(attended, '0')
UPDATE #getstudentmarks
SET exempted = Isnull(exempted, '0')
UPDATE #getstudentmarks
SET total_attendance = Isnull(total_attendance, '0')
--UPDATE #GetStudentMarks SET #GetStudentMarks.AttendancePercentage=CAST(CAST((CAST(#GetStudentMarks.Attended AS NUMERIC(32,2)) * 100)/CAST(#GetStudentMarks.Total_Attendance AS NUMERIC(32,2)) AS NUMERIC(32,2)) AS VARCHAR(30))
--WHERE #GetStudentMarks.Total_Attendance <> '0'
UPDATE #getstudentmarks
SET
#getstudentmarks.attendancepercentage = Cast(
Cast(
Cast(( Cast(#getstudentmarks.attended AS NUMERIC(32, 2)) * 100 ) +
(
Cast(
#getstudentmarks.exempted AS
NUMERIC(32, 2)) *
100 ) AS NUMERIC(32, 2)) /
Cast
(
#getstudentmarks.total_attendance AS
NUMERIC(
32, 2)) AS NUMERIC(32, 2)) AS
VARCHAR
(30))
WHERE #getstudentmarks.total_attendance <> '0'
--+ '' + '%'
UPDATE #getstudentmarks
SET attendancepercentage = Isnull(attendancepercentage, '0.00')
--UPDATE #GetStudentMarks SET #GetStudentMarks.AttendancePercentage = '0 %' WHERE #GetStudentMarks.AttendancePercentage IS NULL
-------------end calculation---------------------
DECLARE #Roll_No VARCHAR(20),
#Stud_Id VARCHAR(20),
#Stud_Name VARCHAR(200),
#Subject_Name VARCHAR(200),
#Stud_Last_Name VARCHAR(200),
#AttendancePercentage VARCHAR(200)
-- select * from #GetStudentMarks
DECLARE #cntColm INT
SELECT #cntColm = Count(roll_no)
FROM #getstudentmarks
GROUP BY roll_no
CREATE TABLE #subject
(
sub_name VARCHAR(200)
)
INSERT INTO #subject
(sub_name)
SELECT DISTINCT( subject_name )
FROM #getstudentmarks
--print #cntColm
CREATE TABLE #getmarkssheet
(
student_id VARCHAR(100),
roll_no VARCHAR(100),
student_name VARCHAR(200)
)
--select * from #Subject
--------------- Cursor To add Columns As a Subject Name-------------------
-- print'text'
DECLARE #Stubject VARCHAR(max)
DECLARE curaddcol CURSOR FOR
SELECT sub_name
FROM #subject
OPEN curaddcol
FETCH next FROM curaddcol INTO #Stubject
WHILE ##FETCH_STATUS = 0
BEGIN
--print #Stubject
DECLARE #column VARCHAR(max)
SELECT #column = Replace(#Stubject, '&', '')
SELECT #column = Replace(#column, 'and', '')
SELECT #column = Replace(#column, '.', '')
SELECT #column = Replace(#column, ',', '')
SELECT #column = Replace(#column, ' ', '')
SELECT #column = Replace(#column, '-', '')
SELECT #column = Replace(#column, '(', '')
SELECT #column = Replace(#column, ')', '')
SELECT #column = Replace(#column, '/', '')
SELECT #column = Replace(#column, '//', '')
SELECT #column = Replace(#column, '\', '')
SELECT #column = Replace(#column, ':', '')
--print 'columnname' + #column
IF( #column <> '' )
BEGIN
EXEC('ALTER TABLE #GetMarksSheet ADD ' + #column +
' VARCHAR(max)')
--print #column
END
SET #cntColm=#cntColm - 1
FETCH next FROM curaddcol INTO #Stubject
END
---------------------------- Cursor Ends------------------------------------------
DECLARE cursubject CURSOR FOR
SELECT roll_no,
student_id,
stud_name,
stud_last_name,
subject_name,
attendancepercentage
FROM #getstudentmarks
OPEN cursubject
FETCH next FROM cursubject INTO #Roll_No, #Stud_Id, #Stud_Name,
#Stud_Last_Name, #Subject_Name, #AttendancePercentage
-- while #cntColm >0
WHILE ##FETCH_STATUS = 0
BEGIN
DECLARE #ColName VARCHAR(max)
SELECT #ColName = Replace(#Subject_Name, '&', '')
SELECT #ColName = Replace(#ColName, 'and', '')
SELECT #ColName = Replace(#ColName, ',', '')
SELECT #ColName = Replace(#ColName, '.', '')
SELECT #ColName = Replace(#ColName, ' ', '')
SELECT #ColName = Replace(#ColName, '-', '')
SELECT #ColName = Replace(#ColName, '(', '')
SELECT #ColName = Replace(#ColName, ')', '')
SELECT #ColName = Replace(#ColName, '/', '')
SELECT #ColName = Replace(#ColName, '//', '')
SELECT #ColName = Replace(#ColName, '\', '')
SELECT #ColName = Replace(#ColName, ':', '')
DECLARE #MarkInPer VARCHAR(max)
SET #MarkInPer=Cast(#AttendancePercentage AS VARCHAR(max))
IF NOT EXISTS (SELECT student_id
FROM #getmarkssheet
WHERE student_id = #Stud_Id)
BEGIN
DECLARE #FullName VARCHAR(400)
--print #AttendancePercentage
--print #ColName+'----' +Cast(#MarkInPer as varchar(max)) +'----'
-- select #MarkInPer=COALESCE(#MarkInPer,0)
-- print #Roll_No + 'roll num'
EXEC('insert into #GetMarksSheet(Student_Id,'+ #ColName +
') values ('+
#Stud_Id +','+#MarkInPer + ')')
-- print #Stud_Name
UPDATE #getmarkssheet
SET student_name = #Stud_Name
WHERE student_id = #Stud_Id
--exec('UPDATE #GetMarksSheet SET Student_Name='+#Stud_Name +' where Roll_No='+#Roll_No)
END
ELSE
EXEC('UPDATE #GetMarksSheet SET '+ #ColName +' = '+#MarkInPer +
' where Student_Id='+#Stud_Id)
SET #cntColm=#cntColm - 1
FETCH next FROM cursubject INTO #Roll_No, #Stud_Id, #Stud_Name,
#Stud_Last_Name, #Subject_Name, #AttendancePercentage
END
SELECT *
FROM #getmarkssheet
END
Ouch. Do it without temp tables and try it in one statement. It looks doable.
Your approach kills the query optimizer's ability to optimize because you give it detailed steps including a (OUCH) cursoer which means single processor loop behavior as you DEMAND it.
This is NOT sql, this is a perversion - sql deals with sets.
I am sure you CAN describ the result in one statement, without the use of temporary tables, and the result may surprise you performance wise ;)