我的C＃ListView比较函数太慢

我想排序ListView其中也有一个DateTime列。这是我使用的代码：我的C＃ListView比较函数太慢

public bool isDate(Object obj) 
{ 
    string strDate = obj.ToString(); 
    try 
    { 
     DateTime dt = DateTime.Parse(strDate); 
     if (dt != DateTime.MinValue && dt != DateTime.MaxValue) 
      return true; 
     return false; 
    } 
    catch 
    { 
     return false; 
    } 
} 

public int Compare(object o1, object o2) 
{ 
    if (!(o1 is ListViewItem)) 
     return (0); 
    if (!(o2 is ListViewItem)) 
     return (0); 

    ListViewItem lvi1 = (ListViewItem)o2; 
    string str1 = lvi1.SubItems[ByColumn].Text; 
    ListViewItem lvi2 = (ListViewItem)o1; 
    string str2 = lvi2.SubItems[ByColumn].Text; 

    int result; 
    if (lvi1.ListView.Sorting == SortOrder.Ascending) 
    { 
     if (isDate(str1) && isDate(str2)) 
      result = DateTime.Compare(DateTime.Parse(str1), DateTime.Parse(str2)); 
     else 
      result = String.Compare(str1, str2); 
    } 
    else 
     result = String.Compare(str2, str1); 

    LastSort = ByColumn; 
    return result; 
}

的ListView持有约2000项，问题是，这是非常缓慢的。我究竟做错了什么？有任何想法吗？

在此先感谢！

编辑：非常感谢。我是新手，现在是我的代码。它快得多，我确定了我的逻辑。

public int Compare(object o1, object o2) 
{ 
    var lvi1 = o2 as ListViewItem; 
    var lvi2 = o1 as ListViewItem; 
    if (lvi1 == null || lvi2 == null) 
     return 0; 

    string str1 = lvi1.SubItems[ByColumn].Text; 
    string str2 = lvi2.SubItems[ByColumn].Text; 

    int result; 
    DateTime dateValue1 = new DateTime(); 
    DateTime dateValue2 = new DateTime(); 

    if (lvi1.ListView.Sorting == SortOrder.Ascending) 
    { 
     if (DateTime.TryParse(str1, out dateValue1) && DateTime.TryParse(str2, out dateValue2)) 
      result = DateTime.Compare(dateValue1, dateValue2); 
     else 
      result = String.Compare(str1, str2); 
    } 
    else 
    { 
     if (DateTime.TryParse(str1, out dateValue1) && DateTime.TryParse(str2, out dateValue2)) 
      result = DateTime.Compare(dateValue2, dateValue1); 
     else 
      result = String.Compare(str2, str1); 
    } 

    LastSort = ByColumn; 
    return result; 
}

来源

2012-02-24 pelz

这是使用WPF？如果是这样，你能标记它吗？ – mindandmedia 2012-02-24 23:59:38

为什么不使用'DateTime.TryParse（）'而不是捕获解析异常？它返回一个'bool'，并使用一个输出参数来设置成功的值。 – 2012-02-25 00:02:46

另外我认为你的逻辑有问题，当排序顺序是升序时，你关心的是DateTime字段，但在其他情况下（降序）你不关心这个问题。 – 2012-02-25 00:04:08

一些优化来在我的脑海：

相反而isDate使用：DateTime.TryParse
而不是做两个时间铸造：

if (!(o1 is ListViewItem)) return (0);

ListViewItem lvi1 = (ListViewItem)o2;

使用一次性铸造：

var lvi1 = o2 as ListViewItem; 
if (lvi1 == null) 
    return 0;

此外，我建议检查你的代码逻辑（正如我在评论中提及）。

来源

2012-02-25 00:10:09

谢谢。我将它改为TryParse，我想我修正了逻辑。 – pelz 2012-02-25 00:32:48

你不能只将ListViewItem.Tag设置为DateTime吗？然后比较日期时间：

DateTime d = (DateTime)ListViewItem.Tag;

来源

2012-02-25 00:10:29 user826259

我同意这个建议，这将避免从字符串到'DateTime'的所有转换，这些转换会在两次（在isDate（）内部），然后在比较'Compare（）'中的两个DateTime值时进行。如果'Tag'为null，那么你仍然可以回退到像你原来的逻辑正在做的字符串比较。 – 2012-02-25 00:31:08

我的C＃ListView比较函数太慢

回答

相关问题