ObservableCollection→RichTextItemのDataBind(WPF)

Question

現在
http://kisuke0303.sakura.ne.jp/blog/?p=165
を参考に、出力Windowsを作ろうとしています。

namespace SigmoidColorFastAviUtlDownloader
{
	public partial class MainWindow : Window
	{
		private ObservableCollection<RichTextItem> OutMessage { get; set;}
		public MainWindow()
		{
			InitializeComponent();
			OutMessage = new ObservableCollection<RichTextItem>();
			OutMessage.Add(new RichTextItem {
				Text = "SigmoidColorFastAviUtlDownloader起動"
			});
		}
	}
}

ほぼ参考サイトどおり実装し、上記のようにObservableCollection<RichTextItem>型の値を定義し、

        <local:BindableRichTextBox Document="{Binding OutMessage, Converter={StaticResource RichTextItemsToDocumentConverter}}" Margin="10,75,10,10" VerticalScrollBarVisibility="Visible" IsReadOnly="True" />

のようにxamlには記述しました。

しかし、"SigmoidColorFastAviUtlDownloader起動"が表示されません。つまりうまくdatabindできていないのだとは思いますが、どうすればbindできるのかわかりません。

ソースコード全文は

https://github.com/yumetodo/SigmoidColorFastAviUtlDownloader/tree/master

にあります。

---

追記1

https://twitter.com/espresso3389/status/799111690151333888

namespace SigmoidColorFastAviUtlDownloader
{
	public partial class MainWindow : Window
	{
		private ObservableCollection<RichTextItem> OutMessage { get; } = new ObservableCollection<RichTextItem>();
		public MainWindow()
		{
			InitializeComponent();
			OutMessage.Add(new RichTextItem {
				Text = "SigmoidColorFastAviUtlDownloader起動"
			});
		}
	}
}

としてもだめでした

Comments

[espresso3389]

XAML側を抜粋ではなく、できれば全体を掲載していただけませんか？

[yumetodo]

https://github.com/yumetodo/SigmoidColorFastAviUtlDownloader/blob/master/SigmoidColorFastAviUtlDownloader/MainWindow.xamlにあります

Answer 1

結論から言うと、Bindingが動かないのは、OutMessage が private であるのが原因のようです。

public ObservableCollection<RichTextItem> OutMessage { get; } = new ObservableCollection<RichTextItem>();

一方で、ここを修正してもメッセージは出ませんが、RichTextItem の書式で、 Foreground を指定してないからですね。

Comments

[yumetodo]

なるほど、publicにすれば確かにconverterが呼ばれている！
Foreground はちょっといじってみます。

[yumetodo]

https://github.com/yumetodo/SigmoidColorFastAviUtlDownloader/commit/1f75fca04abdf665d7d51828a7d25a6c5ca4a831
MainWindowのctor内ではちゃんとconverterが呼ばれるようになったのですが、その他(click eventの関数など)からはconverterが呼ばれないのはなぜでしょう・・・

[espresso3389]

これはですね、そもそも Converter を使うのが不適切という結論ではあるのですが、
そもそも、OutMessage.Addによって、Converterが呼び出されていると勘違いされているようですが、それは本当は違います。
Converterにブレークポイントを張ってみればわかると思いますが、実は、1回しか呼び出されてないと思われます。
これは、MainWindowのコンストラクタが完了したのちに、一回だけ、BindableRichTextBox.Documentへの代入が実行され、そのタイミングで呼び出されているだけなのです。

実は、33行目のプロパティの宣言をタダのListに変えても実は動作は変わりません。

```cs
public static List<RichTextItem> OutMessage { get; } = new List<RichTextItem>();
```

参考にされている記事では、あたかも、Converterで全部解決のような書かれ方がしてありますが、実際には、ObservableCollectionとしては何にも動いてないに等しく、かつ、Converterの動作としても非常に効率が悪いと思われます。

このような場合には、ListBox と DataTemplate あるいは、 ItemTemplate と、 ObservableCollection による組み合わせで処理をするのが XAML/WPF としては順当だと思われます。

[yumetodo]

作り直したほうが適当そうなのでこれを踏まえてゆっくり作ることにします。ありがとうございます。