Aunque introducir un TOP 1
, en efecto, elimina el error que recibes, eso simplemente resuelve el síntoma ignorando el verdadero problema. Estoy convencido que esa solución no es adecuada en este caso, y te va a causar problemas con tus datos.
¿Cuál es el verdadero problema?
Lo primero, es importante entender por qué recibistes el error. El error sucede porque la tabla temporal inserted
contiene más de un registro. Esto sucede cuando ejecutas una sentencia update
que afecta más de un registro a la vez en la tabla Calificacion
.
Relacionado a este punto, es importante recordar que, diferente a otras base de datos, en SQL Server, cuando ejecutas una sentencia SQL que afecta varios registros, el disparador dispara una sola vez para todos los registros. De modo que para diseñar un disparador correctamente, debes poder manejar varios registros a la vez.
Nota lo que dice la referencia MSDN siguiente al respecto: Crear desencadenadores DML para administrar varias filas de datos:
Cuando escriba el código para un desencadenador DML, tenga en cuenta que la instrucción que hace que se active el disparador puede ser una sola instrucción que afecte a varias filas de datos, en lugar de una sola fila. Este comportamiento es habitual para los desencadenadores UPDATE
y DELETE
, ya que estas instrucciones suelen afectar a varias filas.
Esa es tu falla. Tu disparador asume que solo vas a manejar un solo registro a la vez. Y el usar TOP 1
solo te sirve para esconder la realidad y crear una ilusión de que aún puedes manejar un registro a la vez. Pero, haciendo esto, estás ignorando registros que deberías manejar.
Otros problemas
Un detalle en apariencia menor, pero importante para poder diseñar un disparador multi-registros correctamente: necesitar definir una clave primaria en tu tabla Calificacion
. No veo que tengas una. Deduzco que se supone que la combinación Id_A, Id_M
es única, pero no lo has definido explícitamente.
Otro problema es que no estás usando la función AVG()
correctamente. Al hacer:
Select AVG(calif_1+calif_2+calif_3+calif_4)
... parece que estás bajo la impresión que eso te va a dar el promedio de las 4 calificaciones. Esto no es el caso en absoluto. AVG(expresión)
te devuelve el promedio de una expresión de entre varios registros. Pero como solo estás manejando un solo registro, en realidad solo estás sumando las 4 calificaciones sin hacer un promedio. Verifica el resultado y verás.
- Si la idea es de mantener al día el campo
calif_f
automáticamente, sería bueno que el trigger dispare también con las sentencias INSERT
.
Solución
Tomando en cuenta el propósito de tu trigger:
me pidieron realizar un triger que actualizara la calificación final de forma automática al actualizar alguna de las calificaciones parciales.
... yo ajustaría los puntos siguientes:
- Formalizar
Id_A, ID_M
como la clave primaria de la tabla Calificacion
(si la clave primaria debería definirse con otras columnas, no es un problema, pero requiere que se ajusta la condición en el join del trigger, ver más abajo).
- Modificar el trigger para que dispare con los
INSERT
s también
- Corregir el diseño del trigger para que maneje varios registros correctamente. Esto se logra haciendo un
join
con la tabla temporal inserted
.
- Corregir el cálculo del promedio. Para hacerlo correctamente, y manejando los
NULL
de la forma estándar, la sentencia se complica un poco con expresiones CASE
, pero esa es la forma correcta de hacerlo, no usando la función AVG()
.
- Optimización: agregar una condición que solo pone al día el promedio si al menos uno de los campos
calif_N
fue modificado. Esto también sirve de protección para evitar disparos recursivos si el servidor está configurado para permitirlos.
El trigger corregido sería:
create trigger tg_promAlu
On Calificacion
After insert, Update
As
Begin
set nocount on;
if update(calif_1)
or update(calif_2)
or update(calif_3)
or update(calif_4)
update c
set c.calif_f = case when i.calif_1 is not null
or i.calif_2 is not null
or i.calif_3 is not null
or i.calif_4 is not null
then
(coalesce(i.calif_1, 0)
+ coalesce(i.calif_2, 0)
+ coalesce(i.calif_3, 0)
+ coalesce(i.calif_4, 0))
/
(case when i.calif_1 is null then 0 else 1 end
+ case when i.calif_2 is null then 0 else 1 end
+ case when i.calif_3 is null then 0 else 1 end
+ case when i.calif_4 is null then 0 else 1 end)
end
from inserted i
join Calificacion c
on c.Id_A = i.Id_A
and c.ID_M = i.ID_M;
End;
Go
Si quieres ver el trigger en acción: Demo