Mejora Procedimiento Almacenado

[Ronald_Leon_Madrid]

Estimados Colegas:

Que tal un gusto en saludarlos, tengo el siguiente esquema del un procedimiento almacenado:

ALTER Procedure [dbo].[usp_get_Cliente]
(
@Periodo char(6),
@Unidad_Negocio char(2),
@Vendedor char(2)
)
As
Begin

Set Nocount On

If (@Vendedor = '') Select @Vendedor = Null

Select Distinct
Cliente,
Cliente_Nombre
From
Datamartbo.montana.v_estimados_generico
Where
convert(char(6),Periodo_Base,112) = @Periodo And
Unidad_Negocio = @Unidad_Negocio And
Vendedor = Coalesce(@Vendedor,Vendedor)
Order By
Cliente_Nombre

Set Nocount Off

End

Por favor necesito comentarios como posibles mejores, tambien criticas constructivas como destructivas.

Saludos cordiales,
Ronald León Madrid.

[flaviovich]

Si Periodo_Base es de tipo datetime, @Periodo tambien deberia serlo.

Código SQL:

ALTER PROCEDURE [dbo].[usp_get_Cliente]
(
@Periodo CHAR(6),
@Unidad_Negocio CHAR(2),
@Vendedor CHAR(2) = NULL
)
AS
BEGIN
 
SET Nocount ON
 
SELECT DISTINCT
Cliente,
Cliente_Nombre
FROM
Datamartbo.montana.v_estimados_generico
WHERE
CONVERT(CHAR(6),Periodo_Base,112) = @Periodo AND
Unidad_Negocio = @Unidad_Negocio AND
(Vendedor = @Vendedor OR @Vendedor IS NULL)
ORDER BY
Cliente_Nombre
 
SET Nocount Off
 
END
 

[Ronald_Leon_Madrid]

Muchas gracias flavio por tu ayuda, me servirá bastante para mejorar mis procedimientos almacenados.

Saludos cordiales,
Ronald León Madrid.