Threadsafe factory pattern

Question

Я кодирую некоторый простой диспетчер графа, и я решил использовать фабричный шаблон для создания ребер для моего графика. Я только начинаю изучать шаблоны дизайна и темы.

Я создал фабрику с возвращением некоторые объекты (ребра), метод создания, код:

public Edge<T> CreateEdge(Vertex<T> firstVertex, Vertex<T> secondVertex) 
{ 
    var edge = new Edge<T>(firstVertex, secondVertex, _nextIndex); 

    _nextIndex++; 

    return edge; 
} 

А вот моя проблема. Что, если эта фабрика будет разделяться несколькими потоками, возможно, что возвратите несколько ребер с одним индексом.

Какое оптимальное решение этой проблемы?
Должен ли я использовать блокировку или некоторые мьютексы?
Или это вина моих плохих классов дизайна, использование фабрики?

Редактировать # 1: Изменить IncreaseIndex() -> _nextIndex ++; ошибка при написании.

Answer 1

Вы можете использовать:

long _index; 

var nextIndex = Interlocked.Increment(ref _index); 

Это обеспечит ваш индекс остается уникальным по потокам. Использование простого поля здесь имеет наибольший смысл, поскольку оно является свойством вашего класса фабрики.

Answer 2

Альтернативой было бы обернуть функциональность IncreaseIndex в замке, или вы могли бы передать управление индексами в одноэлементный (Jon Skeet's Singleton implementation).