在C#中使用多线程锁定DataTable的正确方法？

Question

这是锁定和修改DataTable多个线程共享的正确方法吗？如果没有,那么正确的做法是什么？

private void DoGeocodeLookup(object info)
{
    ParameterRow data = (ParameterRow)info;
    DataRow dr = data.Dr;
    int localIndex = data.Index;
    ManualResetEvent doneEvent = data.m_doneEvent; 

    Geocode_Google.GeoCode gc = new GeoCode();

    gc.Addr_In = m_dtAddress.Rows[localIndex]["AddressStd_tx"].ToString();

    gc.GeoCodeOne();

    if (gc.Status == "OK")
    {
        //write back to temporary datatable
        lock( m_TempTable.Rows.SyncRoot )
        {
            m_TempTable.Rows[localIndex]["GL_Address"] = gc.Thoroughfare_tx; 
        }
    }
    doneEvent.Set(); 
}

我的结构:

struct ParameterRow
{
    private DataRow m_row;
    private int m_index;

    public DataRow Dr
    {
        get { return m_row; }
        set { m_row = value; }
    }

    public int Index
    {
        get { return m_index; }
        set { m_index = value; }
    }

    public ManualResetEvent m_doneEvent;

    public ParameterRow( DataRow row, int index, ManualResetEvent doneEvent)
    {
        m_row = row;
        m_index = index;
        m_doneEvent = doneEvent; 
    }
}

我启动所有线程的片段:

//make temporary table
m_TempTable = new DataTable();
m_TempTable = m_dtAddress.Copy();

for (int index = 0; index < m_geocodes; index++)
{
    doneEvents[index] = new ManualResetEvent(false);
    ThreadPool.QueueUserWorkItem(DoGeocodeLookup, new ParameterRow( m_dtAddress.Rows[index], index, doneEvents[index]));  
}

WaitHandle.WaitAll(doneEvents);

Answer 1

您的示例不需要任何锁定DataTable.在DoGeocodeLookup你的内心只是执行的阅读DataTable.您在表上执行的唯一访问是查找一行,该行计为读取.该数据表类被标记为多线程读取操作安全.如果您在其中执行类似添加新行的操作,DoGeocodeLookup则需要锁定.

您要更改的唯一内容是DataRow指定的单个数据localIndex.由于每次调用都DoGeocodeLookup使用不同的行 - 表中的一行只会由一个线程更新,因此您不会遇到同步问题.所以这也不需要锁定.