vb.net regex comparison to string

Question

im trying to copy the lines until an empty line is detected but this code lags my computer i dont know what im doing wrong it is because im running while loop inside another while loop? here is my code:

ElseIf String.Compare(line, "the") = 1 And Not line.ToLower().Contains("by") Then
            While True

                Dim title = New Regex("^\s*$").Matches(line).Count
                If title = 1 Then Exit While

                builder.AppendLine(line)
                builder.AppendLine(reader.ReadLine())

            End While

You could probably use the String.IsNullOrEmpty() or String.IsNullOrWhiteSpace() methods instead of the RegEx. Just like Chris mentioned on his edit. — Jonathan
– Jonathan, Commented Feb 26, 2013 at 14:24

Chris Haas · Accepted Answer · 2013-02-26 14:20:12Z

3

You're not resetting the line variable. Something like this should work probably:

While True
    Dim title = New Regex("^\s*$").Matches(line).Count
    If title = 1 Then Exit While
    builder.AppendLine(line)
    line = reader.ReadLine()
End While

EDIT

Instead of regex you could instead just use String.IsNullOrWhiteSpace() which might make your code easier to read in the future.

answered Feb 26, 2013 at 14:20

Chris Haas

55.7k13 gold badges152 silver badges293 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Steven Doggart · Accepted Answer · 2013-02-26 14:21:40Z

2

It's hard to tell exactly what you are trying to do, with that short snippet of code, but the reason why it doesn't work is clear. You will get stuck in an infinite loop if title does not equal 1. You assume, I think, that the value of line will change reach time you call reader.ReadLine inside the While loop. You probably meant to do something like this:

line = reader.ReadLine()
builder.AppendLine(line)

answered Feb 26, 2013 at 14:21

Steven Doggart

43.8k8 gold badges71 silver badges109 bronze badges

Collectives™ on Stack Overflow

vb.net regex comparison to string

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related