[RESOLVED] Asc(65)

I have a string that i am cutting up.
the string is looped through looking at each character (MyStr)

VB Code:

If MyStr = Asc(65) Or Asc(66) Or Asc(67) Or Asc(68) Or Asc(69) Or
 Asc(70) Or Asc(71) Or Asc(72) Or Asc(73) Or Asc(74) Or Asc(75) Or Asc(76)
 Or Asc(77) Or Asc(78) Or Asc(79) Or Asc(80) Or Asc(81) Or Asc(82) Or 
Asc(83) Or Asc(84) Or Asc(85) Or Asc(86) Or Asc(87) Or Asc(88) Or Asc(89) 
Or Asc(90) Or Asc(97) Or Asc(98) Or Asc(99) Or Asc(100) Or Asc(101) Or 
Asc(102) Or Asc(103) Or Asc(104) Or Asc(105) Or Asc(106) Or Asc(107) Or 
Asc(108) Or Asc(109) Or Asc(110) Or Asc(111) Or Asc(112) Or Asc(113) Or 
Asc(114) Or Asc(115) Or Asc(116) Or Asc(117) Or Asc(118) Or Asc(119) Or 
Asc(120) Or Asc(121) Or Asc(122) Then

the above code lists the ascii numbers for A-Z & a-z only! So it should only execute if the character in MyStr is A-Z or a-z but when the character in MyStr is space, or tab it is still processed!!
can anyone tell me why?

You cannot do that (as you are not comparing values, just doing boolean checks with non-zero values [which are therefore all True - so the If is always True]), it would need to be like this instead:

VB Code:

If (MyStr = Asc(65)) Or (MyStr = Asc(66)) Or (MyStr =Asc(67)) Or ....

or alternatively, you could use more efficient code like this:

VB Code:

IF ((Asc(MyStr) >= 65) And (Asc(MyStr) <= 90)) Or ((Asc(MyStr) >= 97) And (Asc(MyStr) <= 122)) Then

or even better:

VB Code:

Select Case MyStr
Case "A" to "Z", "a" to "z"
  'code here
End Select

?? Asc(65) = 54, Chr(65) = "A"

I dont understand what u are doing?
only allow capitals / lowercase letters?

or u could do this.. but I like the select case better as shown above

VB Code:

If InStr(UCase(MyStr), "ABCDEFGHIJKLMNOPQRSTUVWXYZ") <> 0 Then
 
End If

but I still dont undertand why Asc() is being used.. thats not the correct way to use it???

Asc()
Returns anInteger representing the character code corresponding to the first letter in a string.

Asc("A") would be correct...
Chr(65) would be to test for "A"

right??

Doh! Thanks people :thumb:
to hot to think straight is my excuse today ;)

if you're looping through every character of a string you'd be better off using a byte array.

Quote:

---

Originally Posted by Static

but I still dont undertand why Asc() is being used.. thats not the correct way to use it???

---

Doh!! Why didn't I spot that! :eek:

"to hot to think straight", si. :)