[VB.NET + SQL] String concatenation werkt niet

[Praetorian]

Ik heb iets ontdekt in VB.NET waarvan ik vermoed dat het mogelijk een bug is. Als ik een SQL query aan het opbouwen ben stopt VB.NET op een gegeven moment met het opbouwen van de string. Voorbeeld:

Visual Basic .NET:

Dim strQuery As String = ""
Dim strEmail As String = ""
Dim strMessage As String = ""
Dim adoConnection As ADODB.Connection = New ADODB.Connection

strEmail = "[e-mail]"
strMessage  = "Dit is een bericht"

adoConnection.ConnectionString = "DSN=eendsn; UID=iets; PWD=nogiets"
adoConnection.Open()

strQuery = "INSERT INTO Tabelnaam (veld1, veld2, veld3, veld4) VALUES ('waarde1', 'waarde2', '" & strEmail & "', '" & strMessage & "')"

adoConnection.Execute(strQuery)

adoConnection.Close
adoConnection = Nothing

Nu gaat hij op z'n bek met de melding dat m'n query een "unterminated string constant" bevat. Als ik de waarde van strQuery bekijk is hij alles t/m strMessage, maar "')" wordt er niet aan vast geplakt. Als ik het gedeelte dat ik aan strQuery toeken watch geeft hij daarentegen wel de volledige waarde.

Heeft iemand enig idee of hier een workaround voor is? Ik heb al verschillende dingen geprobeerd, maar ik krijg het niet aan de praat. Dit probleem heb ik alleen in VB.NET, niet in VB6 of ASP (waarin ik al honderden soortgelijke queries heb opgebouwd).

[ Voor 19% gewijzigd door Praetorian op 06-03-2003 19:06 ]

[gorgi_19]

Visual Basic .NET:

Private Const strQuery as string = "INSERT INTO Tabelnaam (veld1, veld2, veld3, veld4) VALUES ('waarde1', 'waarde2', '{0}', '{1}')"

....

adoConnection.Execute(String.Format(strQuery,strEmail,strMessage))

ff kijken of dit zo iig klopt..

Geloof het wel..

Maar erhm.. Wat zegt je tracing als je hem net voor je adoConnection.Execute er uit gooit?
Dus

Visual Basic .NET:

Context.Current.Trace.Write(strQuery)
' Je mag zelf even de goede namespace opzoeken.. :)

[ Voor 54% gewijzigd door gorgi_19 op 06-03-2003 19:03 ]

[pjonk]

Vreemd de string is ook niet dermate lang dat hij out of memory zou kunnen raken.
Misschien is er een limiet aan het aantal concats die je mag doen?
En als je die & "')" eens toevoegt aan strQuery?

[gorgi_19]

JonkieXL schreef op 06 March 2003 @ 19:05:
Misschien is er een limiet aan het aantal concats die je mag doen?

Afaik is die er niet. Het enige nadeel is dat hij op een bepaald moment ontzettend traaaaaaag gaat worden. Daarom hebben ze ook de stringbuilder uitgevonden.

[Praetorian]

gorgi_19 schreef op 06 March 2003 @ 19:00:

Visual Basic .NET:

Private Const strQuery as string = "INSERT INTO Tabelnaam (veld1, veld2, veld3, veld4) VALUES ('waarde1', 'waarde2', '{0}', '{1}')"

....

adoConnection.Execute(String.Format(strQuery,strEmail,strMessage))

ff kijken of dit zo iig klopt..

Geloof het wel..

Maar erhm.. Wat zegt je tracing als je hem net voor je adoConnection.Execute er uit gooit?
Dus

Visual Basic .NET:

Context.Current.Trace.Write(strQuery)
' Je mag zelf even de goede namespace opzoeken.. :)

Dit geeft helaas het zelfde negatieve resultaat.....
Ik heb het nu even in VB6 gedaan, en daar gaat het wel goed.....

[gorgi_19]

En wat zegt je tracing?

[whoami]

Ipv je SQL string zo op te bouwen, is het veel beter en veiliger als je werkt met parameters.

Bv:
_C#

code:

SqlConnection = new SqlConnection(connstring);

SqlCommand cmd = new SqlCommand ();
cmd.Connection = conn;

cmd.CommandText = "INSERT INTO tabel (VELD1, VELD2) VALUES (@p_veld1, @p_veld2);

cmd.Parameters.Add ("@p_veld1", SqlDbType.Int);
cmd.Parameters.Add ("@p_veld2", SqlDbType.VarChar);

cmd.Parameters["@p_veld1"].Value = 2;
cmd.Parameters["@p_veld2"].Value = test;
cmd.ExecuteNonQuery();