开发者

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.

0

上一篇:

下一篇:

精彩评论

暂无评论...
验证码 换一张
取 消

最新问答

问答排行榜