15

多くのWebサービス呼び出し（非同期）があります。コールバックで、結果をExcelにプロットします。プロットメソッドを同期させたい。したがって、私は以下を使用しますが、Visual Studioで追跡すると、毎回lock（locker）が成功し、clearcommentIfany、plotを実行しているスレッドが多数あります。なぜこれが期待どおりに機能しないのか理解できません！ありがとう

private readonly object locker = new object();

void ProcessPlot()
{
    lock (locker)
    {
        Debug.WriteLine("currentThreadID: " + Thread.CurrentThread.ManagedThreadId);
        //Helper.Dispatcher.Invoke(new AddClearCommentDelegate(ClearCommentIfAny));
        ClearCommentIfAny();

        if (Response.status != Status.COMPLETE)
        {
            ErrorMessage = ManipulateStatusMsg(Response);
            //Helper.Dispatcher.Invoke(new AddClearCommentDelegate(AddCommentToCell));
            AddCommentToCell();
        }
        else // COMPLETE
        {
            // TODO: Convert this into factory pattern
            Debug.WriteLine("ReportBuilder.Dispatcher's address " + Helper.GetAddress(Helper.Dispatcher));
            //Helper.Dispatcher.Invoke(new PlotDelegate(Plot), Response);
            Plot(Response);
        }               
    } 
}

public void DataRequestJobFinished(DataRequestResponse response)
{
    try
    {
        if (Request.IsRequestCancelled)
        {
            Request.FormulaCell.Dispose();
            return;
        }

        Response = response;

        //if (response.status != Status.COMPLETE)
        //{
        //    ErrorMessage = ManipulateStatusMsg(response);
        //}
        //else // COMPLETE
        //{
        //    // TODO: Convert this into factory pattern
        //    PlotDelegate plotDelegate = Plot;
        //    Debug.WriteLine("ReportBuilder.Dispatcher's address " + Helper.GetAddress(Helper.Dispatcher));
        //    Helper.Dispatcher.Invoke(plotDelegate, response);
        //    //Plot(response);                 
        //}
        var t = new Thread(ProcessPlot);
        t.Start();
        //ProcessPlot();
    }
    catch (Exception e)
    {
        ErrorMessage = e.Message;
        MIMICShared.Helper.LogError(e);
    }
    finally
    {
        //TODO: PutReportBuilderInQueue(this);
        ReadyForPlot = true;
        //Request.FormulaCell.Dispose(); move this after plot
        UnityContainer.Resolve<IEventAggregator>().GetEvent<DataRefreshEvent>().Publish(ID);
    }
}
4

1 に答える 1

39

ここでの問題は、ロックが静的（タイプレベル）メンバーではなくインスタンスメンバーであることにあると思われます。

各スレッドがそれを含むクラスの独自のインスタンスを持っていると仮定すると、それはあなたのロッカーの独自のインスタンスも持っているでしょう-これはあなたが望むものではありません。

代わりに次の宣言を試してください。

private static readonly object locker = new object();

staticキーワードを含めると、このオブジェクトインスタンスがタイプレベルで存在するようになります。つまり、クラスのすべてのインスタンスで共有されます。

于 2012-04-13T15:17:42.220 に答える