Question

我有两个图形对象（比如某种Table s），我想设置它们的样式。简单的代码如下：

table1.BorderWidth = 2;
table1.BorderColor = Color.GloriousPink;

table2.BorderWidth = 2;
table2.BorderColor = Color.GloriousPink;

（真正的代码有更多行。）

更聪明的方法是使用方法。

void Format Table(int tableIndex)
{
    Table table;
    if(tableIndex == 1)
        table = table1;
    if(tableIndex == 2)
        table = table2;
    table.BorderWidth = 2;
    table.BorderColor = Color.GloriousPink;
}

我在想一种让它更具可扩展性的方法（if / switch部分增长很快），我想出了：

foreach(Table table in new List<Table> { table1, table2 })
{
    table.BorderWidth = 2;
    table.BorderColor = Color.GloriousPink;
}

这个更短，并且可以非常简单地添加任何可能的附加表。它有什么缺点吗？

Answer 1

没有什么事情是错误的，但是我会按照你原来的想法去实际把它放在一个方法中，而只是传入实际的表格。

public void Format(Table table)
{
    table.BorderWidth = 2;
    table.BorderColor = Color.GloriousPink;
}

foreach(Table table in tables) 
{
   Format(table);
}

Answer 2

我不知道你的要求，但是一些功能风格如何：

Action<Table> formatTable = (table) => {
    table.BorderWidth = 2;
    table.BorderColor = Color.GloriousPink;
};

new List<Table> { table1, table2 }.ForEach(formatTable);

如果您不喜欢所有这些Action事物：

void FormatTable(Table table)
{
    table.BorderWidth = 2;
    table.BorderColor = Color.GloriousPink;
}

new List<Table>{ table1, table2 }.ForEach(FormatTable);

Answer 3

让编译器创建数组：

void FormatTable(params Table[] tables)
{
    foreach(var table in tables)
    {
        table.BorderWidth = 2;
        table.BorderColor = Color.GloriousPink;
    }
}

并称之为：

FormatTables( table1, table2);

对同一类型的多个对象执行大量操作

3 个答案: