1

Tenho um modelo de estrutura, gostaria de saber se ela está adequada a um bom padrão.

USE [teste]
GO

SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
-- =============================================
-- Author:      <Author,,Name>
-- Create date: <Create Date,,>
-- Description: <Description,,>
-- =============================================
ALTER PROCEDURE [dbo].[sprEstruturaPadrao] 
   --declaração de variáveis
   @idusuario int

AS
BEGIN
   --abre a transação para update e insert
   --BEGIN TRAN sprEstruturaPadrao;

   if(@idusuario > 0)
   begin
       --select
       select * from TB_USUARIO where idusuario = @idusuario

      if(@@ERROR <> 0)
         PRINT N'Error = ' + CAST(@@ERROR AS NVARCHAR(20));
         -- finaliza transação para update e insert
         --  ROLLBACK TRAN sprEstruturaPadrao 
      else
           PRINT 'tudo certo';
         -- finaliza transação para update e insert
         --  COMMIT TRAN  sprEstruturaPadrao


   end


END
2
  • Vai dar erro toda vez que executar numa primeira vez quando ainda não existir tal procedure. Poderia iniciar com um if isnull(object_id ... drop e ao invés de ser alter seria sempre um create. 4/05/2017 às 14:32
  • este modelo já tinha sido criado, ele estava sendo alterado apenas
    – Harry
    4/05/2017 às 17:59

1 Resposta 1

1

Algumas sugestões:

(1) utilize o comando

set NOCOUNT on

logo no início do procedimento, antes de qualquer outro comando.

(2) retornar informação sobre se o procedimento executou com ou sem erros através de RETURN valor. Se houve erro, a sugestão é que o valor retornado seja negativo. Por exemplo, se ocorreu erro 488 na instrução SQL, o valor retornado é -488.

(3) não utilizar SELECT *

(4) efetuar tratamento de erro utilizando TRY..CATCH

(5) Padronize como as instruções o T-SQL serão grafadas. No exemplo que postou, o "select" está em minúsculas mas o "PRINT" está em maiúsculas, o que demonstra necessidade de padronização.


Sugestão de leitura: My stored procedure "best practices" checklist, de Aaron Bertrand.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .