4

问题：

我正在使用 Linq.Dynamic 的修改版本根据 ajax 请求对数据表进行排序（参见下面的代码）。到目前为止，它曾经工作得很好。

但是，我有一个问题：

如果我有一个包含 NULL 值的数据表，即使只有一行中的一个字段为 NULL，我也会得到这两个异常：

Object must be of type "string"

如果几个相邻的值为 NULL：

At least one object must implement IComparable

这是我的代码

using System.Linq;
using System.Data;
using System.Linq.Dynamic;


   // Pre:
   // sidx: Sort Field
   // sord: Sort order ["ASC", "DESC"]
   // page: current page number
   // rows: pagesize in rows
   public static string TestPaging(string sidx, string sord, string page, string rows)
   {
        string strReturnValue = null;

        using (System.Data.DataTable dtAllData = GetDataTable())
        {

            IQueryable<System.Data.DataRow> iqOrderedPagedData = null;


            if (string.IsNullOrEmpty(sidx) || string.IsNullOrEmpty(sord))
                iqOrderedPagedData = dtAllData.AsEnumerable().AsQueryable();
            else
                iqOrderedPagedData = dtAllData.AsEnumerable().AsQueryable().OrderBy(sidx + " " + sord);


            Int32 iPageSize = string.IsNullOrEmpty(rows) ? 100 : System.Convert.ToInt32(rows);
            Int32 iPageIndex = System.Convert.ToInt32(page) - 1;

            iqOrderedPagedData = iqOrderedPagedData.Skip(iPageIndex * iPageSize).Take(iPageSize);


            //using (System.Data.DataTable dtOrderedPagedData = MyCopyToDataTable(iqOrderedPagedData))
            using (System.Data.DataTable dtOrderedPagedData = iqOrderedPagedData.CopyToDataTable())
            {
                cjqGrid jqGrid = new cjqGrid();
                //jqGrid.total = dtAllData.Rows.Count / iPageSize + 1;
                jqGrid.total = (int)Math.Ceiling((float)dtAllData.Rows.Count / (float)iPageSize);
                jqGrid.page = iPageIndex + 1;
                jqGrid.records = dtAllData.Rows.Count;
                jqGrid.data = dtOrderedPagedData;
                strReturnValue = null; // Serialize(jqGrid, true);
                jqGrid = null;
            } // End Using dtOrderedPagedData 

        } // End Using dtAllData

        //Response.ContentType = "application/json";
        return strReturnValue;
    }


TestPaging("USR_Domain", "desc", "1", "10");

问题似乎是扩展方法CopyToDataTable，在该行：

if (!e.MoveNext ())

这使它对表进行排序，这意味着它调用类 System.Linq.SortSequenceContext中的函数 Compare

在此行引发错误的位置

comparison = comparer.Compare(keys[first_index], keys[second_index]);

这里是来自 mono 的版本以及我的修复，这些修复使该 mono 方法真正起作用。
但是，该错误也发生在普通的旧 MS .NET 4.0 中。
（我需要单声道将我的 Linq 使用方法反向移植到 .NET 2.0）

public static DataTable CopyToDataTable<T> (this IEnumerable<T> source)
            where T : DataRow
        {
            DataTable dt = new DataTable ();
            IEnumerator<T> e = source.GetEnumerator ();
            if (!e.MoveNext ())
                throw new InvalidOperationException ("The source contains no DataRows");
            foreach (DataColumn col in e.Current.Table.Columns)
                dt.Columns.Add (new DataColumn (col.ColumnName, col.DataType, col.Expression, col.ColumnMapping));
            CopyToDataTable<T> (source, dt, LoadOption.PreserveChanges);
            return dt;
        }

        public static void CopyToDataTable<T> (this IEnumerable<T> source, DataTable table, LoadOption options)
            where T : DataRow
        {

            if (object.ReferenceEquals(typeof(T), typeof(System.Data.DataRow)))
            {

                foreach (System.Data.DataRow drRowToCopy in source)
                {
                    System.Data.DataRow drNewRow = table.NewRow();

                    for (int i = 0; i < drRowToCopy.ItemArray.Length; ++i)
                    {
                        drNewRow[i] = drRowToCopy[i];
                    } // Next i

                    table.Rows.Add(drNewRow);
                } // Next dr

            }
            else
                CopyToDataTable<T>(source, table, options, null);
        }

只要一行的一列中只有一个值为NULL，就会出现问题......

谁能建议我如何解决这个问题？
或者，当一个或两个字段为 NULL 时，我如何才能从 IEnumerable 生成 DataTable 而不会出现异常？

现在我通过在比较中捕获异常来“修复”它。
但这只是 .NET 2.0 的修复，我可以这样做，因为这个类不存在。
我需要一个真正适用于 .NET 4.0 的修复程序。

public override int Compare (int first_index, int second_index)
        {
            int comparison = 0;
            try
            {
                comparison = comparer.Compare(keys[first_index], keys[second_index]);
            }
            catch (Exception ex)
            {
                //Console.WriteLine(ex.Message);
            }

            if (comparison == 0) {
                if (child_context != null)
                    return child_context.Compare (first_index, second_index);

                comparison = direction == SortDirection.Descending
                    ? second_index - first_index
                    : first_index - second_index;
            }

            return direction == SortDirection.Descending ? -comparison : comparison;
        }

PS：对于我的 Linq.Dynamic 的 DataTable-OrderBy-Enhanced 版本，请参见此处：
http ://pastebin.com/PuqtQhfa

4

1 回答 1

1

好的，我得到了解决方案：

很明显，它与 System.DbNull 类型有关

首先，因为我的 DataTable 中只有字符串，所以我收到了错误消息：

Object must be of type "string"

但是字符串怎么可能不是字符串类型呢？当然只有当它是 DbNull 时。

如果你有两个相邻的 DbNull.Values，你当然会得到：

At least one object must implement IComparable

因为 DbNull 没有实现 IComparable。

毕竟，有趣的是，它只在排序列是具有 NULL 值的列时才会失败，但如果它是没有 NULL 值的列，它确实可以完美地工作。

由于表本身包含所有空值而与排序无关，因此 CopyToDataTable 有时不起作用是不合逻辑的，因为它每次都复制所有值而与排序无关。

唯一合乎逻辑的结论是 OrderBy 在代码中调用时不会被执行，而只有在某些方法实际使用 OrderBy 生成的数据时才会执行。

一个快速的谷歌搜索把我带到了这个 http://blogs.msdn.com/b/charlie/archive/2007/12/09/deferred-execution.aspx

其中只有前几行需要阅读以了解问题所在：

这篇文章涵盖了最重要且经常被误解的 LINQ 特性之一。了解延迟执行是 LINQ 开发人员在希望充分利用该技术的全部力量之前必须经历的一项仪式。

所以我突然意识到我刚刚通过了通行权:)

显然，缺陷是e.MoveNext()在排序时触发了在 DbNull 上失败的比较，因为如上所述的 DbNull 没有实现 IComparable。

具有讽刺意味的是，数据表也可以使用我最初不知道的 select 语句进行排序（毕竟，我希望将“order”方法称为“order”，而不是“select”......）所以我刚刚将 Linq.Dynamic 中的 OrderBy 更改为

public static IQueryable OrderBy(this IQueryable source, string ordering, params object[] values)
{
    if (source == null) throw new ArgumentNullException("source");
    if (ordering == null) throw new ArgumentNullException("ordering");

    if (object.ReferenceEquals(source.ElementType, typeof(System.Data.DataRow)))
    {
        using (DataTable dt = source.Cast<System.Data.DataRow>().CopyToDataTable())
        {
            return dt.Select("", ordering).AsQueryable();
        }

瞧，虫子不见了。
而且由于它可以单独使用 Select 过滤比使用 Linq.Dynamic 更有效（它会生成所有数据的大约 3 个副本），所以我决定完全放弃 Linq.Dynamic。
我还在用Linq take and skip，不过以后肯定会更不愿意用Linq。

延迟执行是完全危险的，因为它会导致非常难以追踪的错误。
“boooom”所需要的只是在错误位置的空值，或缺少接口（以及在这种情况下缺少检查或缺少泛型限制）......

于 2012-08-25T13:40:02.047 回答