Lennard schreef:Ik had zostraks de variables buiten de quotes alleen dat werkte toen niet. Achteraf bleek dat ik twee keer $ip postte. Zal dus als ik weer thuis ben vanavondnde quotes weghalen.En doe eens wat met de tips die in dit topic gegeven worden, met deze code ga je geheid tegen enorme bugs en andere ellende aanlopen.
- variabelen buiten quotes
- getallen buiten quotes
- fix je uitlijning want dit is echt om te huilen
- werk aan je foutafhandeling en kijk waarom een query fout is i.p.v. random je query te veranderen in de hoop dat het dan wel werkt
toevoeging op 19 maart 2013, 10:25:22:
Rik schreef:Ten eerste test je in je ifs of een string kleiner is dan een andere string. Dat werkt natuurlijk niet. Gebruik hiervoor hetvolgende:
if(strlen($var) < 100) { }Waarbij de 100 uiteraard de minimale lengte is die je wil hebben.
Bovendien zou ik werken met een error array, en dan als laatste kijken of je errors hebt of niet. Als dit niet het geval is dan pas moet je de gegevens posten.
Nu doe je namelijk het eerste wat ik zeg al fout, daardoor voert ie telkens de else uit, en dat betekend dat hij dus de query probeert uit te voeren.
Dus ik zou voor alles gewoon f(strlen($var) < 100) { } moeten gebruiken? En dan geen miljoen elses zoals ik die nu heb? Dus zoiets: (ben op ipad dus kan het niet even netjes doen)
f(strlen($var) < 100) { }
f(strlen($var) < 100) { }
f(strlen($var) < 100) { }
//hier de query en een echo met dat het gelukt is.
Alleen doet hij nu nog precies hetzelfde, hij voert de query gewoon uit terwijl de naam niet eens 5 tekens is (of zelfs leeg).
