Solved

What is wrong with this code?

Posted on 2010-09-07
8
347 Views
Last Modified: 2012-05-10
So I'm converting an application from vb6 to vb..net and have run into an error with the following code in vb.net. If I type a number into "textnodesnum" during runtime, I get an error saying that "Double.Parse(Chr(13))" is in the wrong format.... any ideas?



Private Sub txtNodesNum_KeyPress(ByVal eventSender As System.Object, ByVal eventArgs As System.Windows.Forms.KeyPressEventArgs) Handles txtNodesNum.KeyPress 
  Dim KeyAscii As Short = Asc(eventArgs.KeyChar) 
  Dim char_Renamed As Double 

 
  char_Renamed = Double.Parse(Chr(KeyAscii)) 
  If char_Renamed = Double.Parse(Chr(13)) Then 
  txtNodesNumChange() 
  End If 
  eventArgs.KeyChar = Chr(KeyAscii) 
  If KeyAscii = 0 Then 
  eventArgs.Handled = True 
  End If 
  End Sub

Open in new window

0
Comment
Question by:sn1fflez
8 Comments
 
LVL 51

Expert Comment

by:HainKurt
ID: 33619739
try

char_Renamed = Asc(Chr(KeyAscii))
0
 
LVL 51

Expert Comment

by:HainKurt
ID: 33619750
and Asc(...) should return int, not double...
0
 
LVL 18

Expert Comment

by:John (Yiannis) Toutountzoglou
ID: 33619852
try
Private Sub txtNodesNum_KeyPress(ByVal eventSender As System.Object, ByVal eventArgs As System.Windows.Forms.KeyPressEventArgs) Handles txtNodesNum.KeyPress
        Dim KeyAscii As Short = CShort(Asc(eventArgs.KeyChar))
        Dim char_Renamed As String

        char_Renamed = CStr(Chr(KeyAscii))
        If char_Renamed = Chr(13) Then
            MessageBox.Show("....")
        End If
        eventArgs.KeyChar = Chr(KeyAscii)
        If KeyAscii = 0 Then
            eventArgs.Handled = True
        End If
    End Sub

Open in new window

0
NAS Cloud Backup Strategies

This article explains backup scenarios when using network storage. We review the so-called “3-2-1 strategy” and summarize the methods you can use to send NAS data to the cloud

 
LVL 17

Expert Comment

by:Zhaolai
ID: 33619858
It should be as simple as this:

Private Sub txtNodesNum_KeyPress(ByVal eventSender As System.Object, ByVal eventArgs As System.Windows.Forms.KeyPressEventArgs) Handles txtNodesNum.KeyPress
            Dim KeyAscii As Short = Asc(eventArgs.KeyChar)
            If Chr(KeyAscii) = Chr(13) Then
                txtNodesNumChange()
            End If
            If KeyAscii = 0 Then
                eventArgs.Handled = True
            End If
End Sub

Open in new window

0
 
LVL 18

Expert Comment

by:John (Yiannis) Toutountzoglou
ID: 33619885
Same Result..
0
 

Author Comment

by:sn1fflez
ID: 33620002
I will try these solutions out.. thanks for the help so far everyone
0
 
LVL 18

Expert Comment

by:John (Yiannis) Toutountzoglou
ID: 33620216
as i  noticed from your code i think that you want to achieve the enter Key ...you can accomplish this "also" with key_Down Event
Private Sub txtNodesNum_KeyDown(ByVal sender As Object, ByVal e As System.Windows.Forms.KeyEventArgs) Handles txtNodesNum.KeyDown
        If e.KeyCode = Keys.Return Then
           ...........
        End If
    End Sub

Open in new window

0
 
LVL 5

Accepted Solution

by:
andr_gin earned 500 total points
ID: 33625226
1.)
The Double.Parse method throws an exception, if the string is not valid.
You can also use the Double.TryParse method to check, if a string can be converted.

2.)
Chr(13) is the carriage return. In VB.NET you can also write vbCr instead

3.) Lets look at your code:

  Dim KeyAscii As Short = Asc(eventArgs.KeyChar)

  You do not get the Char of the key that was pressed. You get the code of the key. Sometimes it is the same (the a key is the ascii of the lower case letter a), but often not. For example there is not key code for upper case a. You only get Shift + lower case a.
You should handle the KeyPress event instead. Then you get the Char that was pressed.


  Dim char_Renamed As Double
  char_Renamed = Double.Parse(Chr(KeyAscii))

  This throws an exception if any key other than a number is pressed.


  If char_Renamed = Double.Parse(Chr(13)) Then
  When handling KeyPress it is also not necessary to make a Double.Parse. You simply write:
  If e.KeyChar = vbCr OrElse e.KeyChar = vbLf Then
         txtNodesNumChange()
  End If

    eventArgs.KeyChar = Chr(KeyAscii)
  If KeyAscii = 0 Then
 
I dont know what you want to do with this line, but there is no Chr(0). 0 means "end of string" and is no valid character. It is not possible to get a KeyPressed or KeyDown event, if no key was pressed at all.
0

Featured Post

Is Your AD Toolbox Looking More Like a Toybox?

Managing Active Directory can get complicated.  Often, the native tools for managing AD are just not up to the task.  The largest Active Directory installations in the world have relied on one tool to manage their day-to-day administration tasks: Hyena. Start your trial today.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Introduction While answering a recent question about filtering a custom class collection, I realized that this could be accomplished with very little code by using the ScriptControl (SC) library.  This article will introduce you to the SC library a…
When trying to find the cause of a problem in VBA or VB6 it's often valuable to know what procedures were executed prior to the error. You can use the Call Stack for that but it is often inadequate because it may show procedures you aren't intereste…
Get people started with the utilization of class modules. Class modules can be a powerful tool in Microsoft Access. They allow you to create self-contained objects that encapsulate functionality. They can easily hide the complexity of a process from…
This lesson covers basic error handling code in Microsoft Excel using VBA. This is the first lesson in a 3-part series that uses code to loop through an Excel spreadsheet in VBA and then fix errors, taking advantage of error handling code. This l…

832 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question