WPFのPropertyはふたつ必要

Question

こんにちは。いつもお世話になります。

WPF + Livet + Livet Extensionで、Propertyの処理を共通化できるのでしょうか？
環境は、Window10, Visual Studio2015, WPF4.5, Livet1.3.0.0, LivetExtensions1.0.3.0)の組み合わせです。

下記のとおり、X, Yを入力する二つのtextboxがあります。
値がdoubleの数値か判定して値を入れようとしています。
XとYでPropertyの処理は共通ですが、ふたつにしないとだめですか？

Mothodならたとえば、
private bool Validation(TextBox textbox){
double v;
try{
v = double.Parse(textbox.Text);
return true;
}
catch {
return false;
}
}
とかの処理で、TextBox.TextChangedeventから呼べば、Validationはひとつのコードを使いまわせると思います。

Propertyだと、

        private double x;

        public string XTextBoxText
        {
            get
            {
                return this.x.ToString(CultureInfo.InvariantCulture);
            }

            set
            {
                if (this.XTextBoxText == value)
                {
                    return;
                }

                double v;

                try
                {
                    v = double.Parse(value);
                }
                catch
                {
                    this.RaisePropertyChanged(() => this.XTextBoxText);

                    return;
                }

                this.x = v;

                this.RaisePropertyChanged(() => this.XTextBoxText);
            }
        }

```のコードのXをYに変えてもうひとつ作らないとだめなのでしょうか？


```XAML
<Window x:Class="Livet.Window"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:i="http://schemas.microsoft.com/expression/2010/interactivity"
        xmlns:l="http://schemas.livet-mvvm.net/2011/wpf"
        Height="250" Width="525" ResizeMode="NoResize">

    <i:Interaction.Triggers>
        <i:EventTrigger EventName="ContentRendered">
            <l:LivetCallMethodAction MethodTarget="{Binding}" MethodName="Initialize"/>
        </i:EventTrigger>
        <i:EventTrigger EventName="Closed">
            <l:DataContextDisposeAction/>
        </i:EventTrigger>
    </i:Interaction.Triggers>

    <Grid Margin="10">
        <Grid.ColumnDefinitions>
            <ColumnDefinition Width="150"/>
            <ColumnDefinition Width="*"/>
        </Grid.ColumnDefinitions>
        <Label Grid.Row="0" Grid.Column="0" Content="X" HorizontalAlignment="Left" FontFamily="Meiryo"/>
        <TextBox Grid.Row="0" Grid.Column="1"  Text="{Binding XTextBoxText}" x:Name="LocationXTextBox" TextWrapping="NoWrap" FontFamily="Meiryo" FontSize="11"/>
        <Label Grid.Row="1" Grid.Column="0" Content="Y" HorizontalAlignment="Left" FontFamily="Meiryo"/>
        <TextBox Grid.Row="1" Grid.Column="1"  Text="{Binding YTextBoxText}" x:Name="LocationYTextBox" TextWrapping="NoWrap" FontFamily="Meiryo" FontSize="11"/>
    </Grid>
</Window>

Answer 1

public class HogeVM : ... //いい名前が思いつかなかった
// 何かしら継承する
{
    /*
    何かあるとは思うが省略
    */
    private double x;
    public HogeVM(double initial){/*省略*/}

    public string Text
    {
        get
        {
            return this.x.ToString(CultureInfo.InvariantCulture);
        }

        set
        {
                /*
                長いので省略
                */
        }
    }
}

のような「バリデーションして入力戻す機能付きテキストボックス」1つに対応するクラスを
元のVMから抜き出して作り、

public HogeVM XTextBox {get;} = new HogeVM(initial : 0);
public HogeVM YTextBox {get;} = new HogeVM(initial : 0);

のように元のVMにプロパティで持たせ、

<TextBox Grid.Row="0" Grid.Column="1"  Text="{Binding XTextBox.Text}" ...

とバインドしてはどうでしょうか。

---

こんな感じです。

class MainVM : ViewModel
{
    public HogeVM TextBoxX { get; set; } = new HogeVM(initial: 2);
}

class HogeVM : ViewModel
{


    private double x ;
    public HogeVM(double initial)
    {
        x = initial;
    }
    public string Text
    {
        get
        {
            return this.x.ToString();
        }

        set
        {
            Console.WriteLine("set");
            if (this.Text == value)
            {
                return;
            }

            double v;

            try
            {
                v = double.Parse(value);
            }
            catch
            {
                this.RaisePropertyChanged(() => this.Text);

                return;
            }

            this.x = v;

            this.RaisePropertyChanged(() => this.Text);
        }
    }
}

<Window ...
        >
    <Window.DataContext>
        <local:MainVM/>
    </Window.DataContext>
    <StackPanel>
        <TextBox Text = "{Binding TextBoxX.Text}"/>
    </StackPanel>
</Window>

Comments

[picko]

ありがとうございます。
試してみました。errorが二つあります。

ひとつは、
public HogeVM XTextBox { get; } = new HogeVM(initial = 0);
でinitialがないというもの。
//private static int initial = 0;//new HogeVM(initial = 0);でinitialがないので追加。
でOKでしょうか。

もうひとつは、 'HogeVM' に 'RaisePropertyChanged' の定義が含まれていないというもの。
こちらの対応方法に苦慮しております。
アドバイスほしいです。

using System;
using System.Globalization;

namespace LivetWPFApplication1.ViewModels
{
using Livet;

public class HogeVM
{
private int v;
private double x;

public string XTextBoxText
{
get
{
return this.x.ToString(CultureInfo.InvariantCulture);
}

set
{
if (this.XTextBoxText == value)
{
return;
}

double v;
if (!double.TryParse(value, out v))
{
this.RaisePropertyChanged(() => this.XTextBoxText);//エラー CS1061 'HogeVM' に 'RaisePropertyChanged' の定義が含まれておらず、型 'HogeVM' の最初の引数を受け付ける拡張メソッド 'RaisePropertyChanged' が見つかりませんでした。using ディレクティブまたはアセンブリ参照が不足していないことを確認してください。 LivetWPFApplication1 LivetWPFApplication1\LivetWPFApplication1\ViewModels\HogeVM.cs

return;
}
this.x = v;

this.RaisePropertyChanged(() => this.XTextBoxText);//上と同じエラー
}
}
}
}

[ozwk]

> RaisePropertyChanged

定義されてないんですから当然エラー出ます。
MVVMフレームワーク使っているなら当然VMの基底クラスがあるはずなので
継承してください。

> initial

名前付き引数の書き方間違えてました。
xの初期値をコンストラクタで設定したいなってことで
別に本題じゃないです。
(当然のようにstaticで定義しようとしたところに何か闇を感じました。)

[picko]

あ。なるほど。継承するのですね。これは解決しました。ありがとうございます。

闇。。ですか。
staticなしで、
private int initial=0;
エラー CS0236 フィールド初期化子は、静的でないフィールド、メソッド、またはプロパティ 'MainWindowViewModel.initial' を参照できません LivetWPFApplication
とエラーなのです。

仮にinitialをなくしてみました。
C#
public HogeVM XTextBox { get; } = new HogeVM();
public HogeVM YTextBox { get; } = new HogeVM();

xaml
<TextBox Grid.Row="0" Grid.Column="1" Text="{Binding XTextBox}" x:Name="LocationXTextBox" TextWrapping="NoWrap" FontFamily="Meiryo" FontSize="11"/>
//TwoWayまたはOneWayToSourceバインドは、型'LivetWPFApplication1.ViewModels.MainWindowViewModel'の読み取り専用プロパティ’XTextBox’では動作できません。とerror.

<TextBox Grid.Row="1" Grid.Column="1" Text="{Binding YTextBox.Text}" x:Name="LocationYTextBox" TextWrapping="NoWrap" FontFamily="Meiryo" FontSize="11"/>
//PropertyPath Cannot resolve property 'Text' in data content of type 'LivetWPFApplication1.ViewModels.HogeVM'
とerror。

[picko]

下記のとおりで起動はできますが、肝心の数値チェックをしなくなってしまいました。

C#
private static int initial = 0;
public HogeVM XTextBox { get; set; } = new HogeVM(initial = 0);//setしないと起動できない。

HogeVM.cs
using System;
using System.Globalization;

namespace LivetWPFApplication1.ViewModels
{
using Livet;

public class HogeVM : ViewModel//継承した。
{
private int v;
private double x;

public HogeVM(int v)//コンストラクタ作った。
{
this.v = v;
}
以下同じ。

[ozwk]

setterが呼ばれるタイミングは、テキストボックスからフォーカス外れたときですが、
大丈夫ですか?
私はこれに気づかず30分悩みました

[picko]

それは大丈夫です。keyboardのtabキーで飛ばしてます。

[ozwk]

xamlのバインディングパス間違えてません?
コメントのコードだとText="{Binding XTextBox}"になってますが。
Text="{Binding XTextBox.Text}"です。

[picko]

それでした。ありがとうございました。できました。

Answer 2

・文字列を double 型に変換する
・戻り値はbooleanで成功or失敗を返す
・変換した値は参照渡しで返す
のようなメソッドを作って、それを setter で呼ぶのでは駄目でしょうか？

というか、本当にただ string を double に変換するだけだったら、TryParse とかはどうでしょうか。

if (this.XTextBoxText == value) return;

double v;
if(double.TryParse(value, v)) this.x = v;

this.RaisePropertyChanged(() => this.XTextBoxText);

↑ ざくっと書いたのでコンパイル通るかわからないですが、こんな感じ？
これくらいだったらメソッド化しなくても良いような気もします。
（他にも色々チェックがあるならしても良いですが）

---

あ、TextBox の値と value が一致したら処理しないとか、
最後に RaiseProprtyChanged() するところまで共通化したいとかでしょうか。

だとすると TextBox とかプロパティ名とかも引数で渡すようなメソッドを作る感じでしょうか…？

同じような項目がすごい沢山あるならちょっと考えますが、
X, Y くらいで、処理も 文字列 → double に変換するくらいだったら、私なら setter の中にそのまま書いちゃいますかね。

まあでも setter の中でメソッドを呼んでも良いのではないでしょうか。

Comments

[picko]

なるほどねー。
setterのなかをそっくりmethodにして、XTextBoxTextとYTextBoxTextを引数で渡すのですね。

stringをdoubleに変換して、失敗したら元に戻す、のだそうなのです。
(103→103a→103みたいに）
なので、TryParseではダメかなと。

これでざっくり40行くらいなのです。
setterをmethodにしてもあまり行数が減らない。
おっしゃるとおり、、RaisePropertyChanged()までまるっと共通化したいです。だって完全に同じなんですもん。
さらにこれを、同じprojectの別のwindowでもやるらしいのです。
ざっくり4か所以上あります。いや、まてまて。数えました。42か所あります。

解決方法はWPF UserControlなのでしょうか。

[sk_3122]

- - - - - - - - - -
1) もともと「103」が入っている
2) TextBox に "103a" が入力された
3) 入力値を double にしてみて、変換できればセットする、できなければセットしない(103のまま)
- - - - - - - - - -
という要件であれば、TryParse で良さそうな気もしますけど、どうでしょうか？

＃ 元々提示されていたコードも、私のコードも「変換に失敗したらセットしない」となっています。

どちらにしても 42 箇所ですか。それはちょっとメソッド化したくなりますかねー。
引数として渡すのは
・value と入力値が一致するかチェックしたいから TextBox
・RaisePropertyChanged を呼ぶ為のプロパティ名
・value
とかですかね…

UserControl ... 今回のは ViewModel に突っ込む側の処理だから ... どうだろう
私ならメソッドですかね ...？ 何か Common ぽいやつ ...

[picko]

おっしゃるとおり、tryparseでokでした。

42か所なんですよー。
usercontrolは作ってみましたが、this.RaisePropertyChangedで頓挫しました。
methodなどを迷走中。