0

を実装する基本 EF エンティティ クラスがありますINotifyPropertyChanged。

基本プロパティ Id は私の例です:

  /// <summary>
  /// Entity Id
  /// </summary>
  public int Id {
     get { return id; }
     set { SetValue<int>(() => (Id != value), (v) => id = v); } // < can this be simplified into a single call?
  }

... SetValue が定義されている場所: 

  protected void SetValue<TValue>(Expression<Func<bool>> evalExpr, Action<TValue> set) {
     // Compile() returns a Func<bool>
     var doSetValue = evalExpr.Compile();

     if (doSetValue()) {
        var expr = evalExpr.Body as BinaryExpression;
        //  this is not compiling - how do I decompose the expression to get what I need?
        var propertyName = ((PropertyExpression)expr.Left).Name;
        var assignValue = (TValue)((ConstantExpression)expr.Right).Value;

        set(assignValue);
        _propertyChangedHandler(this, new PropertyChangedEventArgs(propertyName));
     }
  }

私が見つけることができるすべてのサンプルは、パラメーターを期待しています。セッター (SetValue 呼び出し) はできるだけ単純なものを好みます。つまり、入力パラメーターを 1 に減らす方法はありますか?

4

2 に答える 2

1

あなたは変わるべきです

var propertyName = ((PropertyExpression)expr.Left).Name;

に

var propertyName = ((MemberExpression)expr.Left).Member.Name;

コードはコンパイルされますが、実行していることは最適ではなく、信頼できるものではありません。そして、あなたはInvalidCastException！

呼び出しごとにコンパイルするのExpression<T>は最適ではありません。また、ユーザーが次のようなメソッドにラムダを渡すことをどのように確認できますか。

() => (Id != value)

そしてそうではない

() => (id != value) // using the field instead of property

また

() => (value != Id) // passing the property as the second operand

?

また、valueあなたの表現ではConstantExpression. それvalue自体は、プロパティの一部に対する単なるローカル変数でsetあり、ラムダ式に渡されると、クラス フィールドに昇格されます (値がキャプチャされます - 詳細については、こちらを参照してください)。だからあなたが持っているのはMemberExpression両側にあります。

[CallerMemberName].NET 4.5 ( )を使用できない場合は、このアプローチを使用することを強くお勧めします。

public class EntityBase : INotifyPropertyChanged
{
    protected virtual void OnPropertyChanged(string propName)
    {
        var h = PropertyChanged;
        if (h != null)
            h(this, new PropertyChangedEventArgs(propName));
    }

    public event PropertyChangedEventHandler PropertyChanged;

    protected bool ChangeAndNofity<T>(ref T field, T value, Expression<Func<T>> memberExpression)
    {
        if (memberExpression == null)
        {
            throw new ArgumentNullException("memberExpression");
        }

        var body = memberExpression.Body as MemberExpression;
        if (body == null)
        {
            throw new ArgumentException("Lambda must return a property.");
        }

        if (EqualityComparer<T>.Default.Equals(field, value))
        {
            return false;
        }

        field = value;
        OnPropertyChanged(body.Member.Name);
        return true;
    }
}

使い方は簡単です:

public class Person : EntityBase
{
    private int _id;
    public int Id
    {
        get { return _id; }
        set { ChangeAndNofity(ref _id, value, () => Id); }
    }
}
于 2013-04-23T02:58:25.463 に答える
1

あなたが持っているものよりも簡単なさまざまなオプションがあります（ここでは、それぞれがどれだけ好きかを大まかに並べたものをいくつか示します）：

「The .NET 4.5 Way」のコア コード スニペットを次に示します。

protected bool SetProperty<T>(ref T storage, T value, [CallerMemberName] String propertyName = null)
{
    if (object.Equals(storage, value)) return false;

    storage = value;
    this.OnPropertyChanged(propertyName);
    return true;
}

次のように使用します。

  /// <summary>
  /// Entity Id
  /// </summary>
  public int Id {
     get { return id; }
     set { SetValue(ref id, value); }
  }
于 2013-04-23T02:45:37.550 に答える