|
-
Feb 3rd, 2005, 02:46 PM
#14
Re: Stupid inherits issue, or not?.
Ah.. i missed the form... (i didnt open the project)
what i think:
Not bad...
You may find using Regular Expressions a better solution for doing this...
I still would prefer you to use IndexOf (as it is the vb.net way, and it still is using it anyway) (same with mid. (String.Substring)
You are using Option Explict, Option Strict 
Modulization (splitting code up into classes, blocks) ok, you still do alot in your form.
Identing: good.
VB Code:
Private Function isSpace(ByVal var As String) As Boolean
If var = Chr(32) Then isSpace = True
End Function
Private Function isLetter(ByRef var As String) As Boolean
If Asc(var) >= 65 And _
Asc(var) <= 90 Or _
Asc(var) >= 97 And _
Asc(var) <= 122 Then
isLetter = True
End If
End Function
could be replaced with
VB Code:
Char.IsLetter("x"c)
Char.IsWhiteSpace("x"c)
Its pretty good
Tips:
- Google is your friend! Search before posting!
- Name your thread appropriately... "I Need Help" doesn't cut it!
- Always post your code!!!! We can't read your mind!!! (well, at least most of us!)
- Allways Include the Name and Line of the Exception (if one is occuring!)
- If it is relevant state the version of Visual Studio/.Net Framwork you are using (2002/2003/2005)
If you think I was helpful, rate my post  IRC Contact: Rizon/xous ChakraNET/xous Freenode/xous
Posting Permissions
- You may not post new threads
- You may not post replies
- You may not post attachments
- You may not edit your posts
-
Forum Rules
|
Click Here to Expand Forum to Full Width
|