Dispose() 是将数据保存到光盘的好地方吗?

Question

我正在创建一个基于json的平面文件数据库用于学习目的。我正在更改 JsonDatabase 对象包含的数据，但我想仅在完成所有更改后将它们保存到光盘。

这是我目前的方法：

 public class UserMatch
{
    public string FirstName { get; set; }
    public string LastName { get; set; }
}
public class JsonDatabase : IJsonDatabase, IDisposable
{
    private readonly FileStream _jsonDatabaseStream;
    private List<UserMatch> _userMatches;
    public List<UserMatch> UserMatches
    {
        get
        {
            if (_userMatches == default(List<UserMatch>))
            {
                string fileContent;
                using (var sr = new StreamReader(_jsonDatabaseStream))
                {
                    fileContent = sr.ReadToEnd();
                }
                _userMatches = JsonConvert.DeserializeObject<List<UserMatch>>(fileContent);
            }
            return _userMatches;
        }
        set { _userMatches = value; }
    }
    public JsonDatabase(string fileLocation)
    {
        _jsonDatabaseStream = File.Open(fileLocation, FileMode.OpenOrCreate, FileAccess.ReadWrite, FileShare.Read);
    }
    public void Dispose()
    {
        string serialisedJson = JsonConvert.SerializeObject(UserMatches);
        using (var sr = new StreamWriter(_jsonDatabaseStream))
        {
            sr.Write(serialisedJson);
        }
        _jsonDatabaseStream.Dispose();
    }
}

Dispose（） 是正确的地方吗？还是我应该创建一个单独的提交/保存方法？

Answer 1

Dispose（） 是正确的地方吗？

不，它用于清理。不要通过混合其他问题来混淆问题。

还是我应该创建一个单独的提交/保存方法？

是的，这将迫使调用代码专注于如何以及何时调用它。而且它只是更清晰。

Answer 2

我认为，在这种特殊情况下，一个好的模式是：

• 创建单独的提交方法和单独的回滚方法。
• 创建一个单独的 Close 方法，该方法根据关闭数据库的语义调用提交或回滚。
• 从释放方法调用 Close 方法，以便在释放数据库时关闭数据库。

通过这种方式，您可以进行关注点分离，并将语义清楚地分离到每个自己的方法中：

• 回滚，提交仅关注这些操作并保留数据。
• Close 关闭数据库，并定义在该操作事件上打开的事务所发生的情况的逻辑。
• "处置"通过关闭对象来清理对象。

Answer 3

我强烈推荐一个单独的保存方法。

释放

方法仅用于释放垃圾回收器无法清理的非托管资源。