1) Не позволяйте View инициализировать модель презентации, она должна быть наоборот. Модель представления является объектом интереса, а не особым видом.
public interface IView
{
void SetModel(IPresentationModel model);
}
publiv class View : UserControl, IView
{
public void SetModel(IPresentationModel model)
{
DataContext = model;
}
}
public class PresentationModel : IPresentationModel
{
public PresentationModel(IView view)
{
view.SetModel(this);
}
}
2) Не устанавливайте контекст данных подпредставления в коде файла. Обычно представление, использующее подпредставление, устанавливает контекст данных в файле xaml.
3) Обычно каждое представление имеет свою собственную модель представления. Модель представления должна иметь один тип представления. Это означает, что разные представления одной модели презентации могут отличаться по внешнему виду, но не по функциональности (в вашем случае одно представление используется для поиска, другое - для отображения и редактирования данных). Итак, вы избрали принцип единой ответственности.
4) Абстрагируйте свой уровень доступа к данным, иначе вы не сможете провести модульное тестирование вашей модели презентации (потому что для этого нужен прямой доступ к базе данных). Определите интерфейс репозитория и реализацию:
public interface IUserRepository
{
User GetById(int id);
}
public class EntityFrameworkUserRepository : IUserRepository
{
private readonly DBEntities _entities;
public EntityFrameworkUserRepository(DBEntities entities)
{
_entities = entities;
}
public User GetById(int id)
{
return _entities.SingleOrDefault(u => u.UserId == id);
}
}
5) Не используйте FirstOrDefault, поскольку идентификатор уникален, поэтому для одного идентификатора не должно быть нескольких пользователей. SingleOrDefault (используется в приведенном выше фрагменте кода) выдает исключение, если найдено более одного результата, но возвращает ноль, если ничего не найдено.
6) Привязать непосредственно к вашей сущности:
public interface IPresentationModel
{
User User { get; }
}
<StackPanel DataContext="{Binding Path=User}">
<TextBox Text="{Binding Path=FirstName}" />
<TextBox Text="{Binding Path=LastName}" />
</StackPanel>
7) Используйте CommandParameter, чтобы предоставить идентификатор пользователя, который вы ищете, непосредственно с вашей командой.
<TextBox x:Name="UserIdTextBox">
<Button Content="Search" Command="{Binding Path=SearchCommand}"
CommandParameter="{Binding ElementName=UserIdTextBox, Path=Text}" />
public class PresentationModel
{
public ICommand SearchCommand
{
// DelegateCommand<> is implemented in some of Microsoft.BestPractices
// assemblies, but you can easily implement it yourself.
get { return new DelegateCommand<int>(Search); }
}
private void Search(int userId)
{
_userRepository.GetById(userId);
}
}
8) Если проблемы связаны только с привязкой данных, посетите следующий веб-сайт, чтобы узнать, как отлаживать привязки данных wpf: http://beacosta.com/blog/?p=52
9) Не используйте строки, содержащие имена свойств. После того, как вы выполните рефакторинг, ваш код и свойства поменяют свои имена, чтобы у вас было время найти все имена свойств в строках и исправить их. Вместо этого используйте лямбда-выражения:
public class PresentationModel : INotifiyPropertyChanged
{
private string _value;
public string Value
{
get { return _value; }
set
{
if (value == _value) return;
_value = value;
RaisePropertyChanged(x => x.Value);
}
}
public PropertyChangedEventHandler PropertyChanged;
public void RaisePropertyChanged(Expression<Func<PresentationModel, object>> expression)
{
if (PropertyChanged == null) return;
var memberName = ((MemberExpression)expression.Body).Member.Name;
PropertyChanged(this, new PropertyChangedEventArgs(memberName));
}
}
Я желаю вам всего наилучшего в решении вашей проблемы и надеюсь, что смогу вам немного помочь.
С наилучшими пожеланиями
Оливер Ханаппи