2

Необходим простенький класс-счетчик, который будет возвращать число и увеличивать на 1. Вот такой класс есть сейчас:

class Sequence
{
    private ulong _number { get; set; }
    public Sequence()
    {
        this._number = ((UInt64)(DateTime.UtcNow.Subtract(new DateTime(1970, 1, 1))).TotalMilliseconds);
    }

    public ulong Next()
    {
        return _number++;
    }
}

Доступ к нему имеют несколько потоков, и практически без задержек получают новое число функцией Next(). Иногда вылетала вот такая ошибка InvalidOperationException: out of sync, и бывали моменты когда в нескольких потоках было одинаковое число. Как можно сделать синхронизацию, при этом не в ущерб производительности?

1 ответ 1

2

Вы не должны экономить на спичках и жалеть одну несчастную блокировку. Но если очень хочется:

class Sequence
{
    private long number;

    public Sequence()
    {
        long startValue = ...; // вычисляйте как угодно
        Interlocked.Exchange(ref number, startValue);
    }

    public ulong Next()
    {
        return (ulong)Interlocked.Increment(ref number);
    }
}

Обратите внимание, что InvalidOperationException не могло быть выброшено при простом увеличении переменной, даже пускай несинхронизированном. Ищите эту ошибку ещё где-то.

4
  • Только компилятор ругается на то что переменную класса нельзя передавать как ref 12 фев 2016 в 23:07
  • @DimaGvozdev: Странно, сейчас уточню.
    – VladD
    12 фев 2016 в 23:09
  • 1
    @DimaGvozdev: Вы случайно не сделали number свойством?
    – VladD
    12 фев 2016 в 23:10
  • А, да. Не убрал { get; set; } 12 фев 2016 в 23:19

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.