Controllare se il record esiste “update”, se sì, se non “inserto”
-
02-10-2019 - |
Domanda
voglio verificare tavolo PREMIUM_SERVICE_USER
se tutti i record esiste per
strClientID
aggiornamento timeValid
per 30 se nessun record per l'inserimento strClientID
al tavolo premium_service_user
.
Che cosa sto facendo di sbagliato?
Aumenta timeValid
costo di +30 giorni ma inserisce un'altra riga troppo.
SELECT @pre_var = count(*)
FROM PREMIUM_SERVICE_USER
WHERE strClientID = @strClientID
/* bronze premium - 200 cash */
IF @Premium = 1
BEGIN
INSERT INTO PREMIUM_SERVICE_USER
(strClientID, timeReg, timeValid, bCurrent, durum)
VALUES
(@strClientID,getdate(),getdate() + 30,'1','1')
UPDATE TB_USER
SET cash = cash+200
WHERE strAccountID = @strClientID
END
IF @Premium = 1 AND @pre_var = 1
BEGIN
UPDATE PREMIUM_SERVICE_USER
SET timevalid = timevalid+30 where strClientID = @strClientID
UPDATE PREMIUM_SERVICE_USER
SET bCurrent = 1 where strClientID = @strClientID
UPDATE TB_USER
SET cash = cash+200 WHERE strAccountID = @strClientID
END
Soluzione
Il tuo problema è stato eseguito il primo if
senza riguardo al valore della @pre_var
.
Questo è un modo leggermente diverso di farlo, che sarà un po 'più efficiente se PREMIUM_SERVICE_USER
è di grandi dimensioni.
if @Premium = 1
begin
if exists(Select 1 From PREMIUM_SERVICE_USER Where strClientID = @strClientID)
BEGIN
update PREMIUM_SERVICE_USER set timevalid = timevalid+30 where strClientID = @strClientID
update PREMIUM_SERVICE_USER set bCurrent = 1 where strClientID = @strClientID
UPDATE TB_USER SET cash = cash+200 WHERE strAccountID = @strClientID
END
ELSE
BEGIN
INSERT INTO PREMIUM_SERVICE_USER (strClientID, timeReg, timeValid, bCurrent, durum) VALUES (@strClientID,getdate(),getdate() + 30,'1','1')
UPDATE TB_USER SET cash = cash+200 WHERE strAccountID = @strClientID
END
end
Altri suggerimenti
CREATE PROCEDURE sp_UpdateProcessed
AS
BEGIN
DECLARE @Processed_Status NVARCHAR(256)
SET @Processed_Status = 'ACTIVE'
IF(@Processed_Status <> 'Processed')
BEGIN
SET @Processed_Status = 'Active'
UPDATE ST_JnlMediumMoveNew
SET ST_JnlMediumMoveNew.Process_Status = @Processed_Status
END
END
Si sta contando le righe, ma non lo si utilizza nel vostro processo decisionale. Qui è una struttura decisionale che può essere utile
Select @pre_var = count(*) From PREMIUM_SERVICE_USER Where strClientID = @strClientID
IF @pre_var = 0
BEGIN
/* Run Insert Code Here */
END
ELSE
BEGIN
/* Run Update Code Here */
END
IF @Premium = 1
BEGIN
/* Run Premier Members Update Code Here */
END
ELSE
BEGIN
/* Run Non-Premier Members Update Code Here */
END
O questo ..
IF @pre_var = 0
BEGIN
/* Run Insert Code Here */
END
ELSE
BEGIN
IF @Premium = 1
BEGIN
/* Run Premier Members Update Code Here */
END
ELSE
BEGIN
/* Run Non-Premier Members Update Code Here */
END
END
Sembra che si sta controllando la variabile sbagliato nel primo if. Se @Premium = 1 allora si vedrà questo comportamento.
/* bronze premium - 200 cash */
IF @pre_var = 0
BEGIN
INSERT INTO PREMIUM_SERVICE_USER (strClientID, timeReg, timeValid, bCurrent, durum) VALUES (@strClientID,getdate(),getdate() + 30,'1','1')
UPDATE TB_USER SET cash = cash+200 WHERE strAccountID = @strClientID
END
IF @Premium = 1 AND @pre_var = 1
BEGIN
update PREMIUM_SERVICE_USER set timevalid = timevalid+30 where strClientID = @strClientID
update PREMIUM_SERVICE_USER set bCurrent = 1 where strClientID = @strClientID
UPDATE TB_USER SET cash = cash+200 WHERE strAccountID = @strClientID
END