C# 高效的一对多SQL查询

我正在编写一个应用程序，它使用数据库存储应用程序及其所依赖的系统。我正在使用Microsoft Access数据库存储我的信息。我的应用程序是在VS2013中用C编写的，使用OleDbCommand和OleDbDataReader与数据库交互

当我在数据库中存储一个应用程序时，它可以拥有它所依赖的任意数量的系统。为了增加“乐趣”，每个系统还可以有任意数量的屏幕，每个屏幕可以或不能由使用该系统的应用程序使用，每个屏幕可以有任意数量的字段。我使用一对多表跟踪这两种情况，如下面的模拟模式所示：

AppTable
| ID | Name | 

SystemTable
| ID | Name |

AppToSystemTable
| AppID | SystemID |

ScreenTable
| ID | NAME | SystemID | 

AppToScreenTable
| AppID | ScreenID |

FieldTable
| ID | Name | ScreenID |

AppToFieldTable
| AppID | FieldID |

现在，我可以成功读取应用程序的所有数据，但效率非常低。首先，我查询AppTable以获取只有一个值的属性，然后将它们存储到我的App对象中。然后我查询我的一对多表以获得我需要的ID列表。然后查询SystemTable以构建应用程序的系统列表。我查询ScreenTable和FieldTable，并构建屏幕和字段的字典，以跟踪哪个系统/屏幕是其父系统/屏幕。总而言之，我使用7个查询来获取我的系统列表！这很糟糕，我知道

我想问的是，如何重新构造查询以更有效地执行此操作？我很想只回答一个问题，但我不知道这是否可行。我已经研究了连接，我可以看到它们可以帮助立即获取系统、屏幕和字段，而无需抓取ID的中间步骤，但我不知道如何更有效地查询每个应用程序X系统、Y屏幕和Z字段。有人能提出有效查询一对多关系的方法吗

编辑

根据要求，这里是我目前的问题。我知道它们不是以典型的C方式构建的，但这是因为这是我在C中的第一个项目，我一直在学习

*注

实际的应用程序对象比示例架构中显示的更复杂。这被简化了，以隔离我的问题中一对多的部分，我对此感到困惑

还要注意，对GetLanguagelangId、getdeveloperoridevid或其他属性的调用都调用一个简单的方法来从数据库中读取该属性。其中许多是自定义对象。我不担心如何清理这些。我从我已经得到的答案和其他来源中得到了答案。我只是还没有重写。这正在发生

internal static App GetApp(int id)
    {
        App app = new App("");
        int langId = -1, origDevId = -1, mainDevId = -1, updateId = -1;
        List<SystemObj> systemList = new List<SystemObj>();

        bool wasOpen = true;
        try
        {
            // Get app core info.
            StringBuilder appSql = new StringBuilder();
            appSql.Append("SELECT " + DbContract.Apps.NAME + "," + DbContract.Apps.ORIGINAL_DEV_ID + 
                "," + DbContract.Apps.LANGUAGE_ID + "," + DbContract.Apps.MAINTENANCE_DEV_ID + "," + 
                DbContract.Apps.DEACTIVATION_DATE + "," + DbContract.Apps.DEPLOYMENT_METHOD + "," + 
                DbContract.Apps.UPDATE_ID + " FROM " + DbContract.Apps.TABLE + " WHERE " + 
                DbContract.Apps.ID + "=" + id);

            if (readConn.State == System.Data.ConnectionState.Closed)
            {
                wasOpen = false;
                open(readConn);
            }
            OleDbCommand appCommand = new OleDbCommand(appSql.ToString(), readConn);
            OleDbDataReader reader = appCommand.ExecuteReader();
            reader.Read();

            int appOrd = reader.GetOrdinal(DbContract.Apps.NAME);
            int langOrd = reader.GetOrdinal(DbContract.Apps.LANGUAGE_ID);
            int origDevOrd = reader.GetOrdinal(DbContract.Apps.ORIGINAL_DEV_ID);
            int mainDevOrd = reader.GetOrdinal(DbContract.Apps.MAINTENANCE_DEV_ID);
            int updateOrd = reader.GetOrdinal(DbContract.Apps.UPDATE_ID);
            int deployOrd = reader.GetOrdinal(DbContract.Apps.DEPLOYMENT_METHOD);
            int deactiveOrd = reader.GetOrdinal(DbContract.Apps.DEACTIVATION_DATE);

            app = new App(reader.GetString(appOrd), id);
            langId = reader.GetInt32(langOrd);
            origDevId = reader.GetInt32(origDevOrd);
            mainDevId = reader.GetInt32(mainDevOrd);
            updateId = reader.GetInt32(updateOrd);
            app.DeploymentMethod = reader.GetValue(deployOrd).ToString();
            app.DeactivationDate = reader.GetValue(deactiveOrd).ToString();
            reader.Close();
            if (!wasOpen) { close(readConn); }

            // Get app Language.
            app.Language = GetLanguage(langId);
            // Get app Developers.
            app.OriginalDeveloper = GetDeveloper(origDevId);
            app.MaintainingDeveloper = GetDeveloper(mainDevId);
            // Get app Update.
            app.LastUpdate = GetUpdate(updateId);

            // Get ids for app systems, screens & fields.
            List<int> sysIds = new List<int>();
            List<int> screenIds = new List<int>();
            List<int> fieldIds = new List<int>();

            if (!wasOpen) { open(readConn); }
            string appToSysSql = "SELECT " + DbContract.Apps.Systems.SYSTEM_ID + " FROM " +
                DbContract.Apps.Systems.TABLE + " WHERE " + DbContract.Apps.Systems.APP_ID +
                "=" + id;
            OleDbCommand appToSysCommand = new OleDbCommand(appToSysSql, readConn);
            reader = appToSysCommand.ExecuteReader();
            while (reader.Read())
            {
                sysIds.Add(reader.GetInt32(0));
            }
            reader.Close();

            string appToScreenSql = "SELECT " + DbContract.Apps.Screens.SCREEN_ID + " FROM " +
                DbContract.Apps.Screens.TABLE + " WHERE " + DbContract.Apps.Screens.APP_ID +
                "=" + id;
            OleDbCommand appToScreenCommand = new OleDbCommand(appToScreenSql, readConn);
            reader = appToScreenCommand.ExecuteReader();
            while (reader.Read())
            {
                screenIds.Add(reader.GetInt32(0));
            }
            reader.Close();

            string appToFieldSql = "SELECT " + DbContract.Apps.Fields.FIELD_ID + " FROM " +
                DbContract.Apps.Fields.TABLE + " WHERE " + DbContract.Apps.Fields.APP_ID +
                "=" + id;
            OleDbCommand appToFieldCommand = new OleDbCommand(appToFieldSql, readConn);
            reader = appToFieldCommand.ExecuteReader();
            while (reader.Read())
            {
                fieldIds.Add(reader.GetInt32(0));
            }
            reader.Close();
            if (!wasOpen) { close(readConn); }

            List<SystemObj> sysList = GetSystems(sysIds);

            Dictionary<Screen, int> scrnDict = GetScreens(screenIds);
            Dictionary<Field, int> fldDict = GetFields(fieldIds);

            foreach (SystemObj sys in sysList)
            {
                if (scrnDict.Values.Contains(sys.Id))
                {
                    foreach (Screen scrn in scrnDict.Keys)
                    {
                        if (scrnDict[scrn] == sys.Id)
                        {
                            sys.AddScreen(scrn);
                            if (fldDict.Values.Contains(scrn.Id))
                            {
                                foreach (Field fld in fldDict.Keys)
                                {
                                    if (fldDict[fld] == scrn.Id)
                                    {
                                        scrn.AddField(fld);
                                    }
                                }
                            }
                        }
                    }
                }
            }
            app.SystemList = sysList;
        } catch (Exception e)
        {
            if (!wasOpen) { close(readConn); }
            LogWriter.Log(e, DateTime.Now);
        }
        return app;
    }

---

我相信你可以看到这里有很大的优化空间。我第一次只是让它工作阶段；现在我正处于get运行良好的阶段。

我可能误解了您的问题，因为这似乎相对简单-为什么不创建一个查询，作为包含所有数据的平面文件

试着这样做：

SELECT AppTable.Name AS AppName, SystemTable.Name AS SystemName, ScreenTable.NAME AS ScreenName, FieldTable.Name AS FieldName
FROM (((((AppTable INNER JOIN AppToSystemTable ON AppTable.ID = AppToSystemTable.AppID) INNER JOIN SystemTable ON AppToSystemTable.SystemID = SystemTable.ID) INNER JOIN AppToScreenTable ON AppTable.ID = AppToScreenTable.AppID) INNER JOIN ScreenTable ON (ScreenTable.SystemID = SystemTable.ID) AND (AppToScreenTable.ScreenID = ScreenTable.ID)) INNER JOIN ApptoFieldTable ON AppTable.ID = ApptoFieldTable.AppID) INNER JOIN FieldTable ON ApptoFieldTable.FieldID = FieldTable.ID;

---

我相信我在这里正确地连接了您的所有ID，并为名称字段分配了别名，这样您就可以真正知道它们属于哪个表。

您可以提供一个您认为效率低下的查询示例吗？@hatchet是对的，如果您愿意，请发布您的查询。下面的答案是你想去哪里。关系查询通过连接进行。拉取一个数据集，然后用n到无穷多个连接调用来打孔数据库，以获得更多的子集数据，这样做效率很低。深入了解连接。我感谢您的帮助，但在我的代码中实现了这一点后，我得到一个异常，即不支持连接表达式。用我的OLEDB命令。我见过有人说需要用括号括起来，但我也不太走运……尽管你的确切语法在Access数据库中不起作用，但你还是给出了我的答案。非常感谢你，你帮我省去了一个大麻烦