wiki:ReviewHeuristicLab3.3.0CodeCoreViews

Version 3 (modified by mkommend, 11 years ago) (diff)

--

HeuristicLab 3.3.0 Code Review - HeuristicLab.Core.Views-3.3

Reviewer: mkommend

  • ItemCollectionView: What is the ColumnHeader1 in the ItemCollectionView?
    • mkommend: The ColumnHeader1 is necessary because otherwise no items are displayed in the ListView.
  • VariableCollectionView: The VariableCollectionView produces StackOverFlowExceptions, if a Variable is dragged from the ListView to the DetailsView while it is displayed. Maybe this effects also base classes from VariableCollectionView
  • TypeSelector: In the Configure method Exceptions are caught, consider using more specific classes.