Блог

У меня есть класс Player и класс Stat. Класс Player имеет свойство List, где PlayerStat имеет свойства List и 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; }
}
  

Должен признать, пока я действительно не вижу серьезного недостатка в дизайне вашего класса. Конечно, у меня нет никакого представления о более широкой картине и о том, как устроена ваша игра, поскольку это лишь небольшая ее часть.

Однако вы сказали, что это то GetCompletePlayerStats , что вам действительно не нравится. Мне пришлось прочитать это несколько раз, чтобы понять, что вы пытаетесь здесь сделать. Если я правильно понял, вы просто хотите вернуть PlayerStat объект, соответствующий каждому заданному Stat объекту. Я предполагаю, что 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;
}
  

Этот метод здесь не требует, чтобы a IQueryable а скорее IEnumerable выполнял итерацию через foreach цикл и выдавал соответствующий PlayerStats объект, если таковой имеется, или создавал новый с XP, установленным в 0, в противном случае (оператор объединения нулей ?? очень полезен в этих случаях).

Надеюсь, это поможет!

При существующем дизайне этот метод может быть упрощен таким образом:

 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 помощью, но вам также нужно, чтобы несовпадения приводили к нулевому результату), вы также можете сделать это с помощью операции GroupJoin() LINQ , хотя я подозреваю, что это было бы гораздо менее читабельно.

Что меня беспокоит в вашем дизайне, так это двусмысленность имен, например, у вас есть и PlayerStat класс, и PlayerStatistic свойство, и они означают немного разные вещи; подумайте о том, чтобы называть вещи по-другому, и они могут выглядеть лучше; на самом деле, это, вероятно, лучший совет по дизайну, который я могу вам дать, для любой ситуации 🙂