Bug: string.substrin g.trim = "" (boolean expression issue)

[Darren Anderson]

I have a function that I've tried using in an if then statement and I've
found that no matter how much reworking I do with the code, the expected
result is incorrect.

the code:

If Not (strIn.Substrin g(410, 10).Trim = "") Then
'Something processed
Else
'Something processed
End If

The string.substrin g(410,10).trim is equivelent to an emptry string, e.g.
"", when it comes in. So the expected result is supposed to be TRUE when you
use (strIn.Substrin g(410, 10).Trim = ""), and I use the NOT keyword which
negates the true to a false (or vice versa). I can not get this DAMN
function to give me the proper boolean expression, is this a LOGIC bug...

I've tried using string.substrin g.trim = space(?) and I've tried
string.substrin g.trim <> "" and none of these will yield the correct boolean
value in an If...Then context

[Herfried K. Wagner [MVP]]

"Darren Anderson" <Da************ @discussions.mi crosoft.com> schrieb:

I have a function that I've tried using in an if then statement and I've
found that no matter how much reworking I do with the code, the expected
result is incorrect.

the code:

If Not (strIn.Substrin g(410, 10).Trim = "") Then
'Something processed
Else
'Something processed
End If

The string.substrin g(410,10).trim is equivelent to an emptry string, e.g.
"", when it comes in. So the expected result is supposed to be TRUE when
you
use (strIn.Substrin g(410, 10).Trim = ""), and I use the NOT keyword which
negates the true to a false (or vice versa). I can not get this DAMN
function to give me the proper boolean expression, is this a LOGIC bug...

Your code works just fine for me. The condition in 'If...Then' is true if
the trimmed string is not empty (zero-length string).

--
M S Herfried K. Wagner
M V P <URL:http://dotnet.mvps.org/>
V B <URL:http://dotnet.mvps.org/dotnet/faqs/>

[Jay B. Harlow [MVP - Outlook]]

Darren,
First "'Select isn't Broken!" :-)

http://www.pragmaticprogrammer.com/p...rule_list.html
Do you have any sample code that demonstrates the perceived problem?

The string.substrin g(410,10).trim is equivalent to an empty string, e.g.
"", If you assign string.substrin g(410,10).trim to a variable, what is in the
variable? What is the length of the variable?

Where is strIn coming from? Does it have ChrW(0) characters in it?

ChrW(0) will make the string appear to be "" in the debugger, when its
actually not, as ChrW(0) is a valid string.

If you run the following what do you see?

Dim strIn As String = New String(" "c, 450)

Dim x As String = strIn.Substring (410, 10).Trim()
Debug.WriteLine (x, "x")
Debug.WriteLine (x.Length, "x.length")

Debug.WriteLine (Not (x.Trim() = ""), "not (x.trim = space)")
Debug.WriteLine (x.Trim() <> "", "x.trim <> space")
Debug.WriteLine (x.Trim() = "", "x.trim = space")

Debug.WriteLine (Not (strIn.Substrin g(410, 10).Trim() = ""), "not
(substring.trim = space)")
Debug.WriteLine (strIn.Substrin g(410, 10).Trim() <> "",
"substring. trim <> space")
Debug.WriteLine (strIn.Substrin g(410, 10).Trim() = "",
"substring. trim = space")

What do you see if you use your original string instead of "New String(" "c,
450)" for strIn?

Hope this helps
Jay
"Darren Anderson" <Da************ @discussions.mi crosoft.com> wrote in
message news:96******** *************** ***********@mic rosoft.com...I have a function that I've tried using in an if then statement and I've
found that no matter how much reworking I do with the code, the expected
result is incorrect.

the code:

If Not (strIn.Substrin g(410, 10).Trim = "") Then
'Something processed
Else
'Something processed
End If

The string.substrin g(410,10).trim is equivelent to an emptry string, e.g.
"", when it comes in. So the expected result is supposed to be TRUE when
you
use (strIn.Substrin g(410, 10).Trim = ""), and I use the NOT keyword which
negates the true to a false (or vice versa). I can not get this DAMN
function to give me the proper boolean expression, is this a LOGIC bug...

I've tried using string.substrin g.trim = space(?) and I've tried
string.substrin g.trim <> "" and none of these will yield the correct
boolean
value in an If...Then context

[Phill. W]

"Darren Anderson" <Da************ @discussions.mi crosoft.com> wrote in
message news:96******** *************** ***********@mic rosoft.com...

I have a function that I've tried using in an if then statement and
I've found that no matter how much reworking I do with the code,
the expected result is incorrect.

If Not (strIn.Substrin g(410, 10).Trim = "") Then

Are you sure the substring contains /exactly/ 10 /spaces/?
Could any of them be Null or other whitespace characters?
Trim() will only remove spaces.

Is strIn /at least/ 419 characters long? VB will get upset if you
try to slice out a substring that's doesn't fit within the source string,
as in

"abc".SubString ( 2, 2 ) ' fails

You're /not/ using "On Error Resume Next", /are/ you?

HTH,
Phill W.

[Jay B. Harlow [MVP - Outlook]]

Phill,

Are you sure the substring contains /exactly/ 10 /spaces/?
Could any of them be Null or other whitespace characters?
Trim() will only remove spaces. Interesting, according to MSDN String.Trim() will remove whitespace
characters, it suggests it is using Char.IsWhiteSpa ce to determine what is a
whitespace character!

http://msdn.microsoft.com/library/de...trimtopic1.asp

Testing shows (see below) that String.Trim() works with: ChrW(9), ChrW(10),
ChrW(11), ChrW(12), ChrW(13), ChrW(32), ChrW(12288)

However try any of the following and it fails:
ChrW(133), ChrW(160), ChrW(5760), ChrW(8192), ChrW(8193)
ChrW(8194), ChrW(8195), ChrW(8196), ChrW(8197), ChrW(8198)
ChrW(8199), ChrW(8200), ChrW(8201), ChrW(8202), ChrW(8203)
ChrW(8232), ChrW(8233), ChrW(8239)

According to Char.IsWhiteSpa ce all of the above are whitespace characters!

ChrW(133) is the "NEL" or Next Line control char. ChrW(160) is a non
breaking space
http://www.unicode.org/charts/PDF/U0080.pdf

CharW(5760) is the Ogham space mark.
http://www.unicode.org/charts/PDF/U1680.pdf

CharW(8192) to ChrW(8239) are various width spaces:
http://www.unicode.org/charts/PDF/U2000.pdf
Sample code I used to test with:

Const format As String = "ChrW({0})"
Dim sb As New StringBuilder
For charCode As Integer = AscW(Char.MinVa lue) To AscW(Char.MaxVa lue)
Dim ch As Char = ChrW(charCode)
If Char.IsWhiteSpa ce(ch) Then
sb.Append(ch)
Debug.WriteLine (String.Format( format, charCode),
"IsWhiteSpa ce")
End If
Next

Dim str As String = sb.ToString().T rim()
Debug.WriteLine (str, "str")
Debug.WriteLine (str.Length, "str.length ")
Debug.WriteLine (Nothing)
For Each ch As Char In str
Dim charCode As Integer = AscW(ch)
Debug.WriteLine (String.Format( format, charCode), "not trimmed")
Next

If you really needed them trimmed a workaround would be to use
String.Trim(Cha r())

Static whitespace() As Char = {ChrW(&H9), ChrW(&HA), ChrW(&HB), _
ChrW(&HC), ChrW(&HD), ChrW(&H20), ChrW(&H85), ChrW(&HA0), _
ChrW(&H1680), ChrW(&H2000), ChrW(&H2001), ChrW(&H2002), _
ChrW(&H2003), ChrW(&H2004), ChrW(&H2005), ChrW(&H2006), _
ChrW(&H2007), ChrW(&H2008), ChrW(&H2009), ChrW(&H200A), _
ChrW(&H200B), ChrW(&H2028), ChrW(&H2029), ChrW(&H202F), _
ChrW(&H3000)}

Dim str As String = sb.ToString().T rim(whitespace)

Hope this helps
Jay
"Phill. W" <P.A.Ward@o-p-e-n-.-a-c-.-u-k> wrote in message
news:cv******** **@yarrow.open. ac.uk... "Darren Anderson" <Da************ @discussions.mi crosoft.com> wrote in
message news:96******** *************** ***********@mic rosoft.com...

I have a function that I've tried using in an if then statement and
I've found that no matter how much reworking I do with the code,
the expected result is incorrect.

If Not (strIn.Substrin g(410, 10).Trim = "") Then

Are you sure the substring contains /exactly/ 10 /spaces/?
Could any of them be Null or other whitespace characters?
Trim() will only remove spaces.

Is strIn /at least/ 419 characters long? VB will get upset if you
try to slice out a substring that's doesn't fit within the source string,
as in

"abc".SubString ( 2, 2 ) ' fails

You're /not/ using "On Error Resume Next", /are/ you?

HTH,
Phill W.

[Darren Anderson]

"Jay B. Harlow [MVP - Outlook]" wrote:

Darren,
First "'Select isn't Broken!" :-)

http://www.pragmaticprogrammer.com/p...rule_list.html
[...]
What do you see if you use your original string instead of "New String(" "c,
450)" for strIn?

Hope this helps
Jay

"Jay B. Harlow [MVP - Outlook]" wrote:
Darren,
First "'Select isn't Broken!" :-)

http://www.pragmaticprogrammer.com/p...rule_list.html

[...]

Hope this helps
Jay

Jay,
Thanks it did help!

After going through and debugging it and looking at the actual characters
within the string, I found that TRIM wasn't eliminating chr(0) from the
string. I find it extremely frustrating that something simple like removal
of an unused character is so hard... SO I wrote a real 'TRIM' function.
I'll probably modify it more over time, but I'm in the middle of a project;
again thanks.
-------------------------------
'Use this function to remove chr(0) and chr(32) at beginning and end of string
' Function takes 1 input of string type and outputs string type, without
excessive characters
Public Function CleanUpString(B yVal strIn As String) As String
Dim nStartPos As Integer, nEndPos As Integer 'For finding beginning and
end of string
Dim nLen As Integer 'Length of string

CleanUpString = vbNullString 'Start with an empty string

If strIn.Length = 0 Then Exit Function 'Dummy check

nStartPos = 0 'Starting position is at the beginning of the string

'Find the actual beginning of the string
Do Until (Not strIn.ToCharArr ay(nStartPos, 1) = Chr(0) And Not
strIn.ToCharArr ay(nStartPos, 1) = Chr(32))
nStartPos += 1 'iterate forward
If nStartPos >= Len(strIn) Then Exit Do 'Exit if the whole string is
chr(0) or chr(32)
Loop

If (nStartPos >= Len(strIn)) Then Exit Function 'If the whole string is
nothing but chr(0) and chr(32) then return "" (a 'real' empty string)

nEndPos = (Len(strIn) - 1) 'Set the ending position at the end of string

'Find the actual end of the string
Do Until (Not strIn.ToCharArr ay(nEndPos, 1) = Chr(0) And Not
strIn.ToCharArr ay(nEndPos, 1) = Chr(32))
nEndPos -= 1 'iterate backward
If nEndPos <= 0 Then Exit Do
Loop

nLen = Len(strIn) - (((Len(strIn) - 1) - nEndPos) + nStartPos) 'Get the
length of important characters in the string

CleanUpString = strIn.Substring (nStartPos, nLen) 'Get the "important"
string for output

End Function
-------------------------------------

Darren

[Jay B. Harlow [MVP - Outlook]]

Darren,
Here's a simplified version of your function:

' Use this function to remove chr(0) and chr(32) at beginning and end of
string
' Function takes 1 input of string type and outputs string type, without
' excessive characters
Public Function CleanUpString(B yVal strIn As String) As String
Return strIn.Trim(ChrW (0), ChrW(32))
End Function
Also you can index into the string rather then converting it to a CharArray.

Do While (strIn.Chars(nS tartPos) = Chr(0) OrElse strIn.Chars(nSt artPos)
= Chr(32))
Calling ToCharArray, twice, during each iteraction of the loop, in both
loops, may cause excessive strain on the Garbage Collector. As you are
creating a number of temporary arrays...

Note I find positive logic to be more readable then negative logic, hence I
"flipped" your condition.
Hope this helps
Jay

"Darren Anderson" <Da************ @discussions.mi crosoft.com> wrote in
message news:2B******** *************** ***********@mic rosoft.com...

"Jay B. Harlow [MVP - Outlook]" wrote:

Darren,
First "'Select isn't Broken!" :-)

http://www.pragmaticprogrammer.com/p...rule_list.html

[...]

What do you see if you use your original string instead of "New String("
"c,
450)" for strIn?

Hope this helps
Jay

"Jay B. Harlow [MVP - Outlook]" wrote:

Darren,
First "'Select isn't Broken!" :-)

http://www.pragmaticprogrammer.com/p...rule_list.html

[...]

Hope this helps
Jay

Jay,
Thanks it did help!

After going through and debugging it and looking at the actual characters
within the string, I found that TRIM wasn't eliminating chr(0) from the
string. I find it extremely frustrating that something simple like
removal
of an unused character is so hard... SO I wrote a real 'TRIM' function.
I'll probably modify it more over time, but I'm in the middle of a
project;
again thanks.
-------------------------------
'Use this function to remove chr(0) and chr(32) at beginning and end of
string
' Function takes 1 input of string type and outputs string type, without
excessive characters
Public Function CleanUpString(B yVal strIn As String) As String
Dim nStartPos As Integer, nEndPos As Integer 'For finding beginning and
end of string
Dim nLen As Integer 'Length of string

CleanUpString = vbNullString 'Start with an empty string

If strIn.Length = 0 Then Exit Function 'Dummy check

nStartPos = 0 'Starting position is at the beginning of the string

'Find the actual beginning of the string
Do Until (Not strIn.ToCharArr ay(nStartPos, 1) = Chr(0) And Not
strIn.ToCharArr ay(nStartPos, 1) = Chr(32))
nStartPos += 1 'iterate forward
If nStartPos >= Len(strIn) Then Exit Do 'Exit if the whole string
is
chr(0) or chr(32)
Loop

If (nStartPos >= Len(strIn)) Then Exit Function 'If the whole string is
nothing but chr(0) and chr(32) then return "" (a 'real' empty string)

nEndPos = (Len(strIn) - 1) 'Set the ending position at the end of
string

'Find the actual end of the string
Do Until (Not strIn.ToCharArr ay(nEndPos, 1) = Chr(0) And Not
strIn.ToCharArr ay(nEndPos, 1) = Chr(32))
nEndPos -= 1 'iterate backward
If nEndPos <= 0 Then Exit Do
Loop

nLen = Len(strIn) - (((Len(strIn) - 1) - nEndPos) + nStartPos) 'Get the
length of important characters in the string

CleanUpString = strIn.Substring (nStartPos, nLen) 'Get the "important"
string for output

End Function
-------------------------------------

Darren

[Herfried K. Wagner [MVP]]

Darren,

"Darren Anderson" <Da************ @discussions.mi crosoft.com> schrieb:

After going through and debugging it and looking at the actual characters
within the string, I found that TRIM wasn't eliminating chr(0) from the
string. I find it extremely frustrating that something simple like
removal
of an unused character is so hard... SO I wrote a real 'TRIM' function.
I'll probably modify it more over time, but I'm in the middle of a
project;
again thanks.
-------------------------------
'Use this function to remove chr(0) and chr(32) at beginning and end of
string
' Function takes 1 input of string type and outputs string type, without
excessive characters

Notice that 'Trim' is overloaded:

\\\
Dim t As String = _
s.Trim(New Char() {ControlChars.N ullChar, ChrW(&H5)})
///

--
M S Herfried K. Wagner
M V P <URL:http://dotnet.mvps.org/>
V B <URL:http://dotnet.mvps.org/dotnet/faqs/>

[Jay B. Harlow [MVP - Outlook]]

Herfried,
ChrW(&H5)?

Wouldn't that be either ChrW(32) or ChrW(&H20)?

:-)

Jay

"Herfried K. Wagner [MVP]" <hi************ ***@gmx.at> wrote in message
news:%2******** ********@TK2MSF TNGP09.phx.gbl. ..

Darren,

"Darren Anderson" <Da************ @discussions.mi crosoft.com> schrieb:

After going through and debugging it and looking at the actual characters
within the string, I found that TRIM wasn't eliminating chr(0) from the
string. I find it extremely frustrating that something simple like
removal
of an unused character is so hard... SO I wrote a real 'TRIM' function.
I'll probably modify it more over time, but I'm in the middle of a
project;
again thanks.
-------------------------------
'Use this function to remove chr(0) and chr(32) at beginning and end of
string
' Function takes 1 input of string type and outputs string type, without
excessive characters

Notice that 'Trim' is overloaded:

\\\
Dim t As String = _
s.Trim(New Char() {ControlChars.N ullChar, ChrW(&H5)})
///

--
M S Herfried K. Wagner
M V P <URL:http://dotnet.mvps.org/>
V B <URL:http://dotnet.mvps.org/dotnet/faqs/>

[Herfried K. Wagner [MVP]]

Jay,

"Jay B. Harlow [MVP - Outlook]" <Ja************ @msn.com> schrieb:

ChrW(&H5)?

Wouldn't that be either ChrW(32) or ChrW(&H20)?

:-)

Ooops... :-).

--
M S Herfried K. Wagner
M V P <URL:http://dotnet.mvps.org/>
V B <URL:http://dotnet.mvps.org/dotnet/faqs/>