Co Koszt parametrów odlewania

Question

Podczas pracy z MVVM i Prism znajdę się robić dużo odlewania, jak większość parametry są interfejsy

Ex

public void AddCrSubSystemsToPlant(IPlantItem plantItm, CRArticleItem crItm) 
     { 

      OSiteSubSystem itm = (OSiteSubSystem)crItm; 
      itm.PartData.Order = ((OSiteEquipment)plantItm).SubSystems.Count() + 1; 

      ((OSiteEquipment)plantItm).SubSystems.Add(itm); 

     } 

lub

public void DeletePart(IPlantItem plantItem) 
     { 
      IEnumerable<IPlantItem> itmParent = GetParentPartByObjectId(_siteDocument, plantItem); 

      if (plantItem is OSiteEquipment) 
      ((ObservableCollection<OSiteEquipment>)itmParent).Remove((OSiteEquipment)plantItem); 

      if (plantItem is OSiteSubSystem) 
       ((ObservableCollection<OSiteSubSystem>)itmParent).Remove((OSiteSubSystem)plantItem); 

      if (plantItem is OSiteComponent) 
       ((ObservableCollection<OSiteComponent>)itmParent).Remove((OSiteComponent)plantItem); 
     } 

My Pytanie brzmi, jaki jest koszt. Czy te operacje są kosztowne, czy też procesorowe, należy ich unikać.

Jakieś widoki?

Answer 1

Myślę, że ważniejsze pytanie brzmi: dlaczego robisz tak dużo castingów?

W pierwszym przykładzie:
Dlaczego jest pierwszy parametr typ IPlantItem Jeśli trzymasz rzucając go OSiteEquipment? To samo można powiedzieć o drugim parametrze.

W drugim przykładzie:
Dlaczego GetParentPArtByObjectId zwraca wartość IEnumerable<IPlantItem>? Jeśli miałby zwrócić wartość ICollection<IPlantItem>, nie musiałbyś przesyłać go pod numer ObservableCollection<T>. ObservableCollection<T> dziedziczy z Collection<T>, która implementuje zarówno ICollection<T>, jak i ICollection. Powinieneś być w stanie usunąć przedmiot z kolekcji, nawet nie znając jego typu.

Teraz kilka porad.
Nie należy wielokrotnie rzucać tego samego obiektu.
nie rób tego:

if (obj is IPlantItem) 
    ((IPlantItem)obj).DoSomething(); 

Czy zamiast tego

IPlantItem plant = obj as IPlantItem; 
if (plant != null) 
    plant.DoSomething(); 

rodzajów Wykorzystanie bazy w miarę możliwości.To powstrzyma cię od rzucania tak bardzo. Jak wcześniej wspomniano, nie należy odlewać do ObserableCollection<T>, aby wywołać metodę na ICollection

Użyj generycznych. Jeśli potrzebujesz logiki specyficznej dla typu, utwórz abstrakcyjną klasę bazową (lub po prostu interfejs, jeśli nie potrzebujesz żadnej logiki współdzielonej) z parametrem generycznym. Następnie wykonaj implementacje tej klasy dla każdej implementacji interfejsu. Metody mogą być również ogólne. Mogę przepisać drugi przykład jako

public void DeletePart<TPlantItem>(TPlantItem plantItem) 
    where TPlantItem : IPlantItem 
{ 
    IEnumerable<TPlantItem> itmParent = GetParentPartByObjectId(_siteDocument, plantItem); 
    ((ObservableCollection<TPlantItem>)itmParent).Remove(plantItem); 
} 

Answer 2

Ten artykuł może rzucić pewne światło na sposób odlewania wpływa na wydajność

http://www.codeproject.com/Articles/8052/Type-casting-impact-over-execution-performance-in

Oto kilka ogólnych wskazówek dotyczących optymalizacji programów opartych na wyników uzyskanych w poprzednich sekcjach:

• Typowe typy konwersji są zazwyczaj drogie, wyjmij je z pętli i funkcji rekursywnych i użyj tych samych typów liczbowych kiedy jest możliwe .

• Downcasting to świetny wynalazek, ale kontrole typu mają duży wpływ na wydajność wykonania, sprawdzają typy obiektów poza pętlami i funkcjami rekursywnymi, i używają w nich operatora "as".

• Upcasting jest tani !, używaj go wszędzie, gdzie potrzebujesz.

• Twórz lekkie operatory konwersji, aby wykonać niestandardowe odlewy szybciej. Używane narzędzia

Answer 3

użytku

 ((System.Collections.IList)itmParent).Remove(plantItem); 

zamiast

 if (plantItem is OSiteEquipment) 
     ((ObservableCollection<OSiteEquipment>)itmParent).Remove((OSiteEquipment)plantItem); 

     if (plantItem is OSiteSubSystem) 
      ((ObservableCollection<OSiteSubSystem>)itmParent).Remove((OSiteSubSystem)plantItem); 

     if (plantItem is OSiteComponent) 
      ((ObservableCollection<OSiteComponent>)itmParent).Remove((OSiteComponent)plantItem);