Ошибка хранимой процедуры: ошибка преобразования типа данных nvarchar в uniqueidentifier

Итак, если вы просто вызываете свою хранимую процедуру с помощью getdate и newid, почему бы вам просто не добавить их по умолчанию в свою таблицу?

Таблица

  CREATE TABLE [dbo].[Report](
    [ID] [uniqueidentifier] NOT NULL CONSTRAINT [DF_Report_ID]  DEFAULT (newid()),
    [STAFF] [varchar](1000) NOT NULL,
    [EMAIL] [varchar](1000) NOT NULL,
    [LASTCHANGE] [datetime] NOT NULL CONSTRAINT [DF_Report_LASTCHANGE]  DEFAULT (getdate()),
 CONSTRAINT [PK__Report__3214EC27D2D8BF72] PRIMARY KEY CLUSTERED 
(
    [ID] ASC
)WITH (PAD_INDEX = OFF, STATISTICS_NORECOMPUTE = OFF, IGNORE_DUP_KEY = OFF, ALLOW_ROW_LOCKS = ON, ALLOW_PAGE_LOCKS = ON) ON [PRIMARY]
) ON [PRIMARY]

Процедура

create PROCEDURE spInsertOrUpdate(

@STAFF VARCHAR(1000),
@EMAIL VARCHAR(1000)

) AS
BEGIN
INSERT INTO Report(STAFF, EMAIL)
    VALUES(@STAFF, @EMAIL)
END

Выполнить инструкцию

EXEC spInsertOrUpdate  'Evlyn Dawson', '[email protected]'

Изменить. Также обратите внимание, что ваш последний измененный столбец имеет тип DATE, однако, если вам нужна дата с отметкой времени, вы должны использовать datetime.

Это сообщение об ошибке похоже на погоню за диким гусем, проблема в том, что и NEWID(), и GETDATE() являются функциями, поэтому требуются круглые скобки. К сожалению, вы не можете передать функцию в качестве параметра хранимой процедуре, поэтому вам сначала нужно присвоить значения переменной:

DECLARE @ID UNIQUEIDENTIFIER = NEWID(),
        @Date DATE = GETDATE();

EXEC #spInsertOrUpdate @ID, 'Evlyn Dawson', '[email protected]', @Date;

Кроме того, UNIQUEIDENTIFIER столбец очень плохой выбор для ключа кластеризации

Я бы начал с правильного вызова функций:

EXEC spInsertOrUpdate NEWID(), 'Evlyn Dawson', '[email protected]', GETDATE();

NEWID() и GETDATE() - это функции, поэтому вам нужны скобки после них.

Однако я не думаю, что отсутствие круглых скобок может вызвать именно эту ошибку. Вам нужно сначала установить переменные, а затем использовать их для exec.

РЕДАКТИРОВАТЬ:

Лучше всего установить идентификаторы и даты автоматически:

CREATE TABLE Report (
    ID UNIQUEIDENTIFIER PRIMARY KEY NOT NULL DEFAULT NEWID(),
    STAFF VARCHAR(1000) NOT NULL,
    EMAIL VARCHAR(1000) NOT NULL,
    LASTCHANGE DATE NOT NULL DEFAULT GETDATE()
);

CREATE PROCEDURE spInsertOrUpdate (
    @STAFF VARCHAR(1000),
    @EMAIL VARCHAR(1000)
) AS
BEGIN
    INSERT INTO Report(STAFF, EMAIL)
        VALUES (@STAFF, @EMAIL)
END;

EXEC spInsertOrUpdate 'Evlyn Dawson', '[email protected]';

Я бы также отговорил вас от использования уникальных идентификаторов в качестве первичных ключей в таблице. Они довольно неэффективны, так как могут привести к фрагментации страницы. Вместо этого используйте столбец identity.

Спасибо за вашу помощь. Наконец-то я нашел правильный способ сделать это в рамках SP, и теперь я получил правильное представление о SP. Вот как я решил проблему

CREATE PROCEDURE spInsertOrUpdate(@STAFF VARCHAR(1000),@EMAIL VARCHAR(1000),@CARS VARCHAR(1000))

AS
BEGIN
INSERT INTO Report(ID,STAFF,EMAIL,CARS,LASTCHANGE)
VALUES(NEWID(),@STAFF,@EMAIL,@CARS,GETDATE())
END

EXEC spInsertOrUpdate 'Evlyn Dawson','[email protected]','Ferrari'

Обратите внимание, что у меня также есть столбец АВТОМОБИЛИ