VSTO,C#を用いてxmlファイルのタグの属性を動的に変更したい

Question

前提・実現したいこと

C#側の処理を用いてxmlファイルのタグの属性を動的に変更したいです。

発生している問題・エラーメッセージ

xmlファイルの読み込みまではうまくいくが、書き込みの際に例外エラーが発生し、
xmlファイルへの書き込みが反映されない状態となっています。

現在はlabel属性の動的な追加(書き換え)を目標にしているがOnActionというイベントも動的に追加したいと考えています。

書き込みの部分をどう記述すればいいかわからないので有識者の方がいましたら教えていただきたいです。
下記のソースコードと全く違う書き方になってしまうのであればソースも教えていただきたいです。

System.InvalidOperationException: 'シーケンスに要素が含まれていません'

該当のソースコード

        public void Ribbon_Load(Office.IRibbonUI ribbonUI)
        {
            this.ribbon = ribbonUI;
            //xml読み込み
            XElement xml = XElement.Load(@"C:\Users\信一郎\source\repos\ExcelAddIn2\ExcelAddIn2\Ribbon1.xml");

            //xmlファイルが正常に読み取れているかstring型にしてメッセージ表示
            string hoge = xml.ToString();
            System.Windows.Forms.MessageBox.Show(hoge);

            //例外エラーの発生元
            XElement info = (from item in xml.Elements("button")
                             where item.Attributes("label").ToString() == "test"
                             select item).Single();
　　　　　　//ここまで

            //label属性の書き換え
            info.Attribute("label").Value = "aaaaa";
            //保存
            xml.Save(@"C:\Users\信一郎\source\repos\ExcelAddIn2\ExcelAddIn2\Ribbon1.xml");



        }
        public void click_Event(Office.IRibbonControl control)
        {
            System.Windows.Forms.MessageBox.Show("ボタンが押されました");
        }

<?xml version="1.0" encoding="UTF-8"?>
<customUI xmlns="http://schemas.microsoft.com/office/2009/07/customui" onLoad="Ribbon_Load">
  <ribbon>
    <tabs>
      <tab idMso="TabAddIns">
        <group id="MyGroup"
               label="My Group">
			<button id="btniId" label="test" onAction="click_Event"></button>
        </group>
      </tab>
    </tabs>
  </ribbon>
</customUI>

試したこと

        XElement info = (from item in xml.Elements("button")
                         where item.Attributes("label").ToString() == "test"
                         select item).Single();の.SingleをSingleOrDefault()に変更した。

すると
System.NullReferenceException: 'オブジェクト参照がオブジェクト インスタンスに設定されていません。'info が null でした。
という例外エラーに変化した。
発生元のソースコードも info.Attribute("label").Value = "aaaaa";に変化した。

補足情報（FW/ツールのバージョンなど）

VisualStudio 2019
Excel 2016
.NetFramework 4.7.2

Answer 1

xmlファイルの読み込みまではうまくいくが、書き込みの際に例外エラーが発生し、
xmlファイルへの書き込みが反映されない状態となっています。

書き込み以前に、要素（XElement）の取得ができていません。

LINQは便利ですがエラーが出たときにどこで失敗しているかがわかりにくいです。
適当に勘であれこれするのではなく、分解して一つずつ確認してください。

using System;
using System.Linq;
using System.Xml.Linq;

namespace Questions352772
{
    internal class Program
    {
        private static void Main()
        {
            var xml = XElement.Load(@"Ribbon1.xml");

            // まずは分解して一個ずつ確かめていくぞー
            var a = xml.Elements("button");
            Console.WriteLine(a.Count()); // 0

            // あれー取れないなぁ？
            // そもそもElementsってなんだっけ？
            var aa = xml.Elements();
            Console.WriteLine(aa.Count()); // 1

            // １！？ どういうこと？
            // リファレンス見てみよ
            // [XElement クラス (System.Xml.Linq) | Microsoft Docs](https://docs.microsoft.com/ja-jp/dotnet/api/system.xml.linq.xelement?view=netframework-4.7.2)
            // [XContainer.Elements メソッド(System.Xml.Linq) | Microsoft Docs](https://docs.microsoft.com/ja-jp/dotnet/api/system.xml.linq.xcontainer.elements?view=netframework-4.7.2#System_Xml_Linq_XContainer_Elements_System_Xml_Linq_XName_)
            //ああそうか直接の子だけなのか...

            // 子孫を取得する方法を調べよー
            // これっぽいね
            // [XContainer.Descendants メソッド(System.Xml.Linq) | Microsoft Docs](https://docs.microsoft.com/ja-jp/dotnet/api/system.xml.linq.xcontainer.descendants?view=netframework-4.7.2#System_Xml_Linq_XContainer_Descendants_System_Xml_Linq_XName_)

            var b = xml.Descendants();
            Console.WriteLine(b.Count()); // 5

            // うんうん予想通り！
            // じゃあbutton

            var bb = xml.Descendants("button");
            Console.WriteLine(bb.Count()); // 0

            // なんでやねん！
            // ｇｇる 「xelement descendants not working」
            // [c# - XDocument.Descendants not returning descendants - Stack Overflow](https://stackoverflow.com/questions/11933782/xdocument-descendants-not-returning-descendants)
            // なるほど名前空間をつけないといけないのね（めんどくさ）

            XNamespace ns = "http://schemas.microsoft.com/office/2009/07/customui";
            var bbb = xml.Descendants(ns + "button");
            Console.WriteLine(bbb.Count()); // 1

            // よしよしここまではＯＫ！
            // じゃあlabelも

            var c = from item in xml.Descendants(ns + "button")
                    where item.Attributes("label").ToString() == "test"
                    select item;
            Console.WriteLine(c.Count()); // 0

            // だめかぁorz
            // そういえばAttributesもよくわかってないなー

            var button = xml.Descendants(ns + "button").First();
            foreach (var att in button.Attributes())
            {
                Console.WriteLine(att.ToString());
                //id = "btniId"
                //label = "test"
                //onAction = "click_Event"
            }

            // あっ そうだよねToStringじゃおかしいよね
            // もっといいのがあるはず
            // [XAttribute クラス (System.Xml.Linq) | Microsoft Docs](https://docs.microsoft.com/ja-jp/dotnet/api/system.xml.linq.xattribute?view=netframework-4.7.2)
            // [XAttribute.Value プロパティ (System.Xml.Linq) | Microsoft Docs](https://docs.microsoft.com/ja-jp/dotnet/api/system.xml.linq.xattribute.value?view=netframework-4.7.2#System_Xml_Linq_XAttribute_Value)
            // 普通にValueあったやん

            // ええと。。こうか？
            //var cc = from item in xml.Descendants(ns + "button")
            //         where item.Attributes("label").Value == "test"
            //         select item;

            // コンパイルエラー
            // そうだよな複数形（Attributes）だもんな
            // えっと。。こうかな？？
            var cc = from item in xml.Descendants(ns + "button")
                     from att in item.Attributes("label")
                     where att.Value == "test"
                     select item;
            Console.WriteLine(c.Count()); // 1

            // ふぅやっとできたっぽいぞー
            // ほんとにダイジョブかテストをしとこう


            // xmlのbuttonの行を増やす・labelの文字を変える等 あってるかのテスト


            // よーしだいじょぶそう
            // で、なにやってたんだっけ？ｗ
            // あぁそうそう
            var info = (from item in xml.Descendants(ns + "button")
                        from att in item.Attributes("label")
                        where att.Value == "test"
                        select item).Single(); // Singleは必ず１つだけ該当するという前提ならＯＫ

            info.Attribute("label").Value = "aaaaa";
            xml.Save(@"Ribbon1.xml");
        }
    }
}

LINQに不慣れならforeachで書いたほうがいいかもしれません。

using System.Xml.Linq;

namespace Questions352772
{
    internal class Program
    {
        private static void Main()
        {
            var xml = XElement.Load(@"Ribbon1.xml");
            XNamespace ns = "http://schemas.microsoft.com/office/2009/07/customui";

            foreach (var button in xml.Descendants(ns + "button"))
            {
                foreach (var att in button.Attributes("label"))
                {
                    if (att.Value == "test")
                    {
                        button.Attribute("label").Value = "aaaaa";
                        xml.Save(@"Ribbon1.xml");
                        return;
                    }
                }
            }
        }
    }
}

OnActionというイベントも動的に追加したいと考えています。

VSTOについてはなにもわかりません^^;

Comments

[退会済みユーザー]

TN8001様回答ありがとうございます。
Linqは不慣れどころか使ったことが数えるほどですね、、笑
foreachを使うことも頭に入れておきます!
OnActionについてですが
info.Attribute("label").Value = "aaaaa";
のように
info.Attribute("OnAction").Value = "イベント名";
といった感じでイベントの名前をxmlファイルに動的に追加できればいいなと思っているのですが上記の回答の流用でイベント名を追加することは可能でしょうか？

[TN8001]

> info.Attribute("OnAction").Value = "イベント名";
> といった感じでイベントの名前をxmlファイルに動的に追加できればいいなと思っているのですが上記の回答の流用でイベント名を追加することは可能でしょうか？

まあやってみればすぐわかりますが、
もしxmlが
<button id="btniId" label="test"></button>
という状態ならば
info.Attribute("OnAction").Value = "イベント名";
はエラーです。
すでに何か入っているものを変えることはできますが、無い属性の値は変えられないからです。

属性ごと追加したい場合は
info.Add(new XAttribute("onAction", "click_Event"));
です。

[退会済みユーザー]

解説ありがとうございます。
参考にさせていただきます。
ちなみに最初の回答もとにソースコードを変更して試したところ、変更されるようになったのですが、1度目の読み込み時には書き換え前のデータのままで1度閉じて再度開かないと適用されないのですが、.saveの後で.loadを用いて読み込みをすればいいのでしょうか？

[TN8001]

> 1度目の読み込み時には書き換え前のデータのままで1度閉じて再度開かないと適用されないのですが、.saveの後で.loadを用いて読み込みをすればいいのでしょうか？

えーとOfficeの話でしょうか？
VSTOは使ったことも試せる環境もないので私にはわかりません。

このままほかの回答をお待ちいただくか、
あるいはxmlの読み書きは分かったとしてこの質問は閉じ、新たに質問を立ててください。

[退会済みユーザー]

そうですね。
xmlの追加、書き込みの仕方はわかったのでVSTOはVSTOで改めて質問することにします。
今回はありがとうございました。