将 DataGridView 绑定到 DataSource - 引发CurrencyError IndexOutOfRangeException

Question

我已经为这个问题困惑了几天，它让我很沮丧，但说实话，我还没有那么有经验，而且我在使用 DataGridView 时遇到了麻烦 - 这似乎是一个常见的话题。

public partial class frmMain : Form
{
    ServerConnection sabCom;
    private BindingSource jobSource = new BindingSource();
    private void timer1_Tick(object sender, EventArgs e)
    {
            if (bgUpdateThread.IsBusy == false)
            {
                bgUpdateThread.RunWorkerAsync(sabCom);
            }
        }
    }

    private void frmMain_Load(object sender, EventArgs e)
    {
        timer1.Interval = 3000;
        timer1.Start();
    }

    private void bgUpdateThread_DoWork(object sender, DoWorkEventArgs e)
    {
        ServerConnection s = e.Argument as ServerConnection;
        s.update();
        e.Result = s;
    }

    private void bgUpdateThread_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
    {
        this.sabCom = e.Result as ServerConnection;

        if (dgvQueueFormatted == false)
        {
            dgvQueue_Init();  //Applies formatting and loads column width. Inits data sources.
        }
        else
        {
            dgvQueue_Update();

        }
    }

    private void dgvQueue_Update()
    {
        dgvQueue.Refresh();
    }

    private void dgvQueue_Init()
    {
        try
        {
            jobSource.DataSource = sabCom.queue.jobs;
            dgvQueue.DataSource = jobSource;
            try
            {
                //Apply saved column spacing to the dgvQueue
                //Uses reflection to set dgvQueue to DoubleBuffer
            }
            catch
            { }
        }
        catch
        { }
    }

    private void frmMain_FormClosing(object sender, FormClosingEventArgs e)
    {
        //Saves information about the dgvQueue on shutdown.
    }

队列类：

public class Queue  
{
    private string _status;
    public string status { get { return _status; } set { _status = value; } }
    private string _eta;
    public string eta { get { return _eta; } set { _eta = value; } }
    private List<Job> _jobs;
    public List<Job> jobs
    {
        get
        {
            return _jobs;
        }
        set
        {
            _jobs = value;
        }
    }
    private List<string> _categories;
    public List<string> categories { get { return _categories; } set { _categories = value; } }

    private XmlDocument xmld;
    private ServerConnection m_parent;
    private XmlNodeList _xmljobs;

    public Queue(ServerConnection srvConn)
    {
        //fetch the Queue xml
        m_parent = srvConn;
        xmld = new XmlDocument();
        _jobs = new List<Job>();
    }

    public void update()
    {
        updateXml();
        updateQueue();
        updateJobs();
    }

    private void updateXml()
    {
        //Loads xml file into xmld
    }

    private void updateQueue()
    {
        XmlNodeList elements = xmld.SelectNodes("queue");

        foreach (XmlNode element in elements)
        {
            _status = element.SelectSingleNode("status").InnerText;
            _eta = element.SelectSingleNode("eta").InnerText;
        }
    }

    private void updateJobs()
    {
        _xmljobs = xmld.SelectNodes("queue/job");
        jobs.Clear();

        foreach (XmlNode xmljob in _xmljobs)
        {
            Job t_job;

            _status = xmljob.SelectSingleNode("status").InnerText;
            _eta = xmljob.SelectSingleNode("eta").InnerText;

            //Create temp job to match against list.
            t_job = new Job(_status, _eta);
            jobs.Add(t_job);
        }
    }

作业类： 实际上，它包含大约 30 个不同类型的值，但它们都采用相同的格式：

public class Job
{
    private int _status;
    public int status { get { return _status; } set { _status = value; } }
    private string _eta;
    public string eta { get { return _eta; } set { _eta = value; } }


    public Job(string status, string eta)
    {
        _status = status;
        _eta = eta;
    }
}

与 DataGridView 交互时，出现错误：

DataGridView 中发生以下异常：

System.IndexOutOfRangeException： Index没有价值。 在 System.Windows.Forms.CurrencyManager.get_Item（Int32 索引） 在 System.Windows.Forms.DataGridViewDataConnection.GetError(Int32boundColumnIndex, Int32 columnIndex, Int32 rowIndex)

进入调试器时，它会在初始 Application.Run(new frmMain() 上触发。我到底做错了什么？程序仍然功能和更新正常，但我什至无法处理该事件以抑制默认错误消息

！ 无需清除列表并重新创建它，只需更新其中的值效果会更好。目前我有这段代码：

            t_job = _jobs.FirstOrDefault(c => c.nzo_id == t_nzo_id);
            if (t_job == null) //Job not in list, insert
            {
                t_job = new Job(t_status, i_index, t_eta, i_timeLeft, t_age, i_mbleft, i_mb, t_filename, i_priority, t_category, i_percentage, t_nzo_id, this);
                jobs.Add(t_job);
            }
            else //update object in current list
            {
                jobs[t_job.Index].status = t_status;
                jobs[t_job.Index].priority = i_priority;
                jobs[t_job.Index].category = t_category;
                jobs[t_job.Index].percentage = i_percentage;
                jobs[t_job.Index].timeleft = i_timeLeft;
                jobs[t_job.Index].mbleft = i_mbleft;
            } 

这可以阻止它！

Answer 1

问题似乎是，刷新间隔为 1 秒，当用户滚动或尝试访问数据字段时，数据会不断被删除和读取。
这会导致绑定到调用 try 调用一个它认为可能仍然存在但实际上并不存在的值，这就是为什么您会遇到索引超出范围异常的原因

我建议做的第一件事也是在您的 updateJobs 方法中与像队列列表一样刷新的所有其他列表一样，不是每次都清除列表，而是首先检查 xml 中的作业是否存在于当前作业列表中，如果存在，那么您可以更改当前值（如果值）已经改变，否则什么也不做。

要检查作业是否存在，很简单：

t_job = _jobs.FirstOrDefault(c => c.Filename == t_filename);

如果作业不存在，这将返回 null，我假设文件名可能不是唯一的，因此可能需要更改它以使其确实是唯一的，即

t_job = _jobs.FirstOrDefault(c => c.Filename == t_filename && c.nzo_id == t_nzo_id);

您会做的一件事现在需要考虑的是旧作业不会自动删除，因此每当添加新的历史作业时，首先检查队列中是否存在它，然后将其删除，然后再将其添加到历史列表中。

因此，将您的属性更改为：

public int Index 
        { 
            get { return _index; } 
            set 
            { 
                if (_index != value)
                    _index = value; 
            } 
        }

而不是：

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

另一件事是 try catch 异常的成本很高，因此不要使用：

try { i_percentage = double.Parse(t_percentage); } catch { }

因为您知道如果有一个值它将是一个双精度值，您可以将其更改为：

if (!string.IsNullOrEmpty(t_percentage))
    i_percentage = double.Parse(t_percentage);

现在如果您不知道 t_percentage 中的值是否为双精度，您可以使用 try-parse：

if (!string.IsNullOrEmpty(t_percentage))
    double.TryParse(t_percentage,out i_percentage);

这样可以避免异常引起的开销。这可能是微观优化，如果它实际上不会引起问题，则并不总是必要的，但考虑到您可以有数百个作业，每个作业每秒刷新 10 个左右的属性，如果甚至有 2 个作业，事情实际上会变得明显更慢10 个属性抛出异常。

另一件事是，在后台工作程序完成后，在方法：dgvQueue_Update()中，您正在调用ResetBindings(true);，这会导致整个数据网格刷新，而不仅仅是刷新个别项目。

尝试将其更改为：

for (int i = 0; i < jobSource.List.Count; i++)
    jobSource.ResetItem(i);

区别在于：

this.OnListChanged(new ListChangedEventArgs(ListChangedType.ItemChanged, itemIndex));

与您说 ResetBindings 时相比：

this.OnListChanged(new ListChangedEventArgs(ListChangedType.Reset, -1));