Событие 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;
}
}
Я знаю, что это НЕ идеальный способ, но я думаю, что это простой способ сделать это. И это работает.