refactor: 重构代码

Author: ArgoZhang

src/BootstrapBlazor/Components/Table/Table.razor

@@ -107,7 +107,7 @@
                                 <span class="caret"></span>
                             </button>
                             <div class="dropdown-menu dropdown-menu-end shadow">
-                                @foreach (var item in VisibleColumns)
+                                @foreach (var item in _visibleColumns)
                                 {
                                     <div class="dropdown-item">
                                         <Checkbox ShowAfterLabel="true" DisplayText="@item.DisplayName" IsDisabled="@GetColumnsListState(item)"

src/BootstrapBlazor/Components/Table/Table.razor.Toolbar.cs

@@ -466,9 +466,7 @@ public Func<TItem, bool>? ShowDeleteButtonCallback
     /// <summary>
     /// 获得/设置 各列是否显示状态集合
     /// </summary>
-    private List<ColumnVisibleItem> VisibleColumns { get; } = [];
-
-    private HashSet<string> _visibleColumnNames = [];
+    private readonly List<ColumnVisibleItem> _visibleColumns = [];
 
     /// <summary>
     /// 获得当前可见列集合
@@ -477,10 +475,11 @@ public Func<TItem, bool>? ShowDeleteButtonCallback
     public IEnumerable<ITableColumn> GetVisibleColumns()
     {
         // 不可见列
-        return Columns.Where(i => !i.GetIgnore() && _visibleColumnNames.Contains(i.GetFieldName()) && ScreenSize >= i.ShownWithBreakPoint);
+        var items = _visibleColumns.Where(i => i.Visible).ToList();
+        return Columns.Where(i => !i.GetIgnore() && items.Find(v => v.Name == i.GetFieldName()) != null && ScreenSize >= i.ShownWithBreakPoint);
     }
 
-    private bool GetColumnsListState(ColumnVisibleItem item) => VisibleColumns.Find(i => i.Name == item.Name) is { Visible: true } && VisibleColumns.Where(i => i.Visible).DistinctBy(i => i.Name).Count(i => i.Visible) == 1;
+    private bool GetColumnsListState(ColumnVisibleItem item) => _visibleColumns.Find(i => i.Name == item.Name) is { Visible: true } && _visibleColumns.Where(i => i.Visible).DistinctBy(i => i.Name).Count(i => i.Visible) == 1;
 
     private bool ShowAddForm { get; set; }
 

src/BootstrapBlazor/Components/Table/Table.razor.cs

@@ -1276,9 +1276,9 @@ private void InternalResetVisibleColumns(List<ITableColumn> columns, IEnumerable
                 }
             }
         }
-        VisibleColumns.Clear();
-        VisibleColumns.AddRange(cols);
-        _visibleColumnNames = [.. VisibleColumns.Where(i => i.Visible).Select(a => a.Name)];
+        _visibleColumns.Clear();
+        _visibleColumns.AddRange(cols);
+        _visibleColumnNames = [.. _visibleColumns.Where(i => i.Visible).Select(a => a.Name)];
     }
 
     /// <summary>