[VB.Net] access db: record toevoegen

[DenDries]

Hallo,

Als onderdeel van mijn spelletje, zou ik graag de score toevoegen in een access database.

code:

'BEGIN SCORE WEGSCHRIJVEN
                Dim verbindingsString As String = "Provider=Microsoft.Jet.OLEDB.4.0;" & "Data Source=scores.mdb;"
                Dim cnnScores As New OleDb.OleDbConnection(verbindingsString)
                Dim query As String = "INSERT INTO tblScores(Spelernaam) VALUES(frmStart.strSpelernaam)"
                Dim opdracht As New OleDb.OleDbCommand(query, cnnScores)

                Try
                    cnnScores.Open()
                    opdracht.ExecuteNonQuery()
                    opdracht.Dispose()
                Catch x As Exception
                    MessageBox.Show(x.Message, "Fout", MessageBoxButtons.OK, MessageBoxIcon.Error)
                Finally
                    cnnScores.Close()
                    MessageBox.Show("Gegevens toegevoegd", "Info", MessageBoxButtons.OK, MessageBoxIcon.Information)
                End Try
                'EINDE SCORE WEGSCHRIJVEN

Nu zie ik nergens een fout in bovenstaande code, maar wanneer ik ze uitvoer krijg ik volgende error:
"No value give for one or more given parameters"

frmStart.strSpelernaam bevat nochtans de naam van de speler.


Weet er iemand wat ik fout doe? Alvast bedankt!

[MMUilwijk]

Mijn MS Access kennis is wat roesting, maar veelal wijst dit op of een verkeerde kolomnaam, of een verkeerd datatype.

Volgens mij moet je in een "tekst" kolom in je VALUES deel ook quotes om je waarde neerzetten?!
En ga aub gebruik maken van geparametrizeerde queries

[ Voor 14% gewijzigd door MMUilwijk op 01-06-2008 11:45 ]

[RobIII]

DenDries schreef op zondag 01 juni 2008 @ 11:36:
"INSERT INTO tblScores(Spelernaam) VALUES(frmStart.strSpelernaam)"

DenDries schreef op zondag 01 juni 2008 @ 11:36:
Weet er iemand wat ik fout doe? Alvast bedankt!

Heb je de query zoals 'ie uitgevoerd wordt al eens afgedrukt? Je kunt niet zomaar in je string een variabele zetten en hopen dat VB.net (of eender welke taal) dan snapt dat die vervangen moet worden door de waarde ervan. Je zult moeten escapen, maar om dan meteen slechte gewoontes te voorkomen kun je je denk ik beter even inlezen in Over het gebruik van Parametrized Queries. Ik zie trouwens ook niet waarom je alleen een naam zou inserten en niet meteen de score erbij.

Dan vraag ik me overigens ook nog even het volgende af: is je applicatie (primair) een spelletje? Waarom gebruik je dan een (Access) DB om scores op te slaan en niet gewoon een los bestandje?

Al met al een beetje basic. Een tutorialletje of 2 kan geen kwaad denk ik

[ Voor 31% gewijzigd door RobIII op 01-06-2008 11:46 ]

[DenDries]

RobIII schreef op zondag 01 juni 2008 @ 11:43:
[...]


[...]

Heb je de query zoals 'ie uitgevoerd wordt al eens afgedrukt? Je kunt niet zomaar in je string een variabele zetten en hopen dat VB.net (of eender welke taal) dan snapt dat die vervangen moet worden door de waarde ervan. Je zult moeten escapen, maar om dan meteen slechte gewoontes te voorkomen kun je je denk ik beter even inlezen in Over het gebruik van Parametrized Queries. Ik zie trouwens ook niet waarom je alleen een naam zou inserten en niet meteen de score erbij.

Dan vraag ik me overigens ook nog even het volgende af: is je applicatie (primair) een spelletje? Waarom gebruik je dan een (Access) DB om scores op te slaan en niet gewoon een los bestandje?

Al met al een beetje basic. Een tutorialletje of 2 kan geen kwaad denk ik

Het is een project voor school en we moeten verplicht gebruik maken van een access database om de gegevens bij te houden.

Allesinds bedankt, ik weet min of meer hoe ik hier in VS2003 gebruik kan van maken, maar in 2005 lukt het me nog niet echt.
Ik ga inderdaad eens wat tutorials lezen, 't is nodig

[SierdW]

DenDries schreef op zondag 01 juni 2008 @ 11:36:

code:

Dim query As String = "INSERT INTO tblScores(Spelernaam) VALUES(frmStart.strSpelernaam)"

Daar zou ik het volgende van maken, maar geen idee of dat in VB.Net ook zo moet

Visual Basic:

Dim query As String = "INSERT INTO tblScores(Spelernaam) VALUES('" & frmStart.strSpelernaam & "')"

/edit zie bovenstaande reacties...

[ Voor 12% gewijzigd door SierdW op 01-06-2008 13:50 ]

[ATS]

AU! Dat zou ik toch niet doen, SierdW!
Dit soort dingen zijn een recept voor SQL injection.
Queries bouw je bij voorkeur op als parameter queries, niet als strings. Dat heeft als voordeel dat dingen als correct escapen en vragen van "wat zet je nu ook alweer om een date field heen" automatisch opgelost worden.

[Niemand_Anders]

Een beetje offtopic: maar moet je behalve de spelersnaam ook niet de score opslaan in tblScores?

[DenDries]

Niemand_Anders schreef op maandag 02 juni 2008 @ 13:41:
Een beetje offtopic: maar moet je behalve de spelersnaam ook niet de score opslaan in tblScores?

Ja, 't was gewoon om eens te proberen met een tabel of dat lukte

Ik heb geprobeerd en kwam op dezelfde methode als SierdW uit, bedankt voor de hulp allemaal

[kenneth]

Tot er een speler komt die Jeanne d'Arc heet ... toen werkte het niet meer.

Lees RobIII in "\[VB.Net] access db: record toevoegen" nog eens ...

[whoami]

ja, of een speler die melp';delete from tblscores;-- heet ...

[SierdW]

ATS schreef op maandag 02 juni 2008 @ 13:36:
AU! Dat zou ik toch niet doen, SierdW!
Dit soort dingen zijn een recept voor SQL injection.
Queries bouw je bij voorkeur op als parameter queries, niet als strings. Dat heeft als voordeel dat dingen als correct escapen en vragen van "wat zet je nu ook alweer om een date field heen" automatisch opgelost worden.

Jaja, daar ben ik me van bewust, wat ik echter geef is hoe de query van TS wel zou werken.

[CodeCaster]

whoami schreef op maandag 02 juni 2008 @ 17:25:
ja, of een speler die melp';delete from tblscores;-- heet ...

Little Bobby Tables?

[SierdW]

CodeCaster schreef op dinsdag 03 juni 2008 @ 11:40:
[...]

Little Bobby Tables?

offtopic:

[ Voor 5% gewijzigd door SierdW op 03-06-2008 12:42 ]