他にもif else else sql

Question

ELSEが自分のコードでなぜハイライテリアであるのか知っていますか？最後のELSE。私はプロシージャを作成しようとしていますが、if節に問題があります

誰か助けてもらえますか？私はidの最大値を持つ解を見つけ、それに1を加えることは良いことではないが、それは一時的なものであることを知っている。後、私は唯一の

CREATE PROCEDURE dodajAuto 
    (@marka VARCHAR, 
    @model VARCHAR, 
    @kolor VARCHAR, 
    @moc INT, 
    @dataProdukcji DATETIME, 
    @cena DECIMAL, 
    @typ VARCHAR, 
    @przebieg INT) 
AS 
BEGIN 
    DECLARE @tmpMarka VARCHAR = (SELECT MARKA FROM MARKA WHERE MARKA=MARKA) 
    DECLARE @tmpModel VARCHAR = (SELECT MODEL FROM MODEL WHERE MODEL=MODEL) 

    DECLARE @tmpIdModelu INT = (SELECT IDMODELU FROM MODEL WHERE MODEL=MODEL) 
    DECLARE @tmpIdMarki INT = (SELECT IDMARKI FROM MARKA WHERE MARKA=MARKA) 

    DECLARE @tmpLastIdSamochodu INT = (SELECT MAX(IDSAMOCHODU) FROM SAMOCHOD) 
    DECLARE @tmpLastIdModelu INT = (SELECT MAX(IDMODELU) FROM MODEL) 
    DECLARE @tmpLastIdMarki INT = (SELECT MAX(IDMARKI) FROM MARKA) 


    IF (@tmpMarka IS NOT NULL) 
     IF (@tmpModel IS NOT NULL) 
      INSERT INTO SAMOCHOD 
      VALUES (@tmpLastIdSamochodu+1, @tmpIdModelu, @kolor, @moc, @przebieg, @dataPRodukcji, @typ); 
     ELSE 
      INSERT INTO MODEL 
      VALUES (@tmpLastIdModelu+1, @tmpIdMarki, @model, @cena) 

    SET @tmpIdModelu = (SELECT IDMODELU FROM MODEL WHERE MODEL = MODEL) 

    INSERT INTO SAMOCHOD 
    VALUES (@tmpLastIdSamochodu+1, @tmpIdModelu, @kolor, @moc, @przebieg, @dataProdukcji, @typ); 

    ELSE --THIS IS WHERE SYNTAX IS HIGLIHTED, 

    INSERT INTO MARKA VALUES (@tmpLastIdMarki+1, @marka) 

    SET @tmpIdMarki = (SELECT IDMARKI FROM MARKA WHERE MARKA = MARKA) 

    INSERT INTO MODEL 
    VALUES (@tmpLastIdModelu+1, @tmpIdMarki, @model, @cena) 

    SET @tmpIdModelu = (SELECT IDMODELU FROM MODEL WHERE MODEL = MODEL) 

    INSERT INTO SAMOCHOD 
    VALUES (@tmpLastIdSamochodu+1, @tmpIdModelu, @kolor, @moc, @przebieg, @dataProdukcji, @typ); 

Answer 1

ifとelseより良い方法を実装し、それらの後に単一のステートメントに適用しようとします。グループ化を実行する複数ステップの操作がある場合は、begin;とend;でグループ化します。

常にvarchar()とnvarchar()データタイプのサイズ、及びdecimal()とnumeric()データ型の精度とスケールを宣言する。

Bad habits to kick : declaring VARCHAR without (length) - Aaron Bertrand

create procedure dodajAuto(
    @marka varchar(30) 
    , @model varchar(30) 
    , @kolor varchar(30) 
    , @moc int 
    , @dataProdukcji datetime 
    , @cena decimal(18,0) 
    , @typ varchar(30) 
    , @przebieg int) 
as 
    begin; 
    set nocount, xact_abort on; 

    declare @tmpMarka varchar(30) = (select marka from marka where marka=@marka); 
    declare @tmpModel varchar(30) = (select model from model where model=@model); 

    declare @tmpIdModelu int = (select idmodelu from model where model=@model); 
    declare @tmpIdMarki int = (select idmarki from marka where marka=@marka); 

    declare @tmpLastIdSamochodu int = (select max(idsamochodu) from samochod); 
    declare @tmpLastIdModelu int = (select max(idmodelu) from model); 
    declare @tmpLastIdMarki int = (select max(idmarki) from marka); 

    if (@tmpMarka is not null) 
    if (@tmpModel is not null) 

     insert into samochod values (@tmpLastIdSamochodu+1, @tmpIdModelu, @kolor, @moc, @przebieg, @dataprodukcji, @typ); 
    else 
     begin; 
     insert into model values (@tmpLastIdModelu+1, @tmpIdMarki, @model, @cena); 
     set @tmpIdModelu = (select idmodelu from model where model=@model); 
     insert into samochod values (@tmpLastIdSamochodu+1, @tmpIdModelu, @kolor, @moc, @przebieg, @dataProdukcji, @typ); 
     end; 
    else 
    begin; 
     insert into marka values (@tmpLastIdMarki+1, @marka); 
     set @tmpIdMarki = (select idmarki from marka where marka=@marka); 
     insert into model values (@tmpLastIdModelu+1, @tmpIdMarki, @model, @cena); 
     set @tmpIdModelu = (select idmodelu from model where model=@model); 
     insert into samochod values (@tmpLastIdSamochodu+1, @tmpIdModelu, @kolor, @moc, @przebieg, @dataProdukcji, @typ); 
    end; 
    end; 
go