WPF聊天室开发问题：ListBox自动滚动与TextBox回车发送异常

Hey there! Let's tackle your two chat room issues one by one:

1. Making the ListBox Auto-Scroll to the Bottom (XAML-First Approach) #

You absolutely can handle this in XAML, and a clean, maintainable way is using behaviors from the Microsoft.Xaml.Behaviors.Wpf NuGet package. Here's how to set it up:

First, add the required namespace to your XAML root element:

xmlns:i="http://schemas.microsoft.com/xaml/behaviors"

Then modify your ListBox to include a behavior that triggers scrolling whenever new messages are added:

<ListBox ItemsSource="{Binding Messages, Mode=OneWay}" 
         HorizontalAlignment="Left" Height="494" Margin="255,126,0,0" 
         VerticalAlignment="Top" ScrollViewer.CanContentScroll="True" Width="416">
    <i:Interaction.Behaviors>
        <local:AutoScrollToBottomBehavior />
    </i:Interaction.Behaviors>
</ListBox>

Next, create the small behavior class in your project (this is minimal code-behind, but the integration is fully XAML-driven):

using Microsoft.Xaml.Behaviors;
using System.Windows.Controls;

namespace YourProjectNamespace
{
    public class AutoScrollToBottomBehavior : Behavior<ListBox>
    {
        protected override void OnAttached()
        {
            base.OnAttached();
            AssociatedObject.ItemsSourceChanged += OnItemsSourceUpdated;
        }

        protected override void OnDetaching()
        {
            AssociatedObject.ItemsSourceChanged -= OnItemsSourceUpdated;
            base.OnDetaching();
        }

        private void OnItemsSourceUpdated(object sender, ItemsControlItemsSourceChangedEventArgs e)
        {
            if (AssociatedObject.Items.Count > 0)
            {
                AssociatedObject.ScrollIntoView(AssociatedObject.Items[^1]);
            }
        }
    }
}

If you prefer not to use behaviors, you could also bind the ScrollViewer.VerticalOffset to a ViewModel property, but the behavior approach keeps your UI logic encapsulated and XAML-focused.

2. Fixing Empty Message on Enter Key Press #

Your issue here stems from two key problems:

a. Binding Update Trigger #

By default, a TextBox's TwoWay binding only updates the source property when the control loses focus. When you press Enter, the TextBox still has focus, so MsgToSend hasn't been updated yet. Fix this by adding UpdateSourceTrigger=PropertyChanged to your binding:

<TextBox Text="{Binding MsgToSend, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" 
         HorizontalAlignment="Left" Height="65" Margin="60,625,0,0" 
         TextWrapping="Wrap" VerticalAlignment="Top" Width="520" 
         FontSize="20" TextChanged="TextBox_TextChanged" KeyDown="sendbyenter"/>

b. Syntax & Logic Gaps in Code-Behind #

Your sendbyenter method has a syntax error (missing curly braces after the if condition) and lacks safeguards. Here's the corrected version:

private void sendbyenter(object sender, KeyEventArgs e)
{
    if (e.Key.Equals(Key.Enter))
    {
        // Prevent Enter from adding a new line in the TextBox
        e.Handled = true;

        string message = Observable.MsgToSend;
        if (string.IsNullOrWhiteSpace(message)) return; // Skip empty messages

        try
        {
            Observable.mychatroom.sendMessage(message);
            Observable.Messages.Add(message);
            Observable.MsgToSend = "";
        }
        catch (Exception ex)
        {
            // Add error handling here (e.g., log the issue or show a user message)
            // MessageBox.Show($"Failed to send message: {ex.Message}");
        }
    }
}

• Added curly braces around the if block (critical for valid syntax)
• Marked the Enter key press as handled to avoid unwanted new lines
• Added a check to skip empty message sends
• Added a catch block (empty try blocks are bad practice for debugging)

That should resolve both issues smoothly! Let me know if you need further tweaks.

内容的提问来源于stack exchange，提问作者Dor Birendorf