SQL Update in While Loop

Question

Я пытаюсь отправить число и идентификаторы и обновить строки в цикле while.

Например, если @Number = 1

и

MAX(Number) 
FROM QuestionnaireQuestions 
WHERE Questionnaire_ID = @Questionnaire_ID = 4 

Результаты значения 1, 1, 1, когда я ожидал бы 3,2,1.

CREATE PROCEDURE [dbo].[DeleteQuestion] 
    (@QuestionnaireQuestions_ID BIGINT, 
    @Questionnaire_ID   BIGINT, 
    @Number     SMALLINT) 
AS 
    DECLARE @i AS SMALLINT 

    SELECT @i = MAX(Number) 
    FROM QuestionnaireQuestions 
    WHERE Questionnaire_ID = @Questionnaire_ID 

    WHILE (@i > @Number) 
    BEGIN 
      UPDATE QuestionnaireQuestions 
      SET Number = (@i - 1) 
      WHERE Number = @i 
       AND Questionnaire_ID = @Questionnaire_ID 

      SET @i = @i - 1 
    END 

    DELETE QuestionnaireQuestions 
    WHERE QuestionnaireQuestions_ID = @QuestionnaireQuestions_ID 

Answer 1

Если номер вопроса не больше, чем заказ в вопроснике, почему бы вам не просто оставить номер, первоначально назначенный на вопрос, а затем перенумеровать их, когда они будут показаны или иным образом обработаны.

Таким образом, изменение данных оказывает локальное воздействие на затронутую строку.

Answer 2

Ниже то, что я думаю, что происходит, если предположить, идентификатор не является уникальным ID ... Скажем, у вас есть строки со значением 4,3,2,1.

При первом запуске вы выбираете, вы получаете строку с 4. Когда вы ее обновляете, таблица будет иметь строки: 3,3,2,1.

Когда вы выбираете снова, вы получаете строку с 3, а когда вы обновляете, вы получаете: 2,3,2,1. Повторяя этот (R означает результат выбора, U означает результат обновления):

Р: 3

U: 2,2,2,1

Р: 2

U: 1,2 , 2,1

Р: 2

U: 1,1,2,1

Р: 1

U: 1,1,1,1

Один из способов решения этой проблемы состоит в том, чтобы отделить ваши критерии поиска, чтобы они не зависели от обновленной колонки.

Answer 3

Я бы переделал его, чтобы получить только один идентификатор. Таким образом, вы никогда не сможете передать его значениям, которые не принадлежат друг другу.

CREATE PROCEDURE [dbo].[DeleteQuestion] (@QuestionnaireQuestions_ID BIGINT) 
AS 
    DECLARE @Questionnaire_ID   BIGINT, 
      @Number     SMALLINT 

    SELECT @Questionnaire_ID = Questionnaire_ID, 
      @Number = Number 
    FROM QuestionnaireQuestions 
    WHERE QuestionnaireQuestions_ID = @QuestionnaireQuestions_ID  

    DELETE QuestionnaireQuestions 
    WHERE QuestionnaireQuestions_ID = @QuestionnaireQuestions_ID 

    UPDATE QuestionnaireQuestions 
    SET Number = Number - 1 
    WHERE Questionnaire_ID = @Questionnaire_ID 
      AND Number > @Number 

END