TableLayoutPanel supprimer des lignes vides

Question

J'essaie de supprimer toutes les lignes vides d'un TableLayoutPanel. Jusqu'à présent, j'ai été capable de le faire de cette façon

private void RemoveEmptyRows() 
{ 
    for (int row = 0; row < tablePanel.RowCount - 1; row++) 
    { 
     bool hasControl = false; 
     for (int col = 0; col < tablePanel.ColumnCount; col++) 
     { 
      if (tablePanel.GetControlFromPosition(col, row) != null) 
      { 
       hasControl = true; 
       break; 
      } 
     } 
     if (!hasControl) 
      tablePanel.RowStyles.RemoveAt(row); 
    } 
} 

Y a-t-il une meilleure façon de faire? Mon approche semble trop dépassée.

Answer 1

Je ne dirais pas qu'il existe un meilleur moyen, sauf que votre code a quelques failles. Vous pouvez compter sur Linq pour le rendre un peu plus concis mais cela peut être inélégant et moins lisible et pas débogable du tout puisque vous ne devriez pas utiliser trop de Linq pour faire ce genre de chose (Personnellement, je l'aime bien!). Voici ce que vous devez faire pour votre code existant:

private void RemoveEmptyRows() 
{ 
    for (int row = tablePanel.RowCount -1; row >= 0; row--) 
    { 
     bool hasControl = false; 
     for (int col = 0; col < tablePanel.ColumnCount; col++) 
     { 
      if (tablePanel.GetControlFromPosition(col, row) != null) 
      { 
       hasControl = true; 
       break; 
      } 
     } 

     if (!hasControl) 
     { 
      tablePanel.RowStyles.RemoveAt(row); 
      tablePanel.RowCount--; 
     } 
    } 
} 

1. Vous devez itérer de haut en bas depuis RowCount peut changer lors de l'évaluation contre row dans votre boucle foreach externe. Vous devez également supprimer la ligne après avoir supprimé le RowStyle simplement en faisant RowCount--.

Voici une version Linq:

Enumerable.Range(0, tablePanel.RowCount) 
    .Except(tablePanel.Controls.OfType<Control>() 
     .Select(c => tablePanel.GetRow(c))) 
    .Reverse() 
    .ToList() 
    .ForEach(rowIndex => 
    { 
     tablePanel.RowStyles.RemoveAt(rowIndex); 
     tablePanel.RowCount--; 
    }); 

Pour décomposer à ce qu'il fait (si vous n'êtes pas familier avec LINQ):

var listOfAllRowIndices = Enumerable.Range(0, tablePanel.RowCount); 
var listOfControlsInTableLayoutPanel = tablePanel.Controls.OfType<Control>(); 
var listOfRowIndicesWithControl = listOfControlsInTableLayoutPanel.Select(c => tablePanel.GetRow(c)); 
var listOfRowIndicesWithoutControl = listOfAllRowIndices.Except(listOfRowIndicesWithControl); 
var listOfRowIndicesWithoutControlSortedInDescendingOrder = listOfRowIndicesWithoutControl.Reverse(); //or .OrderByDescending(i => i); 

Et puis:

listOfRowIndicesWithoutControlSortedInDescendingOrder.ToList().ForEach(rowIndex => 
{ 
    tablePanel.RowStyles.RemoveAt(rowIndex); 
    tablePanel.RowCount--; 
}); 

Un autre la version LINQ qui peut être plus facile à lire, mais moins efficace:

Enumerable.Range(0, tableLayoutPanel1.RowCount) 
    .Where(rowIndex => !tableLayoutPanel1.Controls.OfType<Control>() 
     .Select(c => tableLayoutPanel1.GetRow(c)) 
     .Contains(rowIndex)) 
    .Reverse() 
    .ToList() 
    .ForEach(rowIndex => 
    { 
     tablePanel.RowStyles.RemoveAt(rowIndex); 
     tablePanel.RowCount--; 
    });