XML要素から何番目かを取得したい

C#を用いてコーディングをしているのですが、
以下のようなXMLから特定の要素が何番目に該当するのかを取得したいと考えています。

何かいい方法はありますでしょうか？
(for文でぐるぐる回して条件が一致する数をカウントするくらいしか思いつきません)

例えば以下のXMLから、Attr1=Value4の要素は、
Attr0=1の中の何番目の要素かを知りたいです。

<Element>
  <CElement Attr0="0" Attr1="Value1">
  <CElement Attr0="0" Attr1="Value2">
  <CElement Attr0="1" Attr1="Value3">
  <CElement Attr0="1" Attr1="Value4">
</Element>

現在は以下のような関数を作って対応しています。
もっと綺麗に書く方法がありそうですが。

public int GetCount(XElement Elements,string valueTemp)
{
            var elems = Elements.XPathSelectElements("//CElement[@Attr0!= '1']");
            int count = 1;
            foreach (var element in elems)
            {
                if (element.Attribute("Attr1").Value == valueTemp)
                {
                    break;
                }
                count++;
            }
            return count;
}

VisualStudio2015を使用しています。

takabosoft

2018/07/09 08:24

使用しているVisual Studioのバージョンを記入してください。また、GetCount関数の戻り値が書かれていない事や、Xelement型という不明な型が使われていますので正確なものを載せてください。

tamina

2018/07/09 08:35

質問文を修正させて頂きました。XElementは次の型として使用しています。　https://msdn.microsoft.com/ja-jp/library/system.xml.linq.xelement(v=vs.110).aspx

行動規範の内容に同意します

回答1件

ベストアンサー

十分読みやすいと思いますが、該当する要素が無かった場合が考慮されていない点や、一般的なインデックス値の始まりは0ですので、その辺りが少し気になりました。

素直に書くのでしたらこうかと思います。

csharp
1        public int GetIndex(XElement Elements, string valueTemp)
2        {
3            var elems = Elements.XPathSelectElements("//CElement[@Attr0!= '1']");
4            int index = 0;
5            foreach (var element in elems)
6            {
7                if (element.Attribute("Attr1").Value == valueTemp)
8                {
9                    return index;
10                }
11                index++;
12            }
13            return -1;
14        }

必ず要素が有るのでしたら以下のような書き方もできそうですが、綺麗かと言われると微妙です。

csharp
1        public int GetIndex2(XElement Elements, string valueTemp)
2        {
3            return Elements.XPathSelectElements("//CElement[@Attr0!= '1']").First(el => el.Attribute("Attr1").Value == valueTemp).ElementsBeforeSelf().Count();
4        }