Make realization if the thread-safe priority queue #10
Conversation
|
|
||
| public static class Program | ||
| { | ||
| public static void Main(string[] args) |
There was a problem hiding this comment.
Это не нужно. Как метод Main в современном .NET, так и точка входа в проекте, который по сути является библиотекой
| public void EnqueueFromDifferentThreads() | ||
| { | ||
| var priorityQueue = new PriorityQueue<int>(); | ||
| var threads = new Thread[3]; |
There was a problem hiding this comment.
new Thread в современном .NET не нужен, есть пул потоков и Task.Run (или более высокоуровневые вещи, типа Parallel.ForEach)
| correctValues[0] = "true"; | ||
| correctValues[1] = "false"; | ||
| correctValues[2] = "abobus"; | ||
| Assert.AreEqual(correctValues, values); |
There was a problem hiding this comment.
Это не должно проходить :) Dequeue делается в разных потоках, и кто из них будет первым, тот и запишет в свою ячейку взятое из очереди значение. То есть даже если очередь возвращает всё как надо, values могут быть перемешанными
| [Test] | ||
| public void EnqueueAndDequeueFromDifferentThreadsWithoutExceptions() | ||
| { | ||
| var priorityQueue = new PriorityQueue<string>(); |
There was a problem hiding this comment.
Вынесли бы создание очереди в SetUp
| thread.Join(); | ||
| } | ||
| Assert.AreEqual(0, priorityQueue.Size); | ||
| } |
There was a problem hiding this comment.
Всего два потока, всего один раз...
| /// Interface of thread-safe priority queue | ||
| /// </summary> | ||
| /// <typeparam name="TValue">Type of value</typeparam> | ||
| public interface IPriorityQueue<TValue> |
There was a problem hiding this comment.
Интерфейс не просили, хотя сделать его — вряд ли ошибка :)
| { | ||
| lock (locker) | ||
| { | ||
| while (counter == 0) |
There was a problem hiding this comment.
Кажется, counter должен быть volatile-полем, потому что этот поток может ещё не увидеть изменений из 25-й строки и снова встанет на Wait, будет дедлок. В .NET memory model вроде есть что-то про то, что взятие/освобождение замка влечёт синхронизацию памяти, так что на практике, скорее всего, всё будет хорошо, но мне кажется, лучше избегать таких проблем явно
No description provided.