c# winform中的一段代码赏析

我遇到了一个bug，是客户测试我们的产品，报出来的，而且有异常信息文件，这对于定位问题，很有帮助。

我找到源码看了下，bug还无法重现。于是我随便点点客户端，经过了几次调试，结果报出错误来了。客户端界面如下：

代码如下：

  /// <summary>
         /// 选定项发生改变事件
         /// </summary>
         /// <param name="sender"></param>
         /// <param name="e"></param>
         private void listView_EditBibliographys_SelectedIndexChanged(object sender, EventArgs e)
         {

             if (listView.SelectedItems.Count!= )
             {
                 btnEdit.Enabled = false;
                 btnDelete.Enabled = false;
                 btnRenew.Enabled = false;
                 btnMoveDown.Enabled = false;
                 btnMoveUp.Enabled = false;
             }
             else
             {
                 btnEdit.Enabled = true;
                 btnDelete.Enabled = true;
                 btnRenew.Enabled = true;
                 if (listView.SelectedIndices[] != listView.Items.Count - )
                 {
                     btnMoveDown.Enabled = true;
                 }
                 if (listView.SelectedIndices[] != )
                 {
                     btnMoveUp.Enabled = true;
                 }
             }
             Samson.NoteFirst.Model.RefStyle.Journal journal = Globals.Journals[Globals.MemoryDataKey];
             setCheckBoxes(journal);
             if (!isFirst)
             {
                 PageSet();
                 CheckBoxSet();
             }
             try
             {
                 index = listView.SelectedIndices[];
             }
             catch (Exception ex)
             {
                 EventLogProcess.Write(ex);
            }
         }

代码解释下：9到30行的if   else 是控制编辑引文界面上右侧的按钮的启用和禁用。

40行是获取选中的引文的序号。就是这句报错了。调试发现，原来是listview集合为空，还要取第一项，那当然报错了。报错信息如下：

 <StackTrace>   在 System.Windows.Forms.ListView.SelectedIndexCollection.get_Item(Int32 index)
   在 NoteFirst.KMS.Clients.OfficeAddin.EditBibilography.listView_EditBibliographys_SelectedIndexChanged(Object sender, EventArgs e)</StackTrace>
  <Message>InvalidArgument=“0”的值对于“index”无效。

这段代码，第九行的判断，令人费解。listView.SelectedItems.Count!= 1，表达意图不明确。万一Count为0，面板右侧的按钮，还是启用状态，显然不合理，甚至会报错。第40行的代码，一点防卫意识都没有，就等着catch，你以为在try里面的东西，就全部安全了吗？try...catch，我认为是针对程序意外情况，你确实提前料想不到的异常处理。比如，读取文件的时候，或者打开数据库连接操作的时候，等等。c#中，经常要做的一个操作，便是判断一个对象是不是为null，或者判断参数的正确与否，如果把一些常规的东西，都包括这try块当中，这有什么意义呢？

以下是我改后的代码：

        /// <summary>
        /// 选定项发生改变事件
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void listView_EditBibliographys_SelectedIndexChanged(object sender, EventArgs e)
        {

            if (listView.SelectedItems.Count == 0 || listView.SelectedItems.Count > 1)
            {
                btnEdit.Enabled = false;
                btnDelete.Enabled = false;
                btnRenew.Enabled = false;
                btnMoveDown.Enabled = false;
                btnMoveUp.Enabled = false;
            }
            else
            {
                btnEdit.Enabled = true;
                btnDelete.Enabled = true;
                btnRenew.Enabled = true;
                if (listView.SelectedIndices[0] != listView.Items.Count - 1)
                {
                    btnMoveDown.Enabled = true;
                }
                if (listView.SelectedIndices[0] != 0)
                {
                    btnMoveUp.Enabled = true;
                }
            }

            try
            {
                if (listView.SelectedIndices.Count > 0)
                {
                    index = listView.SelectedIndices[0];
                }
            }
            catch (Exception ex)
            {
                EventLogProcess.Write(ex);
            }
        }