SQL erkennt Variable als Spaltenname [Insert Into]

Marcel1997 · 20. August 2014, 18:05

Wo ihr grade dabei seid. Ich programier so PHP sachen und so ein bisschrn unf bisher hab ich immer in php einfach
$com = "INSERT INTO `tabelle` (`spalte1`, `spalte2`) VALUES ('".$input1."', '".$input2."')";

VB:
String command = "INSERT INTO `tabelle` (`spalte1`, `spalte2`) VALUES ('"&textbox1.Text&"', '"&textbox2.Text&"')"

Ist das so in der form nicht auch möglich in VB? Wenn nein Wieso?

BlackNetworkBit · 20. August 2014, 18:13

In Vb.Net werden Strings nicht mit "+"zusammen gefügt, sondern mit einem "&"
ansonsten würde dein Beispiel auch funktionieren

Marcel1997 · 20. August 2014, 18:15

Oke sorry dacht das is so wie bei c# okey thx

Schnickalodeon · 21. August 2014, 19:19

Ich bekomme immernoch die Fehlermeldung

Falsche Syntax in der Nähe von '?'.

Hier nochmal der Code:

VB.NET-Quellcode

Imports System.Data.Sql
Imports System.Data.SqlClient
Public Module SqlCeExtensions
''' <summary> allgemeine Methode, aus einer SqlCeConnection, einem CommandText und optionalen Werten
''' ein parametrisiertes SqlCeCommand zu erstellen </summary>
''' <remarks>Vorrausgeetzt ist, dass die übergebenen values Datentypen haben, die zu den in der Datenbank
''' festgelegten Spalten passen. Eine Überprüfung findet nicht statt.</remarks>
<Runtime.CompilerServices.Extension()> _
Public Function CreateCommand(ByVal con As SqlConnection, ByVal commandText As String, ByVal ParamArray values As Object()) As SqlCommand
Dim splits = commandText.Split("?"c) ' Parameter-Platzhalter '?' identifizieren
If splits.Count <> values.Length + 1 Then Throw New ArgumentException( _
"Anzahl der Argumente passt nicht zur Anzahl der Parameter-Platzhalter im CommandText")
Dim cmd = New SqlCommand()
' für jeden Platzhalter einen DbParameter adden, und den Param-Namen an den Platzhalter schreiben
For i = 0 To values.Length - 1
Dim param = cmd.Parameters.AddWithValue("@a" & i, values(i))
splits(i) = splits(i) & param.ParameterName
Next
cmd.CommandText = String.Concat(splits)
cmd.Connection = con
Return cmd
End Function
End Module
Public Class Hauptfenster
Private Sub bAnlegen_Click(sender As Object, e As EventArgs) Handles bAnlegen.Click
Dim dbNachname As String = "Nachname"
Dim dbVorname As String = "Vorname"
Dim dbAnmeldedatum As String = "Anmeldedatum"
Dim dbErinnerungsdatum As String = "Erinnerungsdatum"
Dim sqlCon As New SqlConnection With {.ConnectionString = "Server=NICK-PC\SQLEXPRESS;Database=Fahrschüler;Trusted_Connection=Yes;"} ' Erwstelt Connectiostring für SQL Server
Dim querryString As String = "INSERT INTO Fahrschüler (Nachname, Vorname, Anmeldedatum, Erinnerungsdatum) VALUES (?,?,?,?)"
Dim command As SqlCommand = sqlCon.CreateCommand(querryString, dbNachname, dbVorname, dbAnmeldedatum, dbErinnerungsdatum)
SQLCon.Open() ' Öffnet Connection zu SQL Server und der Datenbank
Dim Ds As DataSet = New DataSet()
Dim da As New SqlClient.SqlDataAdapter(querryString, SQLCon)
da.Fill(Ds, "Fahrschüler")
MessageBox.Show("Abgeschlossen")
SQLCon.Close() ' Schließt die Verbindung
End Sub
End Class

Schamash · 21. August 2014, 19:23

Scheib es mal so:

VB.NET-Quellcode

Dim command As SqlCommand = sqlCon.CreateCommand( "INSERT INTO Fahrschüler (Nachname, Vorname, Anmeldedatum, Erinnerungsdatum) VALUES (?,?,?,?)", dbNachname, dbVorname, dbAnmeldedatum, dbErinnerungsdatum)

Schnickalodeon · 21. August 2014, 19:28

Was schreibe ich dann in den SqlDataAdapter? querry String geht dann ja nicht mehr

ErfinderDesRades · 21. August 2014, 19:29

das anners schreiben wird auch nix bringen - guck nochma genau: Du erstellst nu ein schönes Command mit meiner schönen Extension - benutzt das Command aber garnicht, sondern erzeugst den DataAdapter ganz anners.
Sieh nochmal nach inne Doku, wie man einen DataAdapter mit einem Command erzeugt, statt mit einem String.

Edit: Frag lieber nicht, sondern sieh selbst nach (oder frag, wie man sowas nachsieht).
Es ist unabdingbar total wichtig, dass du die Programmierwerkzeuge, die dir zur Verfügung stehen, kennen- und nutzen lernst.

Schnickalodeon · 21. August 2014, 19:57

Ich finde in der Doku den Teil leider nicht, indem das steht

ErfinderDesRades · 22. August 2014, 02:18

Was verstehst du unter Doku?

Guck dir mal folgendes an: Visual-Studio richtig nutzen

zb der ObjectBrowser hat eine ganz einfache Suche-Funktion, in der man einen Klassennamen eingibt, und bekommt die vollständige Doku all ihrer Member angezeigt - was ein Klassen-Member ist, weißt du?
Das sind Methoden, Properties, Events oder Felder.
Was ein Feld ist, weißt du?
Was eine Methode ist, weißt du? das ist entweder eine Sub oder eine Function.

_Arthur_ · 22. August 2014, 08:39

Hi Leute,

ich weiß, ich bin ein bischen spät, aber ich habe diese Problemstellung immer beinahe so gelöst, wie von WhitePage vorgeschlagen, allerdings mit den erforderlichen Hochkommata im String (die DateAndTimePicker wären dann ein weiterer Vorschlag):

VB.NET-Quellcode

Dim DBNachname As String = Me.tbNachname.Text
Dim DBVorname As String = Me.tbVorname.Text
Dim DBAnmeldedatum As Date = dtpAnmeldung.Value 'DateAndTimePicker
Dim DBErinnerungsdatum As Date = dtpErinnerung.Value 'DateAndTimePicker
...
Dim da As New SqlClient.SqlDataAdapter("INSERT INTO Fahrschüler (Nachname, Vorname, Anmeldedatum, Erinnerungsdatum)
VALUES ('" & DBNachname & "','" & DBVorname & "','" & DBAnmeldedatum & "','" & DBErinnerungsdatum & "')", SQLCon)
...

Das erscheint mir als die am wenigsten komplizierte Lösung. Und funktionieren tut sie auch noch ...

HTH

Ciao Arthur

Schnickalodeon · 22. August 2014, 10:51

Yeah es hat geklappt

Vielen Dank Arthur

Dann hatte ich also nur eine falsche Syntax bei meinem 1. Verusch.

Trotzdem Vielen Dank ErfinderDesRades für die Mühe. Ich bin mir sicher, dass deine Methode sicherer und störungsunanfälliger ist, jedoch bin ich noch nicht soweit um sie zu verstehen Ich glaube um so zu denken, dass ich dich verstehe brauche ich noch ein paar Jahre Programmererfahrung oder jemand, der sich neben mich setzt. Tausend Dank für deinen Versuch.

An Alle die mir versucht haben zu Helfen: VIELEN,VIELEN DANK

Gruß Schnickalodeon

ErfinderDesRades · 22. August 2014, 10:58

Gradezu Schade, dasses geklappt hat, weil nun wirst du dir vmtl. nicht mehr die Mühe machen, die in post#29 aufgeworfenen Fragen für dich zu bearbeiten.
Mit dem Ergebnis, dass du hier nur 1% von dem gelernt hast, was du eiglich hättest lernen können (wenn überhaupt).
Und natürlich wurstelst du weiter ohne DbParameter, also prinzipiell unverantwortlich unsicher.
naja - das nächste Problem kommt bestimmt, weil das eiglich notwendige Progger-KnowHow haste hier ja nun verpasst.

mmgg · 25. August 2014, 16:01

Hallo Arthur,
wenn du es nicht gepostet hättest, hät ichs getan, man kann den glauben an die menschheit verlieren, wenn man auf diesen thread trifft.

@erfinder,
du wolltest nur helfen, nich wahr?

timmi31061 · 25. August 2014, 17:33

Die Lösung von @_Arthur_ ist gefährlich. Was passiert, wenn ich als Nachnamen ','','',''); DROP TABLE Fahrschüler; -- angebe? Richtig alle Daten sind gelöscht. Also bleibe bei PreparedStatements.
Zeile 42 ist falsch. Anstatt querryString und SQLCon musst du dort deinen SqlCommand übergeben.

mmgg · 26. August 2014, 21:41

Themenforum ist keine Bildungseinrichtung, wo ich mich für einen Kurs einschreibe und der lehrer vorgibt was zu lernen ist.
Wenn das hier so sein soll, - ob dann der User das vorher wissen solte/darf, damit er entscheiden kann, so wie er es tut wenn er einen kurs besucht?

der TE will es so machen, macht es so in PHP, ihn nun wissen zu lassen wie es geht PLUS Zusatz wie es besser geht, - Das bringt keiner fertig von all den anderen Antwortgebern die sich hie rgemeldet haben???

Und ich will gar nicht wissen warum, allein dass man diese Frage stellen kann, reicht

SQL erkennt Variable als Spaltenname [Insert Into]

VB.NET-Quellcode

VB.NET-Quellcode

VB.NET-Quellcode

Tags

Ähnliche Themen

12 Benutzer haben hier geschrieben