ABP天坑--UOW自动保存修改

ABP项目也做了挺久了，实际上也没碰上太多的问题，但这两天被ABP的自动保存修改（Automatically Saving Changes）这个天坑给恶心坏了

因为项目设计上的问题，原先为了方便加上环境限制，没有文件服务器来进行文件持久化服务，这样多个系统之间如何共享文件就成了个问题，所以当初设计上采取了个偷懒的方式，直接将文件保存到MySql中，所有系统要用这些文件时，都去数据库中读取……

上面描述了问题产生的原因，现在项目已经上线，且运行了一段时间，发现文件存在数据库中导致数据库增长速度很快，而且未来迁移也不方便，所以考虑在原有直接存储到数据库中的方式，增加其他存储方式（注意是增加，不是全部替换），于是很简单的，直接给相应数据表增加一个字段标志存储方式（StorageMode），而原先存储文件数据的字段（Data）则用于存储其他方式持久化文件后对应的定位标志（比如文件路径等）

为了方便未来扩展出其他存储方式，所以我定义了如下存储接口

    /// <summary>
    /// 定义文件保存和获取的方式
    /// </summary>
    public interface IStorage
    {
        /// <summary>
        /// 文件保存，并返回保存后的编码结果
        /// </summary>
        /// <param name="fileType"></param>
        /// <param name="data"></param>
        /// <param name="fileName">要保存的文件名，如果不设置则随机命名</param>
        /// <returns></returns>
        byte[] Save(FileType fileType, byte[] data, string fileName = null);
        /// <summary>
        /// 根据请求编码获取对应的文件数据
        /// </summary>
        /// <param name="fileType"></param>
        /// <param name="input"></param>
        /// <returns></returns>
        byte[] Get(FileType fileType, byte[] input);
        /// <summary>
        /// 获取文件数据对应的文件名，如不支持则返回null
        /// </summary>
        /// <param name="fileType"></param>
        /// <param name="input"></param>
        /// <returns></returns>
        string GetFileName(FileType fileType, byte[] input);
        /// <summary>
        /// 删除文件数据
        /// </summary>
        /// <param name="fileType"></param>
        /// <param name="input"></param>
        /// <returns></returns>
        bool Remove(FileType fileType, byte[] input);
    }

其中FileType是按业务进行划分的文件类型，然后我依次定义了InDatabaseStorage（数据库存储）以及LocalDiskStorage（本地磁盘存储）两种实现，当然这部分在这里就不细写了，毕竟不是本文的关键……

下来到问题产生的重点了，为了统一文件的管理，以及符合ABP的设计风格，所以我们在Core层定义了FileRefManager这样一个Manager类来统一封装处理文件的管理，也就是在这里，我们完整的封装了如何根据存储方式来获取对应的IStorage，然后又如何在存储时进行转化，最后在获取时，为了尽可能少的修改其它代码，我们采用了一个取巧的方式，将文件通过对应途径获取数据流后，赋值回Data字段并予以返回，这样原先通过Data字段来获取文件流的其它代码就不用进行修改

想法是美好的，事实是残酷的，测试时，当我将某种FileType设置为存储到本地磁盘时，读取出来居然报“路径存在非法字符”，反复DEBUG，存进去没错，但读取就是错，这什么鬼？难道DEBUG后程序还做了其它什么我不知道的事情？

徒劳一天，一无所获后，第二天终于有所突破，因为可以确定，在保存成功时，不管是本地磁盘，还是数据库里的byte长度，都是正确的，而且不管保存后多久，只要我不通过程序查询，直接在数据库中查询，Data字段始终正确，而一旦我在程序中查询了，随后就可以立刻发现：数据库中的Data字段变了，又变成了以数据库存储时，存储的文件本身！！！而正是这个现象，脑中突然灵光一闪，当初看ABP资料时，貌似有看到过ABP会自动保存，赶紧去ABP官网一查，果然在UOW对应的说明中有如此一段，既然找到了问题，那解决问题相对就简单了，解决的途径也有好几种可以采纳

a）禁用UOW，但此方法可行性不高，因为按照ABP说明，UOW是否启用是按最外层调用方决定的，而UOW在ABP中其扩散范围并不仅仅只在Core层中

b）Data字段保持不变，返回时另外增加一个FactData字段，但这样需要将所有涉及文件存储的地方均进行代码修改，改动量太大，有些得不偿失

c）在返回文件实体时，既然不能对原数据实体进行修改，那么为何不干脆直接深Copy一个新的数据实体，然后在新实体上进行Data修正，最终返回新实体呢？此方案最终尝试果然可行，而且这样也无需修改其它涉及文件存储的代码

虽然问题解决了，但最终对于ABP自动保存这种设计思路还是觉得有些不可思议，就算认为我对实体做的任何修改，都可能应该进行持久化，但你总得有个途径让我告知系统不要自动保存吧，而禁用UOW这个方案又存在那么大的限制……

2018-06-11补充：

根据@protossyk 同学的评论，现在增加新的方案：

d）ABP自所以能自动保存，主要依赖的就是EF的实体跟踪，所以可以通过指定AsNoTracking来解决自动保存问题，写法示例如下

repository.GetAll().AsNoTracking().FirstOrDefaultAsync()