1

我正在尝试编写一个程序来打印（在字符串变量中）有关 mdb 数据库的以下信息：

表名 表的总列数

列列表如下：

列名：列数据类型：

为此，我使用了两种自定义类型（公共类），当然还有列表。这是我到目前为止的代码（顺便说一下，由于这里收集的问题和答案，它已经进行了不小的调整）：

以下是我创建的用于定义我正在使用的两种新类型的类：

public class ClmnInfo
{
    public string strColumnName { get; set; }
    public string strColumnType { get; set; }
}

public class TblInfo
{
    public string strTableName { get; set; }
    public int intColumnsQty { get; set; }
    public List<ClmnInfo> ColumnList { get; set; }

}

这是实际获取数据的代码。请记住，我使用 OleDB 连接到实际数据并且一切正常，除了我将在下面描述的问题。作为示例，我目前正在使用一个简单的 1 个表 db 测试此代码，其中包含 12 个字符串类型的列，保存为 1 个 int32（Access 中的 Long Int）。

//Here I declare and Initialize all relevant variables and Lists
TblInfo CurrentTableInfo = new TblInfo();
ClmnInfo CurrentColumnInfo = new ClmnInfo();
List<TblInfo> AllTablesInfo = new List<TblInfo>();

 //This loop iterates through each table obtained and imported previously in the program
int i = 0;
foreach (DataTable dt in dtImportedTables.Tables)
{


    CurrentTableInfo.strTableName = Globals.tblSchemaTable.Rows[i][2].ToString(); //Gets the name of the current table
    CurrentTableInfo.intColumnsQty = dt.Columns.Count; //Gets the total number of columns in the current table
    CurrentTableInfo.ColumnList = new List<ClmnInfo>(); //Initializes the list which will house all of the columns 

    //This loop iterates through each column in the current table
    foreach (DataColumn dc in dt.Columns)
    {
        CurrentColumnInfo.ColumnName = dc.ColumnName;  // Gets the current column name
        CurrentColumnInfo.ColumnType = dc.DataType.Name; // Gets the current column data type

        CurrentTableInfo.ColumnList.Add(CurrentColumnInfo); // adds the information just obtained as a member of the columns list contained in CurrentColumnInfo 
     }
//BAD INSTRUCTION FOLLOWS:

        AllTablesInfo.Add(CurrentTableInfo); //This SHOULD add The collection of column_names and column_types in a "master" list containing the table name, the number of columns, and the list of columns
    }

我调试了代码并观察了所有变量。它工作得很好（表名和列数量被正确注册，以及该表的 column_names、column_types 列表），但是当“坏”指令被执行时，AllTablesInfo 的内容根本不是它们应该是的. 表名是正确的，列数也是正确的，列列表甚至应该有 12 个成员，但列表的每个成员都是相同的，即我正在检查的数据库的 LAST 列。谁能向我解释为什么 CurrentTableInfo 在添加到 AllTablesInfo 列表时会以这种方式被覆盖？

4

3 回答 3

8

您正在创建一个对象 TblInfo，然后在每次迭代中更改属性。您的列表包含对同一对象的大量引用。只需移动这一行：

TblInfo CurrentTableInfo = new TblInfo();

到第一个循环的内部，这一行：

ClmnInfo CurrentColumnInfo = new ClmnInfo();

在嵌套foreach循环内，以便您在每次迭代时创建新实例。

下一个：

  • 重要
    确保您了解它之前失败的原因。如果您不确定对象和引用（以及值类型）在 C# 中的工作方式，请阅读我关于引用的文章
  • 对局部变量使用camelCased名称而不是名称CamelCased
  • 考虑使用对象初始化器ClmnInfo
  • 更改您的类型名称以避免不必要的缩写 ( TableInfo, ColumnInfo)
  • 更改您的属性名称以避免伪匈牙利符号，并使它们PascalCased
  • 考虑将整个事情重写为 LINQ 查询（相对高级）

LINQ 之前的更改将使您的代码看起来像这样：

List<TableInfo> tables = new List<TableInfo>();

int i = 0;
foreach (DataTable dt in dtImportedTables.Tables)
{
    TableInfo table = new TableInfo
    {
        Name = Globals.tblSchemaTable.Rows[i][2].ToString(),
        // Do you really need this? Won't it be the same as Columns.Count?
        ColumnCount = dt.Columns.Count,
        Columns = new List<ColumnInfo>()
    };

    foreach (DataColumn dc in dt.Columns)
    {
        table.Columns.Add(new ColumnInfo {
                            Name = dc.ColumnName,
                            Type = dc.DataType.Name
                          });
    }
    tables.Add(table); 
    // I assume you meant to include this?
    i++;
}

使用 LINQ：

List<TableInfo> tables = 
    dtImportedTables.Tables.Zip(Globals.tblSchemaTable.Rows.AsEnumerable(),
        (table, schemaRow) => new TableInfo {
            Name = schemaRow[2].ToString(),
            // Again, only if you really need it
            ColumnCount = table.Columns.Count,
            Columns = table.Columns.Select(column => new ColumnInfo {
                        Name = column.ColumnName,
                        Type = column.DataType.Name
                      }).ToList()
        }
    }).ToList();
于 2013-06-06T20:28:28.867 回答
1

您只创建了一个 TblInfo 实例。

于 2013-06-06T20:28:38.243 回答
1

这是因为您只有一个 TblInfo 实例，您在循环中不断更新它，然后将另一个引用添加到列表中。因此，您的列表对内存中的同一对象有很多引用。

在 for 循环内移动 CurrentTableInfo 实例的创建。

于 2013-06-06T20:28:57.370 回答