Solved

Stored procedure with a parameter

Posted on 2016-10-27
6
43 Views
Last Modified: 2016-10-27
Hello,
Need to add a where clause to a stored procedure only if the column is present in the table .
ALTER PROCEDURE  [dbo].[M_SelectMaxVal_FrmCol]
(
     @tableName varchar(100) = null,
	 @ColumnName1 varchar(100) = null
	
)
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from
	-- interfering with SELECT statements.
	SET NOCOUNT ON;  	
	
	--   declare @SQL varchar(500) = null
	--  -- NONCLUSTERED INDEX [NIX__UNQ__UID_]

	--SET @SQL = 'SELECT MAX ('    
        
	--		SET @SQL = @SQL +  '[' +  @ColumnName1 + ']'  + ' +1 ) ' +   + ' FROM  [dbo].[' + @tableName + '] '  		
	
	--  SET @SQL = @SQL  	
	  
	--  EXEC(@SQL)


	  DECLARE @SQL VARCHAR(1000) = null


SET @SQL = '
	
		SELECT 
		
				CASE 
						WHEN CAST(GETDATE() AS DATE) = DATEADD(yy, DATEDIFF(yy, 0, GETDATE()), 0) THEN 1 
				ELSE 
				
						MAX ([' +  @ColumnName1 + ']'  + ' +1 ) END Output'  
				
				+ ' FROM  [dbo].[' + @tableName + ']'

			  
EXECUTE(@SQL)
		
   
	
END

Open in new window


Change it to
ALTER PROCEDURE  [dbo].[M_SelectMaxVal_FrmCol]
(
     @tableName varchar(100) = null,
	 @ColumnName1 varchar(100) = null

	
)
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from
	-- interfering with SELECT statements.
	SET NOCOUNT ON;  	
	
	--   declare @SQL varchar(500) = null
	--  -- NONCLUSTERED INDEX [NIX__UNQ__UID_]

	--SET @SQL = 'SELECT MAX ('    
        
	--		SET @SQL = @SQL +  '[' +  @ColumnName1 + ']'  + ' +1 ) ' +   + ' FROM  [dbo].[' + @tableName + '] '  		
	
	--  SET @SQL = @SQL  	
	  
	--  EXEC(@SQL)


	  DECLARE @SQL VARCHAR(1000) = null


SET @SQL = '
	
		SELECT 
		
				CASE 
						WHEN CAST(GETDATE() AS DATE) = DATEADD(yy, DATEDIFF(yy, 0, GETDATE()), 0) THEN 1 
				ELSE 
				
						MAX ([' +  @ColumnName1 + ']'  + ' +1 ) END Output'  
				
				+ ' FROM  [dbo].[' + @tableName + ']'


-----Add this only if year_ref column is present in the table 

SET @SQL = @Sql  +'where year_ref ' = '22'
			  
EXECUTE(@SQL)
		
   
	
END

Open in new window

0
Comment
Question by:RIAS
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 3
  • 3
6 Comments
 
LVL 51

Expert Comment

by:Vitor Montalvão
ID: 41861873
You'll need to check that column name in the sys.columns view. If exists then add the WHERE clause:
ALTER PROCEDURE  [dbo].[M_SelectMaxVal_FrmCol]
(
 @tableName varchar(100) = null,
 @ColumnName1 varchar(100) = null	
)
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from interfering with SELECT statements.
	SET NOCOUNT ON;  	
	DECLARE @SQL VARCHAR(1000) = null
	
	SET @SQL = 'SELECT CASE 
			WHEN CAST(GETDATE() AS DATE) = DATEADD(yy, DATEDIFF(yy, 0, GETDATE()), 0) THEN 1 
			ELSE MAX ([' +  @ColumnName1 + ']'  + ' +1 ) END Output'  
		+ ' FROM  [dbo].[' + @tableName + ']'

	IF EXISTS(SELECT 1
		FROM sys.tables t
			INNER JOIN sys.columns c ON t.object_id=c.object_id
		WHERE t.name = @tableName AND c.name = 'year_ref')
		-----Add this only if year_ref column is present in the table 
		SET @SQL = @Sql  +' where year_ref = 22'
			  
	EXECUTE(@SQL)
	
END

Open in new window

0
 

Author Comment

by:RIAS
ID: 41861884
Thanks Vitor,

Is there a way where where year_ref take current year and checks in the look up table for the number my lookup table is like this:
table _YearLookup
Columns are  year_no and year_ref

0	0
1	1995
2	1996
3	1997
4	1998
5	1999
6	2000
7	2001
8	2002
9	2003
10	2004
11	2005
12	2006
13	2007
14	2008
15	2009
16	2010
17	2011
18	2012
19	2013
20	2014
21	2015
22	2016
23	2017
24	2018

Open in new window


So should return '22' for 2016 and so on
0
 
LVL 51

Accepted Solution

by:
Vitor Montalvão earned 500 total points
ID: 41861891
Yes, it can be made but you need to provide me the lookup table structure.
Or, since it looks like a mathematic formula (1995-1994=1; 2016-1994=22; 2018-1994=24; ...) you can add immediately the formula to the query:
ALTER PROCEDURE  [dbo].[M_SelectMaxVal_FrmCol]
(
     @tableName varchar(100) = null,
	 @ColumnName1 varchar(100) = null	
)
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from interfering with SELECT statements.
	SET NOCOUNT ON;  	
	DECLARE @SQL VARCHAR(1000) = null
	
	SET @SQL = 'SELECT CASE 
						WHEN CAST(GETDATE() AS DATE) = DATEADD(yy, DATEDIFF(yy, 0, GETDATE()), 0) THEN 1 
						ELSE MAX ([' +  @ColumnName1 + ']'  + ' +1 ) END Output'  
			+ ' FROM  [dbo].[' + @tableName + ']'

	IF EXISTS(SELECT 1
			FROM sys.tables t
				INNER JOIN sys.columns c ON t.object_id=c.object_id
			WHERE t.name = @tableName AND c.name = 'year_ref')
		-----Add this only if year_ref column is present in the table 
		SET @SQL = @Sql  +' WHERE year_ref = YEAR(GETDATE())-1994'
			  
	EXECUTE(@SQL)
	
END

Open in new window

0
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!

 

Author Comment

by:RIAS
ID: 41861895
Cheers,

This is the query to create the table
ALTER PROCEDURE  [dbo].[SP_TblCreateYEAR_LOOKUP1]
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from
	-- interfering with SELECT statements.
	SET NOCOUNT ON;  

    --Script to create a look up table to dispaly years
	CREATE TABLE YEAR_LOOKUP
		(Year_Ref smallint NOT NULL,
		 Year_no smallint NOT NULL,
		 PRIMARY KEY (Year_Ref))
   
END

Open in new window


Query2 to add values


ALTER PROCEDURE  [dbo].[SP_TblCreateYEAR_LOOKUP1.1]
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from
	-- interfering with SELECT statements.
	SET NOCOUNT ON;  

  
    --Script to insert data in table  Year_Lookup 
		
	INSERT INTO YEAR_LOOKUP (Year_Ref, Year_no) VALUES
				 (0,0)
				,(1,1995),(2,1996),(3,1997),(4,1998)
				,(5,1999), (6,2000),(7,2001 ),(8,2002),(9,2003)
				,(10,2004),(11,2005),(12,2006),(13,2007),(14,2008)
				,(15,2009),(16,2010),(17,2011),(18,2012),(19,2013)
				,(20,2014),(21,2015),(22,2016),(23,2017),(24,2018)
END

Open in new window

0
 

Author Comment

by:RIAS
ID: 41861903
Vitor, Cheers!
It worked like charm!
0
 
LVL 51

Expert Comment

by:Vitor Montalvão
ID: 41861904
So you won't need to use the lookup table in the SP?
1

Featured Post

Use Case: Protecting a Hybrid Cloud Infrastructure

Microsoft Azure is rapidly becoming the norm in dynamic IT environments. This document describes the challenges that organizations face when protecting data in a hybrid cloud IT environment and presents a use case to demonstrate how Acronis Backup protects all data.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

In this article we will get to know that how can we recover deleted data if it happens accidently. We really can recover deleted rows if we know the time when data is deleted by using the transaction log.
The Delta outage: 650 cancelled flights, more than 1200 delayed flights, thousands of frustrated customers, tens of millions of dollars in damages – plus untold reputational damage to one of the world’s most trusted airlines. All due to a catastroph…
This video shows, step by step, how to configure Oracle Heterogeneous Services via the Generic Gateway Agent in order to make a connection from an Oracle session and access a remote SQL Server database table.
Viewers will learn how the fundamental information of how to create a table.

628 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question