LINQ to XMLで同じ属性値の読込みについて

質問

下記の様なXMLファイルがあるとします。
又、言語はC＃を使用しています。

<?xml version="1.0" encoding="utf-8"?>
<staff_list>
<staff id="00001">
<value type="name">ああ</value>
<value type="kana">いい</value>
<value type="address">鳥取</value>
<value type="phone">090-1111-2222</value>
<value type="shikaku">応用</value>
<value type="shikaku">基本</value>
<value type="shikaku">オラクル</value>
<value type="shikaku">高度</value>
</staff>
<staff id="00002">
<value type="name">ええ</value>
<value type="kana">おお</value>
<value type="address">大分</value>
<value type="phone">090-1111-2222</value>
<value type="shikaku">応用</value>
<value type="shikaku">基本</value>
</staff>
<./staff_list>

下記ソースの様にLINQ to XMLで"shikaku"の部分の文字列を取得しようと考えているのですが
　var shikakus = (from v in xConnectionString.Elements("value") where v.Attribute("type").Value == "shikaku" select v);
foreach (var shikaku in shikakus)
{
MessageBox.Show((string)shikaku.Element("shikaku"));
}
空白が表示され上手く取得出来ません。

どの様に取得したらよろしいのでしょうか。
っというか、すべての文字列を取得する方法も教えて頂くと有り難いです。

回答No.1

noname#251971 · ベストアンサー

「"shikaku"の部分の文字列」が何を指しているか質問文からははっきりしませんが(「部分」てどこ？)、おそらくその属性値を持つ要素の値の「応用」や「基本」などですよね？
であれば、問題点はMessageBox.Show()の引数部分にあたる所かと思います。

ループ内の変数「shikaku」はvalue要素自体を表すオブジェクトなので、「(string)shikaku.Element("shikaku")」としてしまうとvalueのさらに子の「shikaku」要素値を取得しようとしてしまいます。
# 「<value type="shikaku"><shikaku>XXXXX</shikaku></value>」のようなXML構造から取得する場合の記述になっている。
value要素自体の値を取得したいのだと思うので、単に「(string)shikaku」が正しいかと思います。

質問文末尾の「すべての文字列を取得する方法」というのはちょっと意味がわかりませんのでスキップします。
# 「<?xml ....</staff_list>」までの全ファイル内容をstringに読み込みたいならば、 File.ReadAllText() が利用出来ますが。

ついでに余計なことかもしれませんが、「クエリ式」方式でLINQを利用されているようですが、当方の個人的な意見としては「メソッドチェーン」方式でLINQを使う方が書きやすく、読みやすく、保守しやすいように思います。
たとえば、(あまり実用上意味はないかもしれませんが)全staffのshikakuを重複するものをまとめて一覧化する、という場合は以下のような。

var xmlDoc = XDocument.Load(@"path/to/file.xml");
var allShikakus = xmlDoc.Root
.Elements("staff")
.Elements("value")
.Where(elem => (string)elem.Attribute("type") == "shikaku")
.Select(elem => (string)elem)
.Distinct();

もうひとつ細かい点ですが、Attributeを参照する際には、.Valueプロパティを使うともしも属性が無い要素があるとNullReferenceExceptionになってしまうので、上記のようにキャストして属性値取得するほうが便利かと思います。