Conversion from string "String" to type 'Double' is not valid.

Question

0.00/5 (No votes)

See more:

hai i am passing the Value in query string ,at the time i am receiving this error this is my code:

VB

Protected Sub Submit_Click(ByVal sender As Object, ByVal e As EventArgs) Handles Submit.Click
        Try
            oDr = oMainForm.oConn.RunQuery("insert into BUSON_ASSIGN(Callid,ClientId,CallSubject,CallDescription,Priority,CallType,Status,FeedBack,RspDateTime,Assignto)values('" + txtCallId.Text + "','" + ClientId.SelectedValue + "','" + CallSubject.Text + "','" + CallDesc.Text + "','" + Priority.SelectedValue + "','" + CallType.SelectedValue + "','" + CallStatus.SelectedValue + "','" + Feedback.Text + "', getDate() ,' " + DropDownConsultant.SelectedItem.Text() + "')")
            Errlbl0.Text = ""

            Dim Body As String = Me.PopulateBody("Hello", _
                                                 "I am Assign one Call Log for You Please Click Hear to View the Call Log", _
                                                "http://localhost:3482/AssignReport.aspx?UserId=" + txtCallId.Text + "&AssName" = +DropDownConsultant.SelectedItem.Text & _
                                               "", _
            ("" & _
            " " & _
            ""))//hear only i amgeting error
            Me.SendHtmlFormattedEmail("recipient@gmail.com", "Reg Serives Call!", Body)
            MsgBox("Your assign the Call log Your assign the case successfully!!!")
            Response.Redirect("AllCallReport.aspx")
        Catch ex As Exception
            Errlbl0.Text = ex.Message.ToString()

        End Try
    End Sub

Posted 1-Jul-13 21:18pm

DhananjayanP

Updated 1-Jul-13 21:20pm

Pheonyx

v2

Add a Solution

Comments

Richard MacCutchan 2-Jul-13 3:31am

The first thing you should do is rewrite the above to use proper parameters. As it stands your code is at serious risk from SQL injection.
You have a = sign in that statement; is that correct?

2 solutions

Solution 3

Use cdbl function where require to convert string into double:
Like suppose in your code, if field CallID and ClientId are of double type in database then

cdbl(txtCallId.Text) and
cdbl(ClientId.SelectedValue)

Posted 2-Jul-13 0:24am

bhuvamehul

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Pheonyx · Accepted Answer · 2013-07-01T21:34:00

Well the error tells you the issue, some of your values that you are using are in the format of a string, but they need to be as a type double.

Without knowing your table structure, I would imagine the following are supposed to be doubles:

txtCallId.Text,
ClientId.SelectedValue

I would also suggest you look up how to user parameters in your sql queries as you are very susceptible to SQL Injection attacks through your form based on this code.
(Google search results[^]

If you want to keep your code as it is, I would suggest using the Double.Parse() method to convert the relevant string values to double values and see how you get on.