Falsche Syntax in der Nähe von etwas

Question

Ich schreibe eine gespeicherte Prozedur, die ich ein Jahr als Parameter in diesem übergeben. Ich habe saledatestr als varchar Spalte. Ich mag erhalten wählen, in den vier ersten Ziffern saledatestr hier meine Parameter gleich ist mein Code:

if exists (select * from dbo.sysobjects 
      where id = object_id(N'[dbo].[spSaleCustomerYear]') 
      and OBJECTPROPERTY(id, N'IsProcedure') = 1) 
    drop procedure [dbo].[spSaleCustomerYear] 
GO 

create procedure spSaleCustomerYear 
    (@year varchar) 
with Encryption 
as 
    select 
     c.CustomerCode, 
     PLE.PLEName, 
     s.SaleNo, 
     s.FinalPrice, 
     s.SaleDateStr 
    from 
     SMS.tblsales s 
    left join 
     SMS.tblCustomers c on c.CustomerID = s.CustomerRef 
    left join 
     GNR.tblPrincipalLegalEntities PLE on PLE.PrincipalLegalEntityID = c.PrincipalLegalEntityRef 
    where 
     s.CustomerRef not in (select CustomerRef 
           from SMS.tblSales s1 
           --left join dbo.tblFiscalYear FY on FY.FiscalYearID = s1.FiscalYearID 
           where (LEFT(s1.SaleDateStr, 4) = @year) 
           and SaleDocType = 4) 

Der Fehler in dieser Zeile ist:

(LEFT(s1.SaleDateStr, 4) = @year) 

Bitte helfen Sie mir Jungen.

Vielen Dank im Voraus.

Answer 1

Siehe Bad habits to kick : declaring VARCHAR without (length) - Sie sollten immer eine Länge für alle varchar Variablen und Parameter, die Sie verwenden

Sie sollten NEVER EVER einen Parameter wie folgt definieren:

create procedure spSaleCustomerYear 
    (@year varchar) 

Diese definiert einen Varchar-Parameter von genau eine Zeichenlänge das ist normalerweise nicht was Sie wollen.

Sie sollten IMMER (keine Ausnahmen!) Definieren eine explizite Länge wenn varchar mit! Dies gilt auch für das Definieren von SQL-Variablen oder das Definieren von Zieltypen in den Anweisungen CAST oder CONVERT.

Veränderung dieser Parameter auf:

(@year varchar(4)) 

und Sie sollten in Ordnung sein.

Answer 2

In der Prozedur, können Sie die Zeile

where (LEFT(s1.SaleDateStr, 4) = @year) 

mit diesem ersetzen und versuchen:

WHERE YEAR(CAST(s1.SaleDateStr AS DATETIME)) = CAST(@year AS INT)