Solved

executenonquery() Syntax Error

Posted on 2010-09-21
6
383 Views
Last Modified: 2013-11-07
I'm getting a "Syntax error converting datetime from character to string".
The returned error says it is located on this line of code: cmd.executenonquery()

Please see code below. All and any help is greatly appreciated. Thanks!
Public Sub btnSubmit_Click (byval s as object, byval e as system.eventargs)

	Dim con as sqlConnection

	dim cmd as sqlcommand

	Dim dtr as Sqldatareader

	Dim str as string

	Dim strAccount as string

	dim BreakTestID as integer

	

	con = new sqlconnection ("Server=sqlserver;UID=sa;PWD=#####;Database=BreakTest")

	

	str = "INSERT tblBreakTest (BreakTestNO, InstallDate, CustomerNO, LoginID,BreaktestDate,Division,Application, TypeofService, Comments, Location, ProdDesc, NoofLines, ProductNO, ReturnReason, Status, SplicerName, SplicerRegion, SplicerCountry, SplicerPhone, SplicerPostalCode, SplicerState) VALUES "

	str = str & "(@BreakTestNo, @InstallDate, @Customer, @Login, @BreakTestDate, @Division, @Application, @TypeofService, @Comments, @Location, @ProdDesc, @NoofLines, @Product, @ReturnReason, @Status, @SplicerName, @Region, @Country, @SplicerPhone, @PostalCode, @State )"

	cmd = New sqlcommand (str, con)

	cmd.parameters.add ("@BreakTestNo", lblBreakTestID.Text)

	cmd.parameters.add ("@InstallDate", txtinstallDate.text)

	cmd.parameters.add ("@Customer", lblCompanyNo.text)

	cmd.parameters.add ("@Login", label2.text)

	cmd.parameters.add ("@BreakTestDate", txtDate.text)

	cmd.parameters.add ("@Division", cboDivision.text)

	cmd.parameters.add ("@Application", cboApplication.text)

	cmd.parameters.add ("@TypeofService", cboService.text)

	cmd.parameters.add ("@Comments", txtComments.text)

	cmd.parameters.add ("@Location", cboLocation.text)

	cmd.parameters.add ("@ProdDesc", txtProdDesc.text)

	cmd.parameters.add ("@NoofLines", txtNoLines.text)

	cmd.parameters.add ("@Product", txtProductNo.text)

	cmd.parameters.add ("@ReturnReason", cboReturnReason.text)

	cmd.parameters.add ("@SplicerName", txtSplicerName.text)

	cmd.parameters.add ("@Region", txtRegion.text)

	cmd.parameters.add ("@Country", txtCountry.text)

	cmd.parameters.add ("@SplicerPhone", txtPhoneNumber.text)

	cmd.parameters.add ("@PostalCode", txtPostalCode.text)

	cmd.parameters.add ("@State", txtState.text)

	

	strAccount = request.cookies("AccountCookie").value

	

	Select Case strAccount

	Case "Regional Sales Manager"

		cmd.parameters.add ("@Status", "Pending-GSM Approval")

	Case "General Sales Manager"

		cmd.parameters.add ("@Status","Pending-Technical Manager Approval")

	Case "R & D Personnel"

		cmd.parameters.add ("@Status", "Approved-Pending Receipt of Rope")

	Case Else

		cmd.parameters.add("@Status", "Initiated")

	End Select

	

	con.open()

	cmd.executenonquery()

	con.close()

Open in new window

0
Comment
Question by:dhaverstraw
6 Comments
 
LVL 4

Expert Comment

by:timexist
ID: 33729667
error is from this:
cmd.parameters.add ("@InstallDate", txtinstallDate.text)
try to change to this:
cmd.parameters.add ("@InstallDate",Cdate(txtinstallDate.text))

Of course you need to user's input first.
0
 

Author Comment

by:dhaverstraw
ID: 33729781
Changed the code. User now recieves this error: Conversion from string "tbd" to type 'Date' is not valid.
Error is on cmd.parameters.add ("@InstallDate", Cdate(txtinstallDate.text))
0
 
LVL 4

Accepted Solution

by:
timexist earned 105 total points
ID: 33729821
you need to check this txtinstallDate.text first.
before you pass this string in, you should put an validation on this field.
something likes a mask.
All Input Data Is Evil.
0
Do You Know the 4 Main Threat Actor Types?

Do you know the main threat actor types? Most attackers fall into one of four categories, each with their own favored tactics, techniques, and procedures.

 
LVL 29

Assisted Solution

by:Kumaraswamy R
Kumaraswamy R earned 20 total points
ID: 33731632


try to change like this:
if (txtinstallDate.text != "")
 cmd.parameters.add ("@InstallDate",Cdate(txtinstallDate.text))
else
 cmd.parameters.add ("@InstallDate",datetime.minvalue)
0
 
LVL 8

Expert Comment

by:Gururaj Badam
ID: 33731909
In the simplest terms how many date columns does the table contain and what are the values you're passing through the command?
0
 
LVL 5

Expert Comment

by:soujanya_g
ID: 33763202
Hi

InstallDate as taken as text parameter insted of date time convert it into date time using cDate and eecute your application
0

Featured Post

Get up to 2TB FREE CLOUD per backup license!

An exclusive Black Friday offer just for Expert Exchange audience! Buy any of our top-rated backup solutions & get up to 2TB free cloud per system! Perform local & cloud backup in the same step, and restore instantly—anytime, anywhere. Grab this deal now before it disappears!

Join & Write a Comment

Pimping Sharepoint 2007 without Server-Side Code Part 1 One of my biggest frustrations with Sharepoint 2007 in the corporate world is that while good-intentioned managers lock down the more interesting capabilities of Sharepoint programming in…
This article explains how to reset the password of the sa account on a Microsoft SQL Server.  The steps in this article work in SQL 2005, 2008, 2008 R2, 2012, 2014 and 2016.
Polish reports in Access so they look terrific. Take yourself to another level. Equations, Back Color, Alternate Back Color. Write easy VBA Code. Tighten space to use less pages. Launch report from a menu, considering criteria only when it is filled…
This video shows how to remove a single email address from the Outlook 2010 Auto Suggestion memory. NOTE: For Outlook 2016 and 2013 perform the exact same steps. Open a new email: Click the New email button in Outlook. Start typing the address: …

760 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

Need Help in Real-Time?

Connect with top rated Experts

20 Experts available now in Live!

Get 1:1 Help Now