login  Naam:   Wachtwoord: 
Registreer je!
 Forum

gegevens in mysql wegschrijven

Offline notf - 20/12/2006 14:36
Avatar van notfOnbekend hallo,
ik heb het volgende script om mijn gegevens toe te voegen aan een database:
http://www.plaatscode.be/3510/
maar er zit blijkbaar nog een fout in. Hij geeft geen foutmelding en schrijft ook niet weg in de database  
kan iemand mij hierbij helpen?

alvast bedankt
Ruben

18 antwoorden

Gesponsorde links
Offline vinTage - 20/12/2006 14:41
Avatar van vinTage Nieuw lid Weleke error ?

En beveilig je inputs 
Offline notf - 20/12/2006 14:42
Avatar van notf Onbekend geen errors 
ik wil eerst de code laten werken en dan kan ik er over gaan denken om mijn inputs te gaan beveiligen  
Offline vinTage - 20/12/2006 14:45
Avatar van vinTage Nieuw lid knal er dan eens een or die(errormelding van sql) achter je insertqry 
Offline Gerard - 20/12/2006 14:46
Avatar van Gerard Ouwe rakker How about je leert eerst fatsoenlijk coden voordat je zoiets maakt ;)

Die password generate kan makkelijk in een functie.
$password = $password . nogwat
kan vervangen worden door
$password .= nogwat

Beveilig EERST je input voordat je verdergaat. En maak geen onnodige variabelen aan als je er toch voor de rest geen fuck mee doet. Verder moet je ook eens je queries duidelijk vorm geven zodat je ze ook fatsoenlijk kan lezen.
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  2. $query = mysql_query("INSERT INTO leden (veld1, veld2, veld3, veld4,
  3.                                          veld5, veld6)
  4.                       VALUES ( " . intval($_POST['id']) . ",
  5.                               '" . mysql_real_escape_string($_POST['veld2']) . "',
  6.                               '" . mysql_real_escape_string($_POST['veld3']) . "',
  7.                               '" . mysql_real_escape_string($_POST['veld4']) . "',
  8.                               '" . mysql_real_escape_string($_POST['veld5']) . "',
  9.                               '" . mysql_real_escape_string($_POST['veld6']) . "')");
Offline Kr4nKz1n - 20/12/2006 14:48
Avatar van Kr4nKz1n Onbekend 1: Waarom onnodige variable? pleur bij de mysql query gewoon $_POST['naam'] erin,
2: Waarom id_leden? Gewoon weg halen, dit is een autonummer en hoeft hier niet te staan,
Offline vinTage - 20/12/2006 14:49
Avatar van vinTage Nieuw lid
Citaat:
2: Waarom id_leden? Gewoon weg halen, dit is een autonummer en hoeft hier niet te staan,


mss heeft hij zn db niet op autoincreament* staan ? 



*schrijf je dat zo ?
Offline Gerard - 20/12/2006 14:51 (laatste wijziging 20/12/2006 14:51)
Avatar van Gerard Ouwe rakker @jvriet
AUTO_INCREMENT 
Offline notf - 20/12/2006 14:52
Avatar van notf Onbekend mijn database staat op auto_increment, maar had zo van alles zitten proberen om het toch maar te laten werken. En buiten het feit dat mijn code een beetje slordig geordend is wil toch nog niet zeggen dat het niet hoord te werken?  
Offline Kr4nKz1n - 20/12/2006 14:53
Avatar van Kr4nKz1n Onbekend
Citaat:
mss heeft hij zn db niet op autoincreament* staan ?
Hij voegt dan een lege id toe!
Offline Gerard - 20/12/2006 14:53 (laatste wijziging 20/12/2006 14:55)
Avatar van Gerard Ouwe rakker Het feit dat jouw code slordig is en niet geordend heeft misschien niet tot gevolg dat het resultaat anders is. Maar zoals je nu ziet is het debuggen van jouw spaghetti een stuk moeilijker dan wanneer je het wel goed geordend had.

Citaat:
Hij voegt dan een lege id toe!

Nee, wanneer je een lege string ('') of een null waarde naar de database toe gooit voor een auto_increment veld, dan zal hij gewoon de waarde van auto_increment pakken. Pas als je echt een goede integer waarde mee gaat geven zal hij deze pakken.
Offline Kr4nKz1n - 20/12/2006 14:55 (laatste wijziging 20/12/2006 15:09)
Avatar van Kr4nKz1n Onbekend Proximus leer lezen man.
Er werd toch vertelt dat hij dat niet had!

eigenwijs :S
Offline Gerard - 20/12/2006 14:57
Avatar van Gerard Ouwe rakker @Kr4nkz1n
Dus ook in dit topic loop je al te schelden. Leer jij maar eens lezen.

Citaat:
mijn database staat op auto_increment
Offline notf - 20/12/2006 14:59 (laatste wijziging 20/12/2006 15:01)
Avatar van notf Onbekend voor de mensen die willen, hier is een iets meer ordelijke code:
http://www.plaatscode.be/3512/
maar dit heeft nog altijd hetzelfde effect als eerst! 

*had juist foute link, nu is het juiste 
Offline vinTage - 20/12/2006 15:00 (laatste wijziging 20/12/2006 15:01)
Avatar van vinTage Nieuw lid nog steeds or die(errormeuk) achter je insert qry zetten ;)


ik wist die errormeuk niet uit mn kop, maar dit is em:

or die(mysql_error())
Offline Kr4nKz1n - 20/12/2006 15:01
Avatar van Kr4nKz1n Onbekend
Citaat:
Dus ook in dit topic loop je al te schelden. Leer jij maar eens lezen
Wat valt er allemaal bij jou onder schelden? En daarnaast was mijn reactie gebaseerd op iemands anders reactie.
Offline Gerard - 20/12/2006 15:01 (laatste wijziging 20/12/2006 15:02)
Avatar van Gerard Ouwe rakker Probeer eens te werken met PHP.net: mysql_error. Kijk eens wat voor eventuele foutmelding je terugkrijgt van MySQL.

Citaat:
Wat valt er allemaal bij jou onder schelden? En daarnaast was mijn reactie gebaseerd op iemands anders reactie.


Oke, flamen dan. Daarnaast zie ik toch echt mijn nickname in de reactie staan. Gek dat je nu dan zegt dat het niet over mijn reactie ging.
Offline Kr4nKz1n - 20/12/2006 15:04 (laatste wijziging 20/12/2006 15:05)
Avatar van Kr4nKz1n Onbekend
Citaat:
jvriet: mss heeft hij zn db niet op autoincreament* staan ?

Citaat:
Kr4nKz1n: Hij voegt dan een lege id toe!


Dat zie ik staan. Maar ik zal wel blind zijn dan.
Offline Gerard - 20/12/2006 15:06
Avatar van Gerard Ouwe rakker Leuk dat je ook duidelijk maakt dat jouw reactie van net weer ging over die van jou op vintage en niet die van mij op jou.
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.204s