Table中的lookup是否可以支持匹配IEnumerable类型的数据

[tiansfather]

目前lookup在查找数据时，只是将数据tostring以后进行匹配，这样的话如果数据类型是IEnumerable的话就匹配不上。帮忙看下这样修改是否可以

internal static RenderFragment RenderValue<TItem>(this ITableColumn col, TItem item) => async builder =>
{
    // 1.增加了一个从IEnumerable转化为List<object>方法
    Func<IEnumerable, List<object>> GetEnumeratorDatas = (enumerableObj) =>
    {
        var enumerator = enumerableObj.GetEnumerator();
        var list = new List<object>();
        while (enumerator.MoveNext())
        {
            list.Add(enumerator.Current);
        }
        return list;
    };

    var val = col.GetItemValue(item);
    if (col.Lookup != null && val != null)
    {
        string? content = val.ToString();
        // 转化 Lookup 数据源
        if ((Nullable.GetUnderlyingType(col.PropertyType) ?? col.PropertyType).IsGenericType && val is IEnumerable v)
        {
            // 2.如果是 IEnumerable 类型，则转化为 List<object> 类型 然后根据 List<object> 类型进行 Lookup 匹配
            var enumeratorDatas = GetEnumeratorDatas(v);
            var lookupVals = col.Lookup.Where(l => enumeratorDatas.Any(v1 => l.Value.Equals(v1?.ToString(), col.LookupStringComparison)));
            if (lookupVals.Any())
            {
                content = string.Join(",", lookupVals.Select(l => l.Text));
            }
            else
            {
                content = string.Join(",", enumeratorDatas);
            }
        }
        else
        {
            var lookupVal = col.Lookup.FirstOrDefault(l => l.Value.Equals(val.ToString(), col.LookupStringComparison));
            if (lookupVal != null)
            {
                content = lookupVal.Text;
            }
        }
        builder.AddContent(10, col.RenderTooltip(content, item));
    }
    else if (val is bool v1)
    {
        builder.AddContent(20, v1.RenderSwitch());
    }
    else
    {
        string? content;
        if (col.Formatter != null)
        {
            // 格式化回调委托
            content = await col.Formatter(new TableColumnContext<TItem, object?>(item, val));
        }
        else if (!string.IsNullOrEmpty(col.FormatString))
        {
            // 格式化字符串
            content = Utility.Format(val, col.FormatString);
        }
        else if (col.PropertyType.IsDateTime())
        {
            content = Utility.Format(val, CultureInfo.CurrentUICulture.DateTimeFormat);
        }
        //else if(val is IEnumerable<object> v)
        //{
        //    content = string.Join(",", v);
        //}
        //3.这里的判断条件调整了一下,为了支持List<int>,List<string> 等等
        else if ((Nullable.GetUnderlyingType(col.PropertyType) ?? col.PropertyType).IsGenericType && val is IEnumerable v)
        {
            content = string.Join(",", GetEnumeratorDatas(v));
        }
        else
        {
            content = val?.ToString();
        }

        builder.AddContent(30, col.RenderTooltip(content, item));
    }
};

[ArgoZhang]

@tiansfather 这代码是哪里的？我记得 RenderFragment 内联是不能使用异步的

async builder 这里第一步就应该有问题了。不知道现在是不是支持了。你可以自己测试一下先

可以参考我们组件库设计的 IModelEqualityComparer 接口，用这个比对很好

[tiansfather]

这个函数是原来组件库里的呀，要不我先提个pr你帮我看看。这个文件是ITableColumnExtensions.cs中的

[ArgoZhang]

看不懂你的意思？你要改动的代码在哪里？

源代码是什么样子？

改成什么样？

尽量简单明了，不要增加沟通成本

[tiansfather]

改动在pr

[ArgoZhang]

好的。我去看看

[ArgoZhang]

@tiansfather 代码有点混乱，改动比较多。能否梳理加以解释？如果是做功能改动。建议增加 单元测试加以覆盖

[tiansfather]

没有功能改动，只是加了个判断处理，我重新提交了一下，有空再帮我看看

[ArgoZhang]

@tiansfather 感谢感谢。现在看起来容易多了。

我看到如下代码

var enumeratorDatas = GetEnumeratorDatas(v);

// 根据 List<object> 数据获取匹配数据
var lookupVals = col.Lookup.Where(l => enumeratorDatas.Any(v1 => l.Value.Equals(v1?.ToString(), col.LookupStringComparison)));

 if (lookupVals.Any())
 {
        //如果有匹配到数据，则将匹配的文本用,连接显示
       content = string.Join(",", lookupVals.Select(l => l.Text));
}
else
{
    //如果未匹配到数据，则直接将原数据用,连接显示
    content = string.Join(",", enumeratorDatas);
}

我觉得这种应该是业务逻辑了。我更倾向于弄个回调，把控制权回调给调用者。组件内部保持一个简单逻辑即可

Lookup 设计上我的本意就是一对一的一个简单翻译，你这个应该是多对多的复杂逻辑了。我并不想在 Lookup 上做这种扩展支持，收益比较小（代码多，使用率少）

[tiansfather]

我看到原来的代码中在没有Lookup的时候，如果是IEnumerable类型，就用string.join连接，所以我想这种类型的数据在有Lookup的时候是否也应该有个默认显示，比如角色是一个Lookup,那么一个用户有多个角色，就可以直接以“角色1，角色2”这样展示了，不然的话是不是就需要用Template来处理？

另外第二个改动是这里：
原来有这样的代码
else if (val is IEnumerable<object> v) { content = string.Join(",", v); }
这种是不是不支持像List<int>,List<string>这样的，这里应该要支持IEnumerable<T>，这块的代码改动是否可行？
var isEnumerableValue = (Nullable.GetUnderlyingType(col.PropertyType) ?? col.PropertyType).IsGenericType && val is IEnumerable; ...... else if (isEnumerableValue && val is IEnumerable v) { content = string.Join(",", GetEnumeratorDatas(v)); }

[ArgoZhang]

@tiansfather 感谢感谢，我还是更倾向这里给一个回调出去，让用户自己去处理这个逻辑

1. 这里简单的逗号拼接是为了支持导航属性是集合
   这种情况应该是一个回调让用户实现自己的业务逻辑，比如是用户 Id，显示可能是 显示名称（显示ID）简单的逗号分隔是不足的。这里是我偷懒一直没有弄

2. 以你 PR 的代码为例这里的逻辑是又一次循环。这样会导致 n * m 复杂度，等于组件自己就把效率拉到最低了。所以我一直说这里应该回调出去，只要显示结果即可。这里的逻辑不应该走 Lookup

大佬的思路是对的。我赞同。但是我的意见是不能从 Lookup 这里入手解决这个问题，需要增加一个回调处理这个情况。我再跟管理员们商量一下

[tiansfather]

好的，大佬辛苦