C# 实体框架：此代码是最佳的吗？我该如何改进它？

Question

我觉得下面的代码不是最理想的，它可以更好。我只是不知道如何在不提交更改的情况下获取密钥。

var now = DateTime.Now;
var update = new DAL.ServerStatus
{
    Accounts = status.Accounts,
    Clients = status.Clients,
    Created = now,
    ServerStarted = status.ServerStarted,
    ServerDateTime = status.ServerDateTime
};

var context = DataContext.GetDataContext();
context.ServerStatus.AddObject(update);
context.SaveChanges();

foreach (var character in characters)
{
    var characterUpdate = new DAL.ServerOnlineCharacter
    {
        Account = character.Account,
        Alliance = character.Alliance,
        Created = now,
        Criminal = character.Criminal,
        DonationPoints = character.DonationPoints,
        EventCredits = character.EventCredits,
        FactionTyped = character.FactionTyped,
        FactionPoints = character.FactionPoints,
        Fame = character.Fame,
        GameTime = character.GameTime,
        Guild = character.Guild,
        GuildAbbreviation = character.GuildAbbreviation,
        GuildTitle = character.GuildTitle,
        Karma = character.Karma,
        Kills = character.Kills,
        MapTyped = character.MapTyped,
        RaceTyped = character.RaceTyped,
        RawName = character.RawName,
        Serial = character.Serial,
        ServerStatusId = update.Id
    };

    context.ServerOnlineCharacters.AddObject(characterUpdate);
}

context.SaveChanges();

理想情况下，我希望这是事务性的，而不必两次访问数据库来完全提交更改。我怎样才能做到这一点？

顺便说一句，这是方法存根：

static void Update(IServerStatusUpdate status, IEnumerable<IServerOnlineCharacterUpdate> characters);

Answer 1

如果您将 update.Id 映射为主键，并且 ServerOnlineCharacter 和 ServerStatus 之间存在映射关系，那么您应该能够这样做，

characterUpdate.ServerStatus = update

而不是

characterUpdate.ServerStatusId = update.Id

这将允许您只需要在最后调用 context.SaveChanges （而不是在中间）。

至于使方法具有事务性，EntityFramework 遵守 TransactionScope，因此如果您将 Update 包装在 TransactionScope 周围并在准备好时提交它，它应该按您的预期执行。

这是事务使用的示例

如果您真的想要的话另外，您可能需要查看类似 AutoMapper 的内容来将您传递到 Update 的界面映射到你的 DAL 课程。这会减少很多代码。