Dodaj tylko unikatowy element do listy

Question

Dodaję urządzenia zdalne do listy, ponieważ ogłaszają się w sieci. Chcę tylko dodać urządzenie do listy, jeśli nie zostało wcześniej dodane.

Ogłoszenia nadchodzą przez odbiornik asynchroniczny gniazda, więc kod dodawania urządzenia może być uruchamiany na wielu wątkach. Nie jestem pewien, co robię źle, ale bez względu na to, co próbuję, kończę z duplikacjami. Oto, co aktualnie mam .....

lock (_remoteDevicesLock) 
{ 
    RemoteDevice rDevice = (from d in _remoteDevices 
          where d.UUID.Trim().Equals(notifyMessage.UUID.Trim(), StringComparison.OrdinalIgnoreCase) 
          select d).FirstOrDefault(); 
    if (rDevice != null) 
    { 
     //Update Device..... 
    } 
    else 
    { 
     //Create A New Remote Device 
     rDevice = new RemoteDevice(notifyMessage.UUID); 
     _remoteDevices.Add(rDevice); 
    } 
} 

Answer 1

Jeśli wymagania są nie mieć duplikaty, należy używać do HashSet.

HashSet.Add powróci fałszywy gdy już istnieje element (jeśli jest, że nawet sprawy do Ciebie).

Można użyć konstruktora że @pstrjds linki poniżej (lub here), aby zdefiniować operator równości lub trzeba zaimplementować metody równości w RemoteDevice (GetHashCode & Equals).

Answer 2

Podobnie jak przyjęta odpowiedź mówi, że HashSet nie ma zamówienia. Jeśli zamówienie jest ważne, możesz nadal korzystać z Listy i sprawdzać, czy zawiera ona przedmiot przed dodaniem.

if (_remoteDevices.Contains(rDevice)) 
    _remoteDevices.Add(rDevice); 

List.Contains sceniczne() na niestandardowej klasy/obiektu wymaga wykonania IEquatable<T> na niestandardowej klasy lub przesłanianie Equals. Dobrym pomysłem jest również wdrożenie GetHashCode w klasie. Jest za dokumentacji na https://msdn.microsoft.com/en-us/library/ms224763.aspx

public class RemoteDevice: IEquatable<RemoteDevice> 
{ 
    private readonly int id; 
    public RemoteDevice(int uuid) 
    { 
     id = id 
    } 
    public int GetId 
    { 
     get { return id; } 
    } 

    // ... 

    public bool Equals(RemoteDevice other) 
    { 
     if (this.GetId == other.GetId) 
      return true; 
     else 
      return false; 
    } 
    public override int GetHashCode() 
    { 
     return id; 
    } 
} 

Answer 3

//HashSet allows only the unique values to the list 
HashSet<int> uniqueList = new HashSet<int>(); 

var a = uniqueList.Add(1); 
var b = uniqueList.Add(2); 
var c = uniqueList.Add(3); 
var d = uniqueList.Add(2); // should not be added to the list but will not crash the app 

//Dictionary allows only the unique Keys to the list, Values can be repeated 
Dictionary<int, string> dict = new Dictionary<int, string>(); 

dict.Add(1,"Happy"); 
dict.Add(2, "Smile"); 
dict.Add(3, "Happy"); 
dict.Add(2, "Sad"); // should be failed // Run time error "An item with the same key has already been added." App will crash 

//Dictionary allows only the unique Keys to the list, Values can be repeated 
Dictionary<string, int> dictRev = new Dictionary<string, int>(); 

dictRev.Add("Happy", 1); 
dictRev.Add("Smile", 2); 
dictRev.Add("Happy", 3); // should be failed // Run time error "An item with the same key has already been added." App will crash 
dictRev.Add("Sad", 2);