Почему не выполняется процедура, не заходит в if?

Question

[Bobik4556Ttft]

CREATE PROCEDURE [Изменение платы за тариф](@Name_tarif NVARCHAR(50), @new_price INT, @now_price int OUTPUT)
AS
BEGIN
	IF(EXISTS(SELECT * FROM [Тарифы всех провайдеров] WHERE [Название тарифа] = @Name_tarif) AND @new_price > 0) BEGIN
		UPDATE [Тарифы всех провайдеров] SET [Плата] = @new_price WHERE [Название тарифа] = @Name_tarif ;
		SET @now_price = (SELECT [Плата] FROM [Тарифы всех провайдеров] WHERE [Название тарифа] = @Name_tarif);
		RETURN 1;
	END 
	ELSE RETURN 0;
END;
DECLARE @RET INT;
DECLARE @NewPrice INT;
EXECUTE @RET = [Изменение платы за тариф] @new_price = 600, @Name_tarif = N'Базовый', @now_price = @NewPrice OUTPUT;

IF(@RET = 1)BEGIN
	print N'Процедура прошла успешно. Новая плата за тариф: ' + CAST(@NewPrice as NVARCHAR);
END ELSE IF(@RET = 0) BEGIN
	print N'Процедура не прошла успешно';
END;

Выводит 'Процедура не прошла успешно'. Возвращает 0. Почему не возвращает 1?

Comments

[versetty777]

Скорее всего вы используете неправильное имя параметра при вызове процедуры. В вашем коде вы используете параметр @new_price для задания новой цены, но в определении процедуры он называется @new_price. И ещё, проверьте имя таблицы, которое вы используете в запросе, что оно существует в базе данных. Поправьте имя параметра в вызове процедуры, удостоверьтесь что имена таблиц и полей правильные, и повторите попытку запуска.

[Константин Цветков]

versetty777,

SELECT IIF( '@new_price' = '@new_price', 'OK', 'NOT OK' )

OK

Answer 1

[Константин Цветков]

CREATE OR ALTER PROCEDURE [Изменение платы за тариф](@Name_tarif NVARCHAR(50), @new_price INT)
AS
BEGIN
  UPDATE [Тарифы всех провайдеров] 
    SET [Плата] = @new_price 
    WHERE [Название тарифа] = @Name_tarif AND @new_price > 0
  RETURN @@ROWCOUNT
END
GO

DECLARE @RET INT;
DECLARE @new_price INT = 600
EXECUTE @RET = [Изменение платы за тариф] N'Базовый', @new_price

IF (@RET > 0 ) PRINT N'Процедура прошла успешно. Новая плата за тариф: ' 
                        + CAST( @new_price AS NVARCHAR )
ELSE PRINT N'Процедура не прошла успешно'

Answer 2

[Bobik4556Ttft]

А всё разобрался.
Там нужно использовать не exitst, а агрегатную функцию count(*)

IF(EXISTS(SELECT * FROM [Тарифы всех провайдеров] WHERE [Название тарифа] = @Name_tarif) AND @new_price > 0)

IF((SELECT COUNT(*) FROM [Тарифы всех провайдеров] WHERE [Название тарифа] = @Name_tarif) > 0 AND @new_price > 0)

Comments

[Bobik4556Ttft]

Exists для подзапросов.

[Константин Цветков]

Bobik4556Ttft, А IF EXISTS нет.

[Bobik4556Ttft]

Константин Цветков, Но он с этим EXITST в IF FALSE выдает.

[Константин Цветков]

Bobik4556Ttft, EXITST не нужен, ведь это условие есть в UPDATE. Вообще IF не нужен. Все условия прописываются в UPDATE. Количество изменений покажет @@ROWCOUNT.