Rare Parse Error

Gestart doorBjarnootjuh15 reacties42 weergaven

16 berichten · pagina 1 van 2

Bjarnootjuh

Lid · Topicstarter

Berichten
7.806
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Hoi

Ik heb deze query:

mysql_query("INSERT INTO `visited` (`id` ,`offer_id` ,`user_id`)VALUES (, ".$_POST[oid].", $userinfo[id])") or die(mysql_error());


Maar ik krijg deze error:

Quote:
Parse error: syntax error, unexpected ", " (T_CONSTANT_ENCAPSED_STRING) in /home/ps/domains/slashbucks.com/public_html/offers.php on line 17
0
Berichten
13.482
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Je bent bij $userinfo[id] iets vergeten.
0
Bjarnootjuh

Lid · Topicstarter

Berichten
7.806
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Nu heb ik dit:

mysql_query("INSERT INTO `visited` (`id` ,`offer_id` ,`user_id`)VALUES (".$_POST[cid]", ".$_POST[oid].", ".$userinfo[id]")") or die(mysql_error());


zelfde error.
0
Berichten
13.482
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Je bent bij $userinfo[id] alweer iets vergeten.

toevoeging op 26 augustus 2012, 14:47:10:
Bij $_POST[cid] ook
0
Bjarnootjuh

Lid · Topicstarter

Berichten
7.806
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Sjoerd schreef:
Je bent bij $userinfo[id] alweer iets vergeten.
toevoeging op 26 augustus 2012, 14:47:10:
Bij $_POST[cid] ook

En wat dan als ik vragen mag?
0
Berichten
93.699
Reputatie
0
Punten
1,27
Lid sinds
Laatst gezien
De punten
0
Bjarnootjuh

Lid · Topicstarter

Berichten
7.806
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Dat werkt nu, nu deze:

Zegt: Unknown Column date_submitted in field list

Maar die column bestaat gewoon:

mysql_query("INSERT INTO `visited` (`id` ,`offer_id` ,`user_id` ,`date_submitted` ,`reward` ,`status` )VALUES (".$_POST[cid].", ".$_POST[oid].", ".$userinfo[id].",  NOW(),  ".$offer[reward].",  1)") or die(mysql_error());
0
Berichten
5.957
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Staat hij na user_id en voor reward? Ook geen hoofdletters gebruikt?
0
Berichten
10.421
Reputatie
4
Punten
0,00
Lid sinds
Laatst gezien
1) Gebruik geen die(). Jij gaat toch ook niet dood als je iets fout doet? Dus je script ook niet...
2) Je bent behoorlijk onveilig bezig. Iedereen kan met je mazzel je query injecteren met andere querys. Dit is SQL-injection en behoorlijk onveilig omdat je heel makkelijk gehacked kan worden.
0
Bjarnootjuh

Lid · Topicstarter

Berichten
7.806
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
O ja, moet mysql_real_escape_string doen.

Ga ik doen.

toevoeging op 26 augustus 2012, 16:02:08:
En ja, staat na user_id en voor Rewards
0
Berichten
10.421
Reputatie
4
Punten
0,00
Lid sinds
Laatst gezien
Haal die backticks ook eens weg... ````````````
0
Bjarnootjuh

Lid · Topicstarter

Berichten
7.806
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Hoezo moeten die weg? Veroorzaakt dat het probleem dan?
0
Berichten
35.796
Reputatie
1
Punten
0,00
Lid sinds
Laatst gezien
Bjarnootjuh schreef:
Dat werkt nu, nu deze:

Zegt: Unknown Column date_submitted in field list

Maar die column bestaat gewoon
Nee die bestaat niet, anders zegt MySQL het niet clown
Maak eens een screen van phpmyadmin ofzo.
toevoeging op 26 augustus 2012, 16:07:18:
Bjarnootjuh schreef:
Hoezo moeten die weg? Veroorzaakt dat het probleem dan?
Nee, maar je hebt er niks aan. Ze kunnen enkel voor meer problemen zorgen.
0
Berichten
10.421
Reputatie
4
Punten
0,00
Lid sinds
Laatst gezien
Het is niet aan te raden die dingen te gebruiken. Zorg altijd voor geldige veldnamen die niet conflicteren met MySQL functies. Zie de reserved words....
0
Bjarnootjuh

Lid · Topicstarter

Berichten
7.806
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Nope, werkt allemaal niet cry
0

Jouw reactie

Deel je idee, stel een vraag of help een ander verder.

Editor laden…

Praat mee met de One2xs-community.

Inloggen om te reageren