- Made Array.IndexOf available via extension method. - Made ToHashSet extension method. - Change collections queried often via Contains into sets. - Avoid Count() extension if Count or Length property exist. - Made Count() > 0 checks and variations calls to Any() instead. - Don't call ToList/ToArray if there is no benefit to materializing the sequence. - If the sequence does benefit from materialization, follow this general pattern: - Collection queried often via Contains use ToHashSet to speed up lookups. - Short lived variables use ToList. This is because ToArray requires an extra copy to output the final size. - Collections persisted into fields or for a long time use ToArray to minimize memory overhead.
45 lines
951 B
C#
45 lines
951 B
C#
#region Copyright & License Information
|
|
/*
|
|
* Copyright 2007-2015 The OpenRA Developers (see AUTHORS)
|
|
* This file is part of OpenRA, which is free software. It is made
|
|
* available to you under the terms of the GNU General Public License
|
|
* as published by the Free Software Foundation. For more information,
|
|
* see COPYING.
|
|
*/
|
|
#endregion
|
|
|
|
using System.Collections.Generic;
|
|
using System.Linq;
|
|
|
|
namespace OpenRA
|
|
{
|
|
public class Group
|
|
{
|
|
Actor[] actors;
|
|
int id;
|
|
|
|
static int nextGroup;
|
|
|
|
public IEnumerable<Actor> Actors { get { return actors; } }
|
|
|
|
public Group(IEnumerable<Actor> actors)
|
|
{
|
|
this.actors = actors.ToArray();
|
|
|
|
foreach (var a in actors)
|
|
a.Group = this;
|
|
|
|
id = nextGroup++;
|
|
}
|
|
|
|
public void Dump()
|
|
{
|
|
/* debug crap */
|
|
Game.Debug("Group #{0}: {1}".F(
|
|
id, actors.Select(a => "#{0} {1}".F(a.ActorID, a.Info.Name)).JoinWith(",")));
|
|
}
|
|
|
|
/* TODO: add lazy group path crap, groupleader, pruning, etc */
|
|
}
|
|
}
|