Check If Stored Procedure Exists, Else Drop It and Recreate – SQL Server

We tend to forget the most basic syntax of all and one of them is to check if a stored procedure exists and then drop it. Here’s the syntax for your reference:

IF EXISTS
(SELECT * FROM dbo.sysobjects
WHERE id = object_id(N'[dbo].[YourStoredProcName]')
and OBJECTPROPERTY(id, N'IsProcedure') = 1)
DROP PROCEDURE [dbo].[YourStoredProcName]
GO
CREATE PROCEDURE dbo.YourStoredProcName
AS
-- Logic Comes Here
GO

Update: A BETTER solution suggested by Madhivanan

IF OBJECTPROPERTY(object_id('dbo.YourStoredProcName'), N'IsProcedure') = 1
DROP PROCEDURE [dbo].[YourStoredProcName]
GO
CREATE PROCEDURE dbo.YourStoredProcName
AS-- Logic Comes Here
GO

The syntax shown above will drop a stored procedure if it exists and recreate it.

at

7 comments:

  1. MadhivananJuly 9, 2010 at 4:12 AM

    Suprotim,

    As object_id looks for entry at sysobjects, you dont need to query it against sysobjects. You can simply write

    if OBJECTPROPERTY(object_id('dbo.YourStoredProcName'), N'IsProcedure') = 1
    DROP PROCEDURE [dbo].[YourStoredProcName]
    GO
    CREATE PROCEDURE dbo.YourStoredProcName
    AS-- Logic Comes Here
    GO

    ReplyDelete
  2. Suprotim AgarwalJuly 9, 2010 at 4:16 AM

    Yes indeed! The post has been updated. Thanks!

    ReplyDelete
  3. UnknownJuly 9, 2010 at 11:44 PM

    Suprotim,

    Following code will also work the same way..

    if object_id('dbo.YourStoredProcName','p') is not null
    DROP PROCEDURE [dbo].[YourStoredProcName]
    GO
    CREATE PROCEDURE dbo.YourStoredProcName
    AS-- Logic Comes Here
    GO

    ReplyDelete
  4. Brad SchulzJuly 12, 2010 at 11:53 AM

    The code that Chintak posted is the easiest, in my opinion.

    However, when you DROP a procedure and then re-CREATE it, you lose any permissions that may have been created against it.

    What many people do instead is to first create a dummy procedure if it doesn't exist, and then use ALTER PROCEDURE (since ALTER PROCEDURE keeps all permissions in place).

    In other words...

    IF OBJECT_ID('MyProcedure','P') IS NULL
    EXEC ('CREATE PROC MyProcedure AS SELECT 1')
    GO
    ALTER PROC MyProcedure AS ...

    --Brad

    ReplyDelete
  5. Suprotim AgarwalJuly 12, 2010 at 8:47 PM

    Chintak: Thanks for the alternate code.

    Brad: Great feedback! I really like the ALTER PROCEDURE trick!

    ReplyDelete
  6. AnonymousNovember 6, 2017 at 1:58 PM

    I always wonder WHY you check if it's a procedure.

    Assume "schema.name" is a non-procedure object in the database. Say, a view.

    If you run "if object_id(N'schema.name') is not null drop procedure schema.name", it will fail because schema.name is a view, not a procedure.

    But if you run "if object_is(N'schema.name','p') is not null drop procedure schema.name" it won't try to drop the view, but your CREATE PROCEDURE schema.name will fail.

    Either way, it fails. You go out of your way to check if schema.name is a procedure, for no gain. If it exists and it is a procedure, "if object_id(N'schema.name') is not null drop procedure" will work.

    ReplyDelete
  7. AnonymousNovember 6, 2017 at 2:01 PM

    It is pointless to check if the object is a procedure or not.

    Assume schema.name is a view, not a procedure.

    "if object_id(N'schema.name','p') is not null drop procedure schema.name" will not try to drop anything, but your CREATE PROCEDURE schema.view will fail because you can't have TWO objects schema.name.

    If you use just "if object_id(N'schema.view') is not null drop procedure schema.view" will fail when you try to "drop procedure" on a view.

    They both fail if schema.name is a non-procedure, they both succeed if schema.name IS a procedure. There is zero benefit in checking of schema.name is a procedure or not.

    ReplyDelete