Windows Phone 7エミュレーターでTextBox.TextChangedイベントが2回発生する


91

Windows Phone 7で遊ぶための非常にシンプルなテストアプリがあります。標準のUIテンプレートにa TextBoxとa TextBlockを追加しました。唯一のカスタムコードは次のとおりです。

public partial class MainPage : PhoneApplicationPage
{
    public MainPage()
    {
        InitializeComponent();
    }

    private int counter = 0;

    private void TextBoxChanged(object sender, TextChangedEventArgs e)
    {
        textBlock1.Text += "Text changed " + (counter++) + "\r\n";
    }
}

TextBox.TextChangedイベントはまで配線されているTextBoxChangedXAMLで:

<TextBox Height="72" HorizontalAlignment="Left" Margin="6,37,0,0"
         Name="textBox1" Text="" VerticalAlignment="Top"
         Width="460" TextChanged="TextBoxChanged" />

ただし、エミュレーターで実行中にキーを押すたびに(画面キーボードまたは物理キーボードのいずれかで、一時停止を押して後者を有効にします)、カウンターを2回インクリメントし、に2行を表示しTextBlockます。私が試したすべては、イベントが本当に2回発生していることを示しており、その理由はわかりません。一度だけサブスクライブされていることを確認しました。MainPageコンストラクターでサブスクライブを解除した場合、テキストが変更されても(テキストブロックに対して)何も起こりません。

通常のSilverlightアプリで同等のコードを試しましたが、そこでは発生しませんでした。現在、これを再現するための物理的な電話はありません。これがWindows Phone 7の既知の問題であるという記録は見つかりませんでした。

誰かが私が間違っていることを説明できますか、それともバグとして報告する必要がありますか?

EDITは:2つのテキストコントロールを持つには、このダウンしている可能性を減らすために、私は削除しようとしたTextBlock、完全に、とまでTextBoxChanged方法を変えるだけインクリメントcounter。次に、エミュレータで実行し、10文字を入力してにブレークポイントを置きますcounter++;(単にデバッガに侵入すると、問題を引き起こしていることをあらゆる可能性を取り除くために)ライン-そしてそれは示してcounter20通り。

編集:私は今、Windows Phone 7フォーラムで質問しました ...何が起こるかを確認します。


ちょうど興味があります-イベントの内部をチェックすると、TextBoxのコンテンツはイベントが発生するたびに同じですか?私は通常、これらのことをイベント処理する代わりにMVVMとデータバインディングを使用するため(SilverlightとWPF、WP7の経験があまりないため)、なぜこれが起こるのか本当にわかりません。
ルーンジェイコブセン2010

@ルーン:はい、「後」のテキストが2回表示されます。したがって、「h」を押しtextBox1.TextてtextBlock1追加の一部として表示すると、両方の行に「h」が表示されます。
Jon Skeet、2010

1
あなたは2つのキーボードについて言及していますが、それは要因でしょうか?無効にできますか?また、TextChangedEventArgsのすべてのメンバーが両方の呼び出しで等しいかどうかを確認できますか?
Henk Holterman、2010

@ヘンク:ほとんどの場合、物理キーボードを有効にすることに悩んでいません...それが効果があるかどうかを確認するためだけです。TextChangedEventArgs実際には多くは利用できません- OriginalSource常にnullであるだけです。
Jon Skeet

3
これはバグのように見えますが、Textプロパティに新しい値を割り当てるだけで同じ結果が得られ、TextChangedが2回発生するため、キーボードとは関係ありません。
AnthonyWJones 2010

回答:


75

TextChangedイベントがWP7で2回発生する理由はTextBox、がMetroの外観用にテンプレート化されている方法の副作用です。

TextBoxBlendでテンプレートを編集するTextBoxと、無効/読み取り専用状態のセカンダリが含まれていることがわかります。これにより、副作用として、イベントが2回発生します。

TextBoxこれらの状態が必要ない場合は、テンプレートを変更して余分な(および関連する状態)を削除するか、セカンダリを使用せずにテンプレートを変更して無効/読み取り専用状態で異なる外観を実現できます。TextBoxます。

これで、イベントは1回だけ発生します。


18

私がバグに行くのは、主にKeyDownKeyUpイベントをそこに配置した場合、それらが1回だけ(それぞれ)TextBoxChanged起動されるが、イベントが2回起動されることを示しているためです


@undertakeror:そのビットをチェックしてくれてありがとう。私はWP7固有のフォーラムで同じ質問をして、その応答が何であるかを確認します...
Jon Skeet

TextInputは何をしますか?これらはWP7の単体テストをすり抜けるのは非常に大きなバグのようですが、それはSLです
Chris S

@Chris S:「何をTextInputするの?」とはどういう意味ですか?私は慣れていませんTextInput...
Jon Skeet

@Jon `OnTextInput(TextCompositionEventArgs e)`は、KeyDownの代わりにSLでテキスト入力を処理する方法です。明らかに、デバイスにキーボードがない可能性があります。「UI要素がデバイスに依存しない方法でテキストを取得すると発生します」msdn.microsoft。 com / en-us / library /…
Chris S

それが2度発火したかどうか、私はちょうど興味がありました
Chris S

8

それは私にはバグのように聞こえます。回避策として、Rxを常に使用できますDistinctUntilChanged。個別のキーを指定できるようにするオーバーロードがあります。

この拡張メソッドは、監視可能なTextChangedイベントを返しますが、連続する重複をスキップします。

public static IObservable<IEvent<TextChangedEventArgs>> GetTextChanged(
    this TextBox tb)
{
    return Observable.FromEvent<TextChangedEventArgs>(
               h => textBox1.TextChanged += h, 
               h => textBox1.TextChanged -= h
           )
           .DistinctUntilChanged(t => t.Text);
}

バグが修正されたら、そのDistinctUntilChanged行を削除できます。


2

いいね!私は関連する問題を検索することでこの質問を見つけ、コードの中にこの厄介なものも見つけました。私の場合、ダブルイベントはより多くのCPUリソースを消費します。だから、私はこのソリューションでリアルタイムフィルターのテキストボックスを修正しました:

private string filterText = String.Empty;

private void SearchBoxUpdated( object sender, TextChangedEventArgs e )
{
    if ( filterText != filterTextBox.Text )
    {
        // one call per change
        filterText = filterTextBox.Text;
        ...
    }

}

1

これは常にCompact Frameworkのバグであると思います。WP7に引き継がれているはずです。


私はそれがCFのより新しいバージョンで修正されたと思っていました... Silverlightへの移行にもかかわらず、それは奇妙になるでしょう。一方、とにかく見るのはかなり奇妙なバグです...
Jon Skeet

変だと思う。私は昨日、CF 2.0アプリケーションでそれに遭遇しました。
ジェロドホーテリング2010

0

確かに私にはバグのように見えます。テキストが変更されるたびにイベントを発生させようとしている場合は、代わりに双方向バインディングを使用できますが、残念ながらこれはキーごとのプレス変更イベントを発生させません(フィールドがフォーカスを失います)。必要な場合の回避策は次のとおりです。

        this.textBox1.TextChanged -= this.TextBoxChanged;
        textBlock1.Text += "Text changed " + (counter++) + "\r\n";
        this.textBox1.TextChanged += this.TextBoxChanged;

私はそれがそれを回避するかどうか確信がありません-問題はtextBlock1.Text変更のためにイベントハンドラが起動することではありません-私はそれを試してみます。(が試そうとした回避策は、前のテキストを覚えて、イベントハンドラーをステートフルにすることでした。実際に変更されていない場合は、無視してください:)
Jon Skeet

0

免責事項-私はxamlのニュアンスに精通しておらず、これは不合理に聞こえますが...とにかく-私の最初の考えは、textchangedeventargsではなく単純なeventargsとして渡すことを試みることです。意味がありませんが、役立つかもしれませんか?このような二重の発火を以前に見たときのようですが、それはバグが原因であるか、何らかの理由で2つの追加イベントハンドラーコールがバックグラウンドで発生しているためです...どちらかわかりませんか?

迅速かつダーティが必要な場合は、繰り返しますが、xamlの経験がないので、次のステップは、テキストボックスのxamlをスキップして、簡単な回避策として...バグを特定できるまで、テキストボックスを完全にc#で実行するか、トリッキーなコード...つまり、一時的な解決策が必要な場合。


イベント引数を渡すのは私ではなく、イベントハンドラーを実装しています。しかし、純粋にC#でイベントハンドラーを追加しても違いはないことを確認しました...それでも2回発生します。
Jon Skeet、2010

うーん。ええ、それが純粋なc#であれば、バグのように聞こえます。最初の提案について-私の言い回しが恐ろしかったので申し訳ありませんが、どのように述べるべきだったのでしょうか-[実装/ TextBoxChangedハンドラーメソッドで] argsパラメータータイプを単純なeventargsに変更してみます。たぶん動かないでしょう…でもね...…それは私の最初の考えでした。
Pimp Juice McJones、2010

言い換えると、おそらく機能しませんが、メソッドシグネチャ= private void TextBoxChanged(object sender、EventArgs e)を試してみましたが、私が試したと言うだけです=)
Pimp Juice

正しい。効果があるとは思いませんが、恐れています。
Jon Skeet

0

バグだとは思いません。textchangedイベント内のテキストプロパティに値を割り当てると、テキストボックスの値が変更され、テキスト変更イベントが再び呼び出されます。

Windowsフォームアプリケーションでこれを試すと、エラーが発生する可能性があります

「タイプ 'System.StackOverflowException'の未処理の例外がSystem.Windows.Forms.dllで発生しました」


質問から:「TextBoxとTextBlockを標準のUIテンプレートに追加しました」-それらは同じものではありません。ユーザーが入力できる1つのTextBoxと、カウントを表示する1つのTextBlockがあります。
Jon Skeet

0

StefanWickは正解です。このテンプレートの使用を検討してください

<Application.Resources>
        <ControlTemplate x:Key="PhoneDisabledTextBoxTemplate" TargetType="TextBox">
            <ContentControl x:Name="ContentElement" BorderThickness="0" HorizontalContentAlignment="Stretch" Margin="{StaticResource PhoneTextBoxInnerMargin}" Padding="{TemplateBinding Padding}" VerticalContentAlignment="Stretch"/>
        </ControlTemplate>
        <Style x:Key="TextBoxStyle1" TargetType="TextBox">
            <Setter Property="FontFamily" Value="{StaticResource PhoneFontFamilyNormal}"/>
            <Setter Property="FontSize" Value="{StaticResource PhoneFontSizeMediumLarge}"/>
            <Setter Property="Background" Value="{StaticResource PhoneTextBoxBrush}"/>
            <Setter Property="Foreground" Value="{StaticResource PhoneTextBoxForegroundBrush}"/>
            <Setter Property="BorderBrush" Value="{StaticResource PhoneTextBoxBrush}"/>
            <Setter Property="SelectionBackground" Value="{StaticResource PhoneAccentBrush}"/>
            <Setter Property="SelectionForeground" Value="{StaticResource PhoneTextBoxSelectionForegroundBrush}"/>
            <Setter Property="BorderThickness" Value="{StaticResource PhoneBorderThickness}"/>
            <Setter Property="Padding" Value="2"/>
            <Setter Property="Template">
                <Setter.Value>
                    <ControlTemplate TargetType="TextBox">
                        <Grid Background="Transparent">
                            <VisualStateManager.VisualStateGroups>
                                <VisualStateGroup x:Name="CommonStates" ec:ExtendedVisualStateManager.UseFluidLayout="True">
                                    <VisualState x:Name="Normal"/>
                                    <VisualState x:Name="MouseOver"/>
                                    <VisualState x:Name="Disabled">
                                        <Storyboard>
                                            <ObjectAnimationUsingKeyFrames Storyboard.TargetProperty="Visibility" Storyboard.TargetName="EnabledBorder">
                                                <DiscreteObjectKeyFrame KeyTime="0">
                                                    <DiscreteObjectKeyFrame.Value>
                                                        <Visibility>Collapsed</Visibility>
                                                    </DiscreteObjectKeyFrame.Value>
                                                </DiscreteObjectKeyFrame>
                                            </ObjectAnimationUsingKeyFrames>
                                        </Storyboard>
                                    </VisualState>
                                    <VisualState x:Name="ReadOnly">
                                        <Storyboard>
                                            <ObjectAnimationUsingKeyFrames Storyboard.TargetProperty="Visibility" Storyboard.TargetName="EnabledBorder">
                                                <DiscreteObjectKeyFrame KeyTime="0">
                                                    <DiscreteObjectKeyFrame.Value>
                                                        <Visibility>Collapsed</Visibility>
                                                    </DiscreteObjectKeyFrame.Value>
                                                </DiscreteObjectKeyFrame>
                                            </ObjectAnimationUsingKeyFrames>
                                        </Storyboard>
                                    </VisualState>
                                </VisualStateGroup>
                                <VisualStateGroup x:Name="FocusStates">
                                    <VisualState x:Name="Focused">
                                        <Storyboard>
                                            <ObjectAnimationUsingKeyFrames Storyboard.TargetProperty="Background" Storyboard.TargetName="EnabledBorder">
                                                <DiscreteObjectKeyFrame KeyTime="0" Value="{StaticResource PhoneTextBoxEditBackgroundBrush}"/>
                                            </ObjectAnimationUsingKeyFrames>
                                            <ObjectAnimationUsingKeyFrames Storyboard.TargetProperty="BorderBrush" Storyboard.TargetName="EnabledBorder">
                                                <DiscreteObjectKeyFrame KeyTime="0" Value="{StaticResource PhoneTextBoxEditBorderBrush}"/>
                                            </ObjectAnimationUsingKeyFrames>
                                        </Storyboard>
                                    </VisualState>
                                    <VisualState x:Name="Unfocused"/>
                                </VisualStateGroup>
                            </VisualStateManager.VisualStateGroups>
                            <VisualStateManager.CustomVisualStateManager>
                                <ec:ExtendedVisualStateManager/>
                            </VisualStateManager.CustomVisualStateManager>
                            <Border x:Name="EnabledBorder" BorderBrush="{TemplateBinding BorderBrush}" BorderThickness="{TemplateBinding BorderThickness}" Background="{TemplateBinding Background}" Margin="{StaticResource PhoneTouchTargetOverhang}">
                                <ContentControl x:Name="ContentElement" BorderThickness="0" HorizontalContentAlignment="Stretch" Margin="{StaticResource PhoneTextBoxInnerMargin}" Padding="{TemplateBinding Padding}" VerticalContentAlignment="Stretch"/>
                            </Border>
                        </Grid>
                    </ControlTemplate>
                </Setter.Value>
            </Setter>
        </Style>
    </Application.Resources>

0

それは古いトピックですが、テンプレートを変更する代わりに(私にとっては機能しません。Blendを使用した他のテキストボックスは表示されません)、ブール値を追加して、イベントがすでに機能しているかどうかを確認できます。

boolean already = false;
private void Tweet_SizeChanged(object sender, EventArgs e)
{
    if (!already)
    {
        already = true;
        ...
    }
    else
    {
    already = false;
    }
}

それは完璧な方法ではないことは承知していますが、それはそれを行う簡単な方法だと思います。そしてそれは機能します。

弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.