C# property refactoring - Should I care?
I have following code:
public class Header
{
    Line Lines { get { ...}}
    public ICryptographer GetCryptographer(FieldGroup group)
    {
        ...
    }
}
public class Line
{
    public Header Header { get; set; }
    public FieldGroup FieldGroup { get; set; }
    ICryptographer CryptoGrapher { get { return Header.GetCryptographer(FieldGroup); } }
    public decim开发者_C百科al? Month1
    {
        get { return _month1; }
        set
        {
            if (_month1 != value)
                Month1_Enc = CryptoGrapher.Encrypt(_month1 = value);
        }
    }
    private decimal? _month1;
    protected byte[] Month1_Enc { get; set; }
    //The same repeated for Month2 to Month12
}
public interface ICryptographer
{
    byte[] Encrypt(decimal? value);
    decimal? DecryptDecimal(byte[] value);
}
public enum FieldGroup
{
   ...
}
Shortly properties Month1 to Month12 are of type decimal? that should be encrypted before they are saved in database. I have also few other classes that have encrypted properties. Every property code looks exactly the same as Month1 I showed here.
Ideally I would like to have something like this:
Encrypted<decimal?> Month1 { get; set;}
but this is impossible because each object may have different Cryptographer (symmetric key).
Is there a way to refactor it to avoid such repeatable code?
Should I care about such repetition ?So for each Encrypted you need a reference to the parent, I'm I correct?
So my first attempt would be trying get a reference to the parent into each usage of Encrypted first. I think lightweight interfaces are good for this kind of job:
public interface IHasEncryptedProperties
{
    string GetKey();
}
And then implement them on classes than need encrypted properties
public class Line : IHasEncryptedProperties
{
    public string GetKey() { /* return instance-specific key; */ }
}
Then on Encrypted you then requires that a parent instance be passed in.
public class Encrypted<T>
{
    private IHasEncryptedProperties _parent;
    public Encrypted(IHasEncryptedProperties parent)
    {
        _parent = parent;
    }
    public T Value
    {
        get
        {
            var encryptor = GetEncryptor(_parent.GetKey());
            // encrypt and return the value
        }
    }
}
..
Hope this helps. If it doesn't, please leave a comment.
 
         加载中,请稍侯......
 加载中,请稍侯......
      
精彩评论