Estoy tratando de implementar tipos de registro para que la comparación sea menos compleja. Pero me encontré con problemas al comparar registros entre sí. Espero que los registros sean iguales cuando las propiedades son iguales entre sí, pero este no es el caso en mi código.
Tengo una clase de registro llamada MyAlarm con algunas propiedades. Hereda de la interfaz IAlarm que está vacía.
public record MyAlarm : IAlarm { public int Prop1 { get; init; } public int Prop2 { get; init; } } Una clase llamada VisibleAlarm tiene IAlarm como propiedad más algunos datos adicionales.
public record VisibleAlarm { public IAlarm InternalAlarm { get; init; } public string Message { get; set; } } En una clase llamada AlarmService las VisibleAlarms se enumeran y actualizan cuando es necesario. Este servicio dispone de una Lista de alarmas visibles.
Además, en este servicio de alarmas, la lista se comprueba si ya existe una alarma en función de la propiedad IAlarm. He probado dos métodos:
public class AlarmService { private List<VisibleAlarm> _alarms = new(); // ... Other code that handles alarm stuff private VisibleAlarm GetExistingAlarm1(IAlarm alarm) { return _alarms.FirstOrDefault(a => a.InternalAlarm == alarm); } private VisibleAlarm GetExistingAlarm2(IAlarm alarm) { foreach (var existingAlarm in _alarms) { if (existingAlarm.InternalAlarm == alarm) { return existingAlarm; } } } } Al depurar el código, veo que las propiedades de registro deexistentAlarm.InternalAlarm y la alarma dada coinciden, pero aún así los métodos GetExistingAlarm devuelven ambos falsos. ¿Me estoy perdiendo algo o debo implementar un IEqualityComparer debido a la interfaz IAlarm?
Salud
En su línea de código:
return _alarms.FirstOrDefault(a => a.InternalAlarm == alarm); la a.InternalAlarm == alarm está llamando a operator== en lugar de object.Equals() para hacer la comparación.
Puede solucionar esto cambiando el código a:
return _alarms.FirstOrDefault(a => a.InternalAlarm.Equals(alarm));El siguiente programa demuestra la diferencia:
using System; static class Program { public static void Main() { MyAlarm ma1 = new MyAlarm { Prop1 = 1, Prop2 = 2 }; MyAlarm ma2 = new MyAlarm { Prop1 = 1, Prop2 = 2 }; VisibleAlarm va1 = new VisibleAlarm { InternalAlarm = ma1, Message = "message" }; VisibleAlarm va2 = new VisibleAlarm { InternalAlarm = ma2, Message = "message" }; Console.WriteLine(ma1 == ma2); // True Console.WriteLine(va1 == va2); // True Console.WriteLine(va1.InternalAlarm == va2.InternalAlarm); // False Console.WriteLine(va1.InternalAlarm.Equals(va2.InternalAlarm)); // True } public interface IAlarm { int Prop1 { get; } int Prop2 { get; } } public record MyAlarm : IAlarm { public int Prop1 { get; init; } public int Prop2 { get; init; } } public record VisibleAlarm { public IAlarm InternalAlarm { get; init; } public string Message { get; set; } } }Producción:
True True False True La razón por la que esto no funciona para usted es porque InternalAlarm es del tipo IAlarm y no MyAlarm . Si cambia el tipo a MyAlarm , obtendrá la comparación como true en lugar de false .
Esto se debe a que IAlarm no define un operator== (nota: dicho operador sería estático).