TextBox.TextChanged двічі запускається подія на емуляторі Windows Phone 7

У мене є дуже проста програма для тестування, щоб просто пограти з Windows Phone 7. Я щойно додав 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Подія підключається до TextBoxChangedв XAML:

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

Однак кожного разу, коли я натискаю клавішу під час запуску в емуляторі (або на екранній клавіатурі, або на фізичній, натиснувши кнопку Пауза, щоб увімкнути останню), він збільшує лічильник двічі, відображаючи два рядки в TextBlock. Все, що я пробував, свідчить про те, що подія справді двічі запускається, і я не уявляю, чому. Я перевірив, що він підписується лише один раз - якщо я скасую підписку в MainPageконструкторі, при зміні тексту взагалі нічого не відбувається (з текстовим блоком).

Я спробував еквівалентний код у звичайній програмі Silverlight, і він там не відбувся. На даний момент у мене немає фізичного телефону для відтворення цього. Я не знайшов жодного запису про те, що це відома проблема в Windows Phone 7.

Хто-небудь може пояснити, що я роблю неправильно, чи я повинен повідомити про це як про помилку?

РЕДАГУВАТИ: Щоб зменшити можливість цього бути лише двома елементами керування текстом, я спробував видалити TextBlockповністю та змінити метод TextBoxChanged на просто збільшення counter. Потім я запустив емулятор, набрав 10 букв, а потім поставив точку зупинки на counter++;рядок (лише для того, щоб позбутися будь-якої можливості того, що проникнення в налагоджувач викликає проблеми) - і це відображається counterяк 20.

EDIT: Зараз я запитав на форумі Windows Phone 7 ... ми побачимо, що станеться.

Причиною того, що TextChangedподія спрацьовує двічі в WP7, є побічний ефект від того TextBox, як шаблон був оформлений для вигляду метро.

Якщо ви відредагуєте TextBoxшаблон у Blend, ви побачите, що він містить вторинну інформацію TextBoxдля стану відключеного / лише для читання. Це спричиняє побічний ефект, що подія запускається двічі.

Ви можете змінити шаблон, щоб видалити зайві TextBox(і пов’язані з ними стани), якщо ці стани вам не потрібні, або змінити шаблон, щоб досягти іншого вигляду у стані відключеного / лише для читання, без використання вторинного TextBox.

При цьому подія запускатиметься лише один раз.

я б пішов на помилку, головним чином тому, що якщо ви помістите туди події KeyDownта KeyUpподії, це показує, що вони запускаються лише один раз (кожен з них), але TextBoxChangedподія запускається двічі

Це звучить як помилка для мене. Як обхідне рішення ви завжди можете використовувати 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рядок.

Приємно! Я знайшов це питання, шукаючи пов’язану проблему, а також знайшов цю надокучливу річ у своєму коді. У моєму випадку подвійна подія з’їдає більше ресурсів процесора. Отже, я зафіксував своє текстове поле фільтру в режимі реального часу таким рішенням:

private string filterText = String.Empty;

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

}

Я вважаю, що це завжди було помилкою в Compact Framework. Це, мабуть, було перенесено в WP7.

Звичайно, це здається мені помилкою, якщо ви намагаєтеся викликати подію кожного разу, коли текст змінюється, ви можете спробувати скористатися двостороннім прив'язуванням, на жаль, це не призведе до подій зміни натискання на клавішу (лише коли поле втрачає фокус). Ось обхідний спосіб, якщо він вам потрібен:

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

Застереження - я не знайомий з нюансами xaml, і я знаю, що це звучить нелогічно ... але в будь-якому випадку - моя перша думка полягає в тому, щоб спробувати пройти як звичайні парні цілі, а не як текстові шрифти. Не має сенсу, але може бути, це може допомогти? Здається, коли я вже бачив подібні подвійні стрільби, що це або через помилку, або через те, що якимось чином додано 2 виклики обробника подій, що відбуваються за лаштунками ... Я не впевнений, які?

Якщо вам потрібно швидко і брудно, знову ж таки, я не маю досвіду з xaml - наступним кроком буде просто пропустити xaml для цього текстового поля як швидкий обхідний шлях ... зробіть це текстове поле повністю на c #, поки не зможете визначити помилку або хитрий код ... тобто якщо вам потрібно тимчасове рішення.

Я не думаю, що це помилка .. Коли ви призначаєте значення властивості тексту всередині події, зміненої текстом, значення текстового поля змінюється, що знову викликатиме зміну тексту.

спробуйте це в додатку Windows Forms, можливо, ви отримаєте помилку

"У System.Windows.Forms.dll стався необроблений виняток типу 'System.StackOverflowException'"

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>

Це стара тема, але замість зміни шаблону (це не працює у мене, я не бачу іншого текстового поля з Blend) ви можете додати логічне значення, щоб перевірити, чи подія вже виконувала цю функцію чи ні.

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

Я усвідомлюю, що це НЕ ідеальний спосіб, але я думаю, що це простий спосіб це зробити. І це працює.