Событие TextBox.TextChanged, дважды срабатывающее на эмуляторе Windows Phone 7

У меня очень простое тестовое приложение, просто чтобы поиграть с Windows Phone 7. Я только что добавил TextBox и 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... мы увидим, что произойдет.

ОТВЕТЫ

Ответ 1

Причина, по которой событие TextChanged срабатывает дважды в WP7, является побочным эффектом того, как шаблон TextBox был настроен для просмотра Metro.

Если вы редактируете шаблон TextBox в Blend, вы увидите, что он содержит вторичный TextBox для состояния с отключенным/доступным только для чтения. Это приводит к тому, что в качестве побочного эффекта событие срабатывает дважды.

Вы можете изменить шаблон, чтобы удалить дополнительные TextBox (и связанные состояния), если вам не нужны эти состояния, или изменить шаблон, чтобы добиться другого взгляда в состоянии "отключено/доступно только для чтения", без использования вторичный TextBox.

При этом событие будет срабатывать только один раз.

Ответ 2

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

Ответ 3

Это звучит как ошибка для меня. В качестве обходного пути вы всегда можете использовать 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.

Ответ 4

Ницца! Я нашел этот вопрос, ища связанную проблему, а также нашел эту неприятную вещь в своем коде. Двойное событие ест больше ресурсов ЦП в моем случае. Итак, я исправил текстовое поле фильтра в реальном времени с помощью этого решения:

private string filterText = String.Empty;

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

}

Ответ 5

Я считаю, что это всегда было ошибкой в ​​Compact Framework. Он, должно быть, перенесен в WP7.

Ответ 6

Конечно, это выглядит как ошибка для меня, если вы пытаетесь поднять событие каждый раз, когда текст изменяется, вы можете попытаться использовать двустороннюю привязку, но, к сожалению, это не приведет к изменению событий смены для каждой клавиши (только когда поле теряет фокус). Вот обходной путь, если вам это нужно:

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

Ответ 7

Отказ от ответственности - я не знаком с нюансами xaml, и я знаю, что это звучит нелогично... но в любом случае - моя первая мысль - попробовать пройти как обычные события, а не textchangedeventargs. Не имеет смысла, но может быть, это может помочь? Похоже, что когда я видел двойные звонки, как это раньше, это либо из-за ошибки, либо из-за чего-то 2 добавляются вызовы обработчика событий, происходящие за кулисами... Я не уверен, хотя?

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

Ответ 8

Я не думаю, что это ошибка. Когда вы присваиваете значение текстовому свойству внутри события textchanged, изменяется значение текстового поля, которое снова вызывает событие с измененным текстом.

попробуйте это в приложении Windows Forms, вы можете получить сообщение об ошибке

"Необработанное исключение типа" System.StackOverflowException "произошло в System.Windows.Forms.dll"

Ответ 9

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>

Ответ 10

Это старая тема, но вместо изменения шаблона (это не работает для меня, я не вижу другого текстового поля с Blend), вы можете добавить логическое значение, чтобы проверить, действительно ли событие выполнило эту функцию или нет.

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

Я знаю, что это НЕ идеальный способ, но я думаю, что это простой способ сделать это. И это работает.