Highlighting in a RichTextBox is taking too long

Question

I have a large list of offsets which I need to highlight in my RichTextBox. However this process is taking too long. I am using the following code:

foreach (int offset in offsets)
{
    richTextBox.Select(offset, searchString.Length);
    richTextBox.SelectionBackColor = Color.Yellow;
}

Is there a more efficient way to do so?

UPDATE:

Tried using this method but it doesn't highlight anything:

richTextBox.SelectionBackColor = Color.Yellow;
foreach (int offset in offsets)
{
    richTextBox.Select(offset, searchString.Length);
}

Sorry, I did not realize the question was about winforms. I'll delete my answer to make your question look unanswered again. — Sergey Kalinichenko, May 21 '12 at 21:06
possible duplicate of [RichTextBox syntax highlighting in real time--Disabling the repaint](http://stackoverflow.com/questions/3282384/richtextbox-syntax-highlighting-in-real-time-disabling-the-repaint) — Hans Passant, May 22 '12 at 02:22
@HansPassant I tried that method before posting this question, but it actually takes more time when using it. — duncanportelli, May 22 '12 at 19:31

score 1 · Answer 1 · answered May 30 '12 at 21:47

1

I've googled your issue and I found that RichTextBox is getting very slow when having many lines.
In my opinion, you have either buy a third part control which you can be satisfied by its performance or you may need threads to devide the whole selection task. I think they can accelerate things up.
Hope it helps !

answered May 30 '12 at 21:47

HichemSeeSharp

3,240
2
22
44

I guess this [control](http://www.telerik.com/products/winforms/richtextbox.aspx) would solve my problem – duncanportelli Jun 02 '12 at 12:50
I heard lot of good things about telerik and devexpress controls. so don't hesitate to expose your issue to them and see what solution can they offer you. – HichemSeeSharp Jun 02 '12 at 12:56
If you end up with one of the solutions mentioned above, please mark it as an answer. – HichemSeeSharp Jun 02 '12 at 13:11

score 1 · Answer 2 · answered May 31 '12 at 17:38

I've had this same problem before. I ended up disregarding all of the methods they give you and manipulated the underlying RTF data. Also, the reason that your second block of code doesnt work is that RTF applies formatting as it goes, so if you call a function (or Property in this case) to change the selection color, it will only apply it for the currently selected block. Any changes made to the selection after that call become irrelavent.

You can play around with the RGB values, or here is a great source on how to do different things within the RTF control. Pop this function in your code and see how well it works. I use it to provide realtime syntax highlighting for SQL code.

    public void HighlightText(int offset, int length)
    {
        String sText = richTextBox.Text.Trim();
        sText = sText.Insert(offset + length - 1, @" \highlight0");
        sText = sText.Insert(offset, @" \highlight1");
        String s = @"{\rtf1\ansi\deff0{\fonttbl{\f0\fnil\fcharset0 Courier New;}}
            {\colortbl ;\red255\green255\blue0;}\viewkind4\uc1\pard";
        s += sText;
        s += @"\par}";
        richTextBox.Rtf = s;
    }

I'm sorry I didn't quite understand your code, but I tried to just copy and paste your code in my application and call this method every time I want to highlight something but it didn't work at all. Anyway, thanks for your help. I give up! — duncanportelli, Jun 02 '12 at 12:45

score 0 · Answer 3 · answered May 29 '12 at 20:34

0

Does it make any difference if you set the SelectionBackColor outside of the loop?

Looking into the RichTextBox with Reflector shows, that a WindowMessage is sent to the control every time when the color is set. In the case of large number of offsets this might lead to highlighting the already highlighted words again and again, leading to O(n^2) behavior.

answered May 29 '12 at 20:34

Tadej Mali

1,143
8
18

Already tried it. This doesn't highlight anything: `richTextBox.SelectionBackColor = Color.Yellow; foreach (int offset in offsets) { richTextBox.Select(offset, searchString.Length); }` – duncanportelli May 30 '12 at 16:46

Highlighting in a RichTextBox is taking too long

3 Answers3