类设计问题：列表联合和所有统计数据

Question

我有一个 Player 类和一个 Stat 类。 Player 类有一个 List 属性，其中 PlayerStat 有一个 List 和 XP 属性。我认为我的设计有缺陷，因为我在做我认为应该很容易的事情时遇到了困难。我想列出所有玩家及其所有统计数据的 XP。下面是这些类和 GetCompletePlayerStats 方法的更多详细信息，这是我真正不喜欢的。我基本上需要列出玩家所有统计数据的 XP，如果玩家没有统计数据，那么它的 XP 应该为零。任何设计帮助/建议将不胜感激。

public class Stat : EntityBase{
  public virtual string Name { get; set; }
  public virtual UnitOfMeasure Unit { get; set; }
  public virtual int UnitXP { get; set; }
}
public class Player : EntityBase{
  public virtual string Name { get; set; }
  public virtual IList<PlayerStat> PlayerStats { get; set; }

  public virtual List<PlayerStat> GetCompletePlayerStats(IQueryable<Stat> stats)
    {
        var allStats = new List<PlayerStat>();
        var playerStatIds = PlayerStats.Select(ps => ps.PlayerStatistic.Id).ToList();
        if (playerStatIds.Count == 0)
        {
            allStats.AddRange(stats.Select(stat => new PlayerStat() {PlayerStatistic = stat, XP = 0}));
        }
        else
        {
            var zeroStats = stats.Where(s => !playerStatIds.Contains(s.Id)).ToList();

            allStats.AddRange(zeroStats.Select(zeroStat => new PlayerStat() {PlayerStatistic = zeroStat, XP = 0}));
            allStats.AddRange(PlayerStats);
        }

        return allStats;
    }

}
public class PlayerStat : EntityBase{
  public virtual Stat PlayerStatistic { get; set; }
  public virtual double XP { get; set; }
}

Answer 1

我必须承认，到目前为止，我并没有真正看到你们的班级设计有什么重大缺陷。当然，我对大局以及游戏的设计方式没有任何见解，因为这只是其中的一小部分。

但是，您说您并不真正喜欢它是 GetCompletePlayerStats 。我必须读几遍才能理解你在这里想做什么。如果我没看错的话，您只想返回与每个给定 Stat 对象相对应的 PlayerStat 对象。我猜 Stat 有一个 Id 字段（您在方法中使用它）来比较其中两个字段的语义相等性。

鉴于到目前为止我做出了正确的假设（不幸的是，您没有提供太多信息），该方法可以简化为：

public virtual IEnumerable<PlayerStat> GetCompletePlayerStats(IEnumerable<Stat> stats)
{
    foreach(Stat stat in stats)
        yield return PlayerStats.FirstOrDefault(s => stat.Id == s.PlayerStatistic.Id) ?? 
                     new PlayerStat() {PlayerStatistic = stat, XP = 0};
    yield break;
}

这里的方法不需要 IQueryable 而是需要 IEnumerable 通过一个 foreach 循环进行迭代，并生成相应的 PlayerStats 对象（如果有的话），或者创建一个新的，否则将 XP 设置为 0（空合并运算符 ?? 在这些情况下非常有用）。

希望有帮助！

Answer 2

使用现有的设计，可以这样简化此方法：

public virtual IList<PlayerStat> GetCompletePlayerStats(IEnumerable<Stat> stats)
{
   // build a dictionary of existing stats by ID to facilitate 
   // the join with requested stats (effectively doing a hash join)
   var playerStatsById = PlayerStats.ToDictionary(ps => ps.PlayerStatistic.Id);

   // for each requested stat, return either the corresponding player stat 
   // or a zero stat if one isn't found, maintaining the original order of stats
   return stats
       .Select(s => playerStatsById.ContainsKey(s.Id) ? 
           playerStatsById[s.Id] : 
           new PlayerStat { PlayerStatistic = s, XP = 0 })
       .ToList();
}

请注意，由于这实际上是一个外连接操作（您将 stats 与 PlayerStats 连接，但您还需要不匹配的结果为零）您也可以使用 LINQ 的 GroupJoin() 操作来完成此操作，尽管我怀疑这会降低可读性。

你的设计让我困扰的是名称的模糊性，例如你同时拥有 PlayerStat 类和 PlayerStatistic 属性，它们的含义略有不同；考虑以不同的方式命名事物，事情可能看起来更好；事实上，对于任何情况，这可能是我能给你的最好的设计建议:)