Foreach 无法正常工作

Question

我正在尝试编写一个从 Unity .config 文件中获取一些元素的方法。每个元素都有 type 和 mapTo 属性，我正在获取它们的字符串值，获取我需要的子字符串并将它们放在两个单独的列表中。之后我想在一些数据网格中写入列表的内容。

问题在于，在完成所有 foreach 循环后，代码会返回到用于检查单个寄存器的代码中，并再次进入其他两个以获取类型和 mapTo 值。 换句话说，在列表中，我得到了无数的字符串值，而不是只得到一次。 我是一个初学者，我已经尝试了很多东西，但没有任何接缝可以完成工作。 有人知道我做错了什么吗？ C#中的方法代码如下：

private void ReadAdvancedConfigFile(string path)
        {
            XElement root = null;
            root = XElement.Load(new XmlTextReader(path));

            if (root != null)
            {
                XNamespace ns = "http://schemas.microsoft.com/practices/2010/unity";
                var registers = root.Element(ns + "unity").Element(ns + "container").Descendants(ns + "register");

                if (registers.Count() > 0)
                {
                    var tipList = registers.Select(x => x.Attribute("type").Value);
                    var mapToList = registers.Select(x => x.Attribute("mapTo").Value);
                    List<string> listresult = new List<string>();
                    List<string> listresultm = new List<string>();

                    foreach (var reg in registers)
                    {
                        foreach (var tpl in tipList)
                        {
                            var end = tpl.IndexOf(',');
                            var start = tpl.LastIndexOf('.', (end == -1 ? tpl.Length - 1 : end)) + 1;
                            var result = tpl.Substring(start, (end == -1 ? tpl.Length : end) - start);
                            listresult.Add(result);
                        }
                        foreach (var mpl in mapToList)
                        {
                            var endm = mpl.IndexOf(',');
                            var startm = mpl.LastIndexOf('.', (endm == -1 ? mpl.Length - 1 : endm)) + 1;
                            var resultm = mpl.Substring(startm, (endm == -1 ? mpl.Length : endm) - startm);
                            listresultm.Add(resultm);
                        }

                        int maxLenList = Math.Max(listresult.Count, listresultm.Count);
                        for (int i = 0; i < maxLenList; i++)
                        {
                            if (i < listresult.Count && i < listresultm.Count)
                            {
                                _obsCollection.Add(new Tuple<string, string>(listresult[i], listresultm[i]));
                            }
                            else if (i >= listresult.Count)
                            {
                                _obsCollection.Add(new Tuple<string, string>(string.Empty, listresultm[i]));
                            }
                            else if (i >= listresultm.Count)
                            {
                                _obsCollection.Add(new Tuple<string, string>(listresultm[i], string.Empty));
                            }
                        }
                    }
                    tabela.ItemsSource = _obsCollection;
                }
            }
        }  

方法从名为 Load 的按钮调用，该按钮从文件系统中的某个位置找到 Unity.config 文件，如下所示：

 private void button1_Click(object sender, RoutedEventArgs e)
        {
            OpenFileDialog fDialog = new OpenFileDialog();

            fDialog.Title = "Open XML file";
            fDialog.Filter = "XML files|*.config";
            fDialog.InitialDirectory = @"C:\";

            bool? control = fDialog.ShowDialog();
            if (control.Value)
            {
                var filePath = fDialog.FileName;
                ReadAdvancedConfigFile(filePath);
            }

        }

在 Unity.config 中是这种格式的 XML 文件（我删除了大部分元素，为了这里的一些空间，它也可以使用）：

<?xml version="1.0" encoding="utf-8" ?>
<configuration>

    <configSections>
        <section name="unity" type="Microsoft.Practices.Unity.Configuration.UnityConfigurationSection, Microsoft.Practices.Unity.Configuration"/>
    </configSections>

    <unity xmlns="http://schemas.microsoft.com/practices/2010/unity">

        <container name="container">

            <register name="configService" type="Web.Common.Interfaces.IConfigService, Web.Common"
                      mapTo="Web.Common.Services.ConfigServiceImpl, Web.Common">
                <lifetime type="singleton" />
                <constructor>
                    <param name="res" value="Resources.ClientStrings"> </param>
                    <param name="configFile" value="webclient.config"> </param>
                </constructor>
                <!--<property name="LocalisationService" dependencyName="LocalisationService" />-->
                <!--This is a property injection from the language plugin -->
            </register>

            <register name="scaleCoefConfigService" type="Web.WebClient.Services.IScaleCoefConfigService, Web.WebClient.TDMSWebApp"
                      mapTo="Web.WebClient.Services.Implementations.ScaleCoefConfigServiceImpl, Web.WebClient.TDMSWebApp">
                <lifetime type="singleton" />
                <constructor>
                    <param name="configService">
                        <dependency name="configService"/>
                    </param>
                </constructor>
            </register>

            <register name="sessionService" type="Web.Common.Interfaces.ISessionService, Web.Common" 
                      mapTo="Web.Common.Services.SessionServiceImpl, Web.Common">
                <lifetime type="singleton" />
            </register>

            <register name="licenseManagerService" type="Web.Common.Interfaces.ILicenseManagementService, Web.Common"
                      mapTo="Web.Common.Services.LicenseManagementServiceImpl, Web.Common">
                <lifetime type="singleton" />
            </register>
        </container>
    </unity>
</configuration>

Answer 1

在我看来，好像您不需要 foreach 循环。当您使用 LINQ 进行查询时，您已经获取了所有 'type' 和 'mapTo' 属性：

var tipList = registers.Select(x => x.Attribute("type").Value);
var mapToList = registers.Select(x => x.Attribute("mapTo").Value);

这有效地为您提供了“寄存器”中 xelements 的所有属性。 您甚至没有在循环中使用 var 'reg'...

Answer 2

我调整了你的代码，现在它可以工作了（删除foreach）：

XElement root = null;
root = XElement.Load(new XmlTextReader(path));

if (root != null)
{
    XNamespace ns = "http://schemas.microsoft.com/practices/2010/unity";
    var registers = root.Element(ns + "unity").Element(ns + "container").Descendants(ns + "register");

    if (registers.Count() > 0)
    {
        var tipList = registers.Select(x => x.Attribute("type").Value);
        var mapToList = registers.Select(x => x.Attribute("mapTo").Value);
        List<string> listresult = new List<string>();
        List<string> listresultm = new List<string>();

        foreach (string tpl in tipList)
        {
            int end = tpl.IndexOf(',');
            int start = tpl.LastIndexOf('.', (end == -1 ? tpl.Length - 1 : end)) + 1;
            string result = tpl.Substring(start, (end == -1 ? tpl.Length : end) - start);
            listresult.Add(result);
        }

        foreach (string mpl in mapToList)
        {
            int endm = mpl.IndexOf(',');
            int startm = mpl.LastIndexOf('.', (endm == -1 ? mpl.Length - 1 : endm)) + 1;
            string resultm = mpl.Substring(startm, (endm == -1 ? mpl.Length : endm) - startm);
            listresultm.Add(resultm);
        }

        int maxLenList = Math.Max(listresult.Count, listresultm.Count);

        for (int i = 0; i < maxLenList; i++)
        {
            if (i < listresult.Count && i < listresultm.Count)
            {
                _obsCollection.Add(new Tuple<string, string>(listresult[i], listresultm[i]));
            }
            else if (i >= listresult.Count)
            {
                _obsCollection.Add(new Tuple<string, string>(string.Empty, listresultm[i]));
            }
            else if (i >= listresultm.Count)
            {
                _obsCollection.Add(new Tuple<string, string>(listresultm[i], string.Empty));
            }
        }
    }
}