Skip to main content
se añadieron 128 caracteres en el cuerpo
Origen Enlace
sstan
  • 16.9k
  • 2
  • 24
  • 48

... 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. VerificaPero que veas el resultado y verásproblema, aquí te dejo un enlace para que veas el resultado de ejecutar select avg(100 + 100 + 100 + 100).

... 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.

... 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. Pero que veas el problema, aquí te dejo un enlace para que veas el resultado de ejecutar select avg(100 + 100 + 100 + 100).

se añadió 1 carácter en el cuerpo
Origen Enlace
sstan
  • 16.9k
  • 2
  • 24
  • 48

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 afectados. De modo que para diseñar un disparador correctamente, debes poder manejar varios registros a la vez.

me pidieron realizar un trigertrigger que actualizara la calificación final de forma automática al actualizar alguna de las calificaciones parciales.

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.

me pidieron realizar un triger que actualizara la calificación final de forma automática al actualizar alguna de las calificaciones parciales.

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 afectados. De modo que para diseñar un disparador correctamente, debes poder manejar varios registros a la vez.

me pidieron realizar un trigger que actualizara la calificación final de forma automática al actualizar alguna de las calificaciones parciales.

Origen Enlace
sstan
  • 16.9k
  • 2
  • 24
  • 48

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

  1. 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.

  2. 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.

  1. 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:

  1. 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).
  2. Modificar el trigger para que dispare con los INSERTs también
  3. Corregir el diseño del trigger para que maneje varios registros correctamente. Esto se logra haciendo un join con la tabla temporal inserted.
  4. 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().
  5. 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