我想实现一个优先级队列,可以通过它来指定队列的类型(通用队列或唯一队列)。
public class PriorityQueue<TItem, TKey, TQueue> : IEnumerable<TItem>
where TKey : IComparable
where TQueue : Queue<TItem>, new ()
{
private readonly Func<TItem, TKey> _keySelector;
private readonly SortedDictionary<TKey, TQueue> _data = new SortedDictionary<TKey, TQueue>();
public PriorityQueue(Func<TItem, TKey> keySelector)
{
_keySelector = keySelector;
}
public void Enqueue(TItem item)
{
var key = _keySelector(item);
if (!_data.ContainsKey(key))
{
_data.Add(key, new TQueue());
}
var queue = _data[key];
queue.Enqueue(item);
}
public TItem Dequeue()
{
if (IsEmpty)
{
throw new ArgumentException("Queue is EMPTY");
}
var key = _data.Keys.First();
var queue = _data[key];
var item = queue.Dequeue();
if (queue.Count == 0)
{
_data.Remove(key);
}
return item;
}
public bool IsEmpty => _data.Count == 0;
private int Count
{
get
{
var count = 0;
foreach (var key in _data.Keys)
{
count += _data[key].Count;
}
return count;
}
}
public void Clear()
{
_data.Clear();
}
public IEnumerator<TItem> GetEnumerator()
{
return new Enumerator(_data);
}
IEnumerator IEnumerable.GetEnumerator()
{
return GetEnumerator();
}
private class Enumerator : IEnumerator<TItem>
{
private readonly SortedDictionary<TKey, TQueue> _dictionary;
private readonly int _dictionaryCount;
private int _currentPosition;
public Enumerator(SortedDictionary<TKey, TQueue> data)
{
_dictionary = data;
_dictionaryCount = DictionaryCount;
_currentPosition = 0;
Current = default(TItem);
}
private int DictionaryCount
{
get
{
var count = 0;
foreach (var key in _dictionary.Keys)
{
count += _dictionary[key].Count;
}
return count;
}
}
public void Dispose()
{
}
public bool MoveNext()
{
if (_currentPosition >= _dictionaryCount)
{
return false;
}
Current = GetCurrent();
_currentPosition++;
return true;
}
public void Reset()
{
_currentPosition = 0;
}
public TItem Current { get; private set; }
private TItem GetCurrent()
{
var sum = 0;
var item = default(TItem);
foreach (var key in _dictionary.Keys)
{
var queue = _dictionary[key];
sum += queue.Count;
if (sum > _currentPosition)
{
item = queue.Take(queue.Count - (sum - _currentPosition) + 1).Last();
break;
}
}
return item;
}
object IEnumerator.Current => Current;
}
}
和唯一队列
public class UniqueQueue<T> : Queue<T>
{
private HashSet<T> _hashSet;
public UniqueQueue()
{
_hashSet = new HashSet<T>();
}
public new void Enqueue(T item)
{
if (_hashSet.Add(item))
{
base.Enqueue(item);
}
}
public new T Dequeue()
{
var item = base.Dequeue();
_hashSet.Remove(item);
return item;
}
public new void Clear()
{
_hashSet.Clear();
base.Clear();
}
}
但是在单元测试中,它失败并显示以下代码
[TestMethod]
public void TestPriorityUniqueQueue()
{
var puq = new PriorityQueue<Node, int, UniqueQueue<Node>>(node => node.Key);
var node1 = new Node(1, "One");
var node2 = new Node(2, "Two");
var node3 = new Node(1, "One 1");
puq.Enqueue(node1);
puq.Enqueue(node1);
puq.Enqueue(node1);
puq.Enqueue(node2);
puq.Enqueue(node3);
var list = new List<Node>();
foreach (var node in puq)
{
list.Add(node);
}
Assert.AreEqual(list[0], node1);
**Assert.AreEqual(list[1], node3);**
Assert.AreEqual(list[2], node2);
puq.Dequeue();
puq.Dequeue();
puq.Dequeue();
Assert.IsTrue(puq.IsEmpty);
}
我调试了测试用例,发现在调用puq.Enqueue(node1)时,它从Queue而不是从UniqueueQueue中调用Enqueue()函数。但是,当我调试时,我发现在PriorityQueue的Enqueue中,该变量是UniqueQueue的类型,但它没有调用UniqueQueue的Enqueue()。我真的很想知道为什么,或者给我一些建议,这可能吗?
以粗体显示失败,并且堆栈跟踪为
Assert.AreEqual失败。预期:。实际:。 在ToolsTests.PriorityQueueTest.TestPriorityUniqueQueue()中的位置somefolder \ ToolsTests \ PriorityQueueTests.cs:行号118
the debug info of the queue variable from PriorityQueue<>.Enqueue() methord
答案 0 :(得分:0)
您的通用类型参数TQueue<T>
的类型应为Queue<T>
,因为您是在PriorityQueue
的通用约束中定义的:
where TQueue : Queue<TItem>, new ()
现在只要您的代码通过:
_data.Add(key, new TQueue());
它将创建Queue<T>
而不是UniqueQueue<T>
的实例,因此它将永远不会到达Enqueue
的{{1}}方法,因为它调用了UniqueQueue<T>
Enqueue
的方法。
之所以这样做,是因为您提供了一般约束(上面的第一个代码块)。它根本不知道Queue<T>
的类型,而是UniqueQueue<T>
的类型,因为Queue<T>
的期望是这样(因为您这样“告诉”了他)。因此,只要执行PriorityQueue<TItem, TKey, TQueue>
,就会实例化一个新的new TQueue()
对象,而不是Queue<T>
。
为了使测试通过,并使UniqueQueue<T>
实际使用PriorityQueue<TItem, TKey, TQueue>
,必须在通用约束中明确指定要使用UniqueQueue<T>
:>
UniqueQueue<T>