多行TextBox最长字符串所在行判定及实现异常排查

Hey there! Let's dig into why your multi-line TextBox longest line detection isn't giving you the expected output. I’ve seen a handful of common pitfalls that cause this—let’s break them down one by one, along with fixes.

1. Accidentally trimming whitespace (undercounting line length) #

You mentioned needing to count all space characters, but it’s easy to slip up and use Trim() when measuring line length. That cuts off leading/trailing spaces, which throws your length calculation way off.

// ❌ Wrong: Trims spaces, so actual length is undercounted
int lineLength = line.Trim().Length;

// ✅ Correct: Uses the full line length including every space
int lineLength = line.Length;

2. Mishandling line splitting #

Multi-line TextBoxes use system-specific line endings (like \r\n on Windows or \n on Unix). If you’re manually splitting the text with the wrong delimiter, you’ll end up with broken lines (or extra empty entries). Skip the manual split and use the built-in Lines property—it does the heavy lifting for you:

// ❌ Risky: Might split incorrectly depending on line endings
string[] lines = textBox1.Text.Split('\n');

// ✅ Safe: TextBox handles line endings automatically
string[] lines = textBox1.Lines;

3. Only tracking one longest line (missing ties) #

If you’re storing just a single line number, you’ll miss other lines that share the maximum length. Use a collection (like a List<int>) to keep track of all matching line numbers. Here’s a working snippet that handles ties correctly:

List<int> longestLineNumbers = new List<int>();
int maxLength = 0;
string[] lines = textBox1.Lines;

for (int i = 0; i < lines.Length; i++)
{
    int currentLength = lines[i].Length;
    
    if (currentLength > maxLength)
    {
        // Found a new longest line—reset the list
        maxLength = currentLength;
        longestLineNumbers.Clear();
        longestLineNumbers.Add(i + 1); // Line numbers are 1-based, not 0-based!
    }
    else if (currentLength == maxLength)
    {
        // Tie for longest—add this line number to the list
        longestLineNumbers.Add(i + 1);
    }
}

// Build the message box output
string message;
if (longestLineNumbers.Count == 1)
{
    message = $"MAX: {longestLineNumbers[0]}";
}
else
{
    message = $"MAX lines: {string.Join(", ", longestLineNumbers)}";
}
MessageBox.Show(message);

4. Off-by-one line number errors #

The Lines array is 0-indexed (starts at 0), but users expect line numbers to start at 1. If you’re adding i instead of i+1 to your list, you’ll show line 2 instead of line 3, etc.—that’s a super common slip-up!

5. Not handling empty lines #

If all lines in your TextBox are empty, your max length will be 0, and your message box might show nothing useful. Add a quick check to handle this case:

if (maxLength == 0)
{
    MessageBox.Show("All lines are empty!");
    return;
}

If you share your existing code, I can pinpoint exactly where it’s going wrong—but these are the top issues that usually cause unexpected results.

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