DIG Digital Corner
Alles wat je altijd al over computers, hardware, software, internet en elektronische gadgets had willen weten, maar niet op Tweakers.net durft te vragen.
ja nee ma nu kan zien wat het probleeem is he of ni ?? pls help! ty alvast!!
ik ben iemand die spel wil maken
jammer dat ik er verder geen verstand van ehb
Daar had ik zend niet voor nodig om dat te zienquote:Op zaterdag 13 december 2008 23:00 schreef Swetsenegger het volgende:
Nou ik heb hem ff in zend gepaste, maar mijn god wat een baggerscript. Het zit vol met fouten
Ik kan in ieder geval makkelijk curly brackets bekijken... en daar ontbreken er nogal wat van. En haakjes en quotes.quote:Op zaterdag 13 december 2008 23:01 schreef glasbak het volgende:
[..]
Daar had ik zend niet voor nodig om dat te zien
Waarschijnlijk iets met haakjes of accolades. En zonder indenting is dat nauwelijks na te gaan.quote:
ja nee ma nu kan zien wat het probleeem is he of ni ?? pls help! ty alvast!!
de sluitbracket op regel 56 matched met de opening van de functie user_register. Volgens mij is die hele if/else constructie verrot want er volgt een else op een else..quote:Op zaterdag 13 december 2008 23:02 schreef Swetsenegger het volgende:
[..]
Ik kan in ieder geval makkelijk curly brackets bekijken... en daar ontbreken er nogal wat van. En haakjes en quotes.
Dat kun je nog recht praten door te zeggen dat je regels niet te lang wilt laten worden.quote:
staan ook gewoon harde enters in echo's en dergelijke
quote:Op zaterdag 13 december 2008 21:49 schreef Swetsenegger het volgende:
zet het eens tussen php tags en pas eens iNdenting toe.....
Ik vind de eis dat mailadressen maximaal 50 tekens mogen bevatten wat vreemd... Domeinnamen mogen 63 tekens zijn, exclusief TLD.
| 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 | function user_register() { global $supersecret_hash_padding; if (strlen($_POST['user_name']) <= 25 && strlen($_POST['password1']) <= 25 && ($_POST['password1'] == $_POST['password2'] && strlen($_POST['email']) <= 50 && validate_email ($_POST['email']))) { if (account_namevalid($_POST['user_name']) || strlen($_POST['password1'] >= 6)) $user_name = strtolower($_POST['user_name']); $user_name = trim($user_name); $email = $_POST['email']; $query = "SELECT user_id FROM user WHERE user_name = '$user_name' AND email = '$email'"; $result = mysql_query($query); if ($result && mysql_nam_rows($result) > 0) { $feedback = 'ERROR-Username or email address already exists'; return $feedback; } else { $first_name = $POST['first_name']; $last_name = $POST['last_name']; $password = md5($_POST['password1']); $user_ip = $_SERVER['REMOTE_ADDR']; $has = md5($email.$supersecret_hash_padding); $query = "INSERT INTO user (user_name, first_name, last_name, password, email, remote_addr, confrim_hash, is_confrim, date__created) VALUES ('$user_name', '$first_name', '$last_name', '$password', '$email', '$user_ip', '$hash', '0', NOW())"; $result = mysql_query($query); if (!$result) { $feedback= 'ERROR-Datebase error'; return $feedback; } else { $encoded_email = urlencode($_POST['email']); $mail_body= " eomailbody Tank you for registering at Example.com. click this link to confrim your registration: http://localhost/confrim.php?hash=$hash&email=$encoded_email once you see a confrimation message, you will be logged into Example.com eomailbody"; mail ($email, 'example.com registration Confrimation', $email_body, 'from: noreply@example.com'); $feedback = ' YOU HAVE SUCCESFULLY REGISTERD. YOU WILL RECIVE A CONFRIMATION MAIL SOON'; return $feedback; } } } else { $feedback = 'ERROR-Username or password is invalid'; return $feedback; } } else { $feedback ='ERROR-Please fill in all fields correctly'; return $feedback; } } ?> |
Een meer leesbare versie van de eerste functie, waaruit het probleem ook snel duidelijk wordt. En dat met alleen het toepassen van indenting.
Light, je bent te goed voor deze wereld
In theory there is no difference between theory and practice. In practice there is.
snap het nog niet helemaal, behalve dat het van 113 regels naar 59 is gegaan....... wat betekenen die kleurtjes?
Rood zijn strings, blauw zijn de variablen en functies, groen is het overige.. if/else statements, for/while-loops etc etc.
Lees de php manual als je de taal wilt snappen
Lees de php manual als je de taal wilt snappen
In theory there is no difference between theory and practice. In practice there is.
ik moest lachen, klootzakquote:Op zaterdag 13 december 2008 20:43 schreef admiraal_anaal het volgende:
En een spelfout
EOMAILBODY; moet zijn emailbuddy
Jij ook?
Ik pakte maar een deel van het scriptquote:
snap het nog niet helemaal, behalve dat het van 113 regels naar 59 is gegaan....... wat betekenen die kleurtjes?
En die kleurtjes zijn om dingen makkelijk te kunnen herkennen... tekst is rood, reserved words en haakjes enzo zijn groen, namen van functies en variabelen zijn blauw. Dat maakt het al een stuk duidelijker dan wanneer alles zwart is. Je ziet bijvoorbeeld veel sneller of een string wel correct wordt afgesloten.
goedzo..quote:
en nu kan je zelf zien waar het fout gaat
SPOILER: kleine hintOm spoilers te kunnen lezen moet je zijn ingelogd. Je moet je daarvoor eerst gratis Registreren. Ook kun je spoilers niet lezen als je een ban hebt.
Zonder escaping je invoer in een SQL query rammen is geen puik plan trouwens.
Een dag zonder vlees is een dag niet geleefd
Dat doet ie ook helemaal niet voor zo ver ik kan zien, validate_email functie niet gezien? En hij valideert username en password, dus dat zit wel goed.
In theory there is no difference between theory and practice. In practice there is.
Ik heb me bijna door het hele script geworsteld, diverse quotes teveel en te weinig, curly brackets en haakjes te weinig opgelost. En zend gaat nu over zijn nek over de ereg in de validate emial functie. Nu zijn a. mijn reguliere expressies niet het best en doe ik dat b altijd met preg en niet met ereg.
dus wellicht kan een ander roepen waarom zend dit een unexpected string vind. Zal ook weer met quotes te maken hebben als ik het zo zie.
vanaf de quote na '@'
dus wellicht kan een ander roepen waarom zend dit een unexpected string vind. Zal ook weer met quotes te maken hebben als ik het zo zie.
| 1 2 3 | return (ereg('^[-!#$%&\'*\\./0-9=A-Z^_`a-z{|}~]+'. '@' '[-!#$%&\'*+\\/0-9=?A-Z^_`a-z{|}~]+\.' . '[-!#$%&\'*+\\./0-9=?A-Z^_`a-z{|}~]+$', $_POST[email'])); ?> |
vanaf de quote na '@'
Dat is sowieso het beste advies in dit topic.quote:Op zondag 14 december 2008 08:36 schreef Light het volgende:
Ik denk dat TS het beste bij het begin kan beginnen. Bij Hello world dus.
Hey,
Bedankt dat julie er zo mee bezig zijn..
Ma kan iemand de verder opgeloste versie poste (((swetsenegger))
tyty
Bedankt dat julie er zo mee bezig zijn..
Ma kan iemand de verder opgeloste versie poste (((swetsenegger))
tyty
ik ben iemand die spel wil maken
quote:Op zondag 14 december 2008 08:08 schreef Swetsenegger het volgende:
Ik heb me bijna door het hele script geworsteld, diverse quotes teveel en te weinig, curly brackets en haakjes te weinig opgelost. En zend gaat nu over zijn nek over de ereg in de validate emial functie. Nu zijn a. mijn reguliere expressies niet het best en doe ik dat b altijd met preg en niet met ereg.
dus wellicht kan een ander roepen waarom zend dit een unexpected string vind. Zal ook weer met quotes te maken hebben als ik het zo zie.
[ code verwijderd ]
vanaf de quote na '@'
| 1 2 3 | return (ereg('^[-!#$%&\'*\\./0-9=A-Z^_`a-z{|}~]+'. '@'.'[-!#$%&\'*+\\/0-9=?A-Z^_`a-z{|}~]+\.' . '[-!#$%&\'*+\\./0-9=?A-Z^_`a-z{|}~]+$', $_POST[email'])); ?> |
zoiets?
Nee joh, die bagger heb ik ondertussen al gedelete.quote:
Hey,
Bedankt dat julie er zo mee bezig zijn..
Ma kan iemand de verder opgeloste versie poste (((swetsenegger))
tyty
Ga php leren en kom dan maar eens terug.
Ik kan php ma ben nu met dit bezig en heb al een deel opgelost ma zie de fout niet in line 55
Ik zie niet wat die else daar verkeerd doet !!
Ik zie niet wat die else daar verkeerd doet !!
ik ben iemand die spel wil maken
quote:
Ik kan php ma ben nu met dit bezig en heb al een deel opgelost ma zie de fout niet in line 55
Ik zie niet wat die else daar verkeerd doet !!
|
|
| Forum Opties | |
|---|---|
| Forumhop: | |
| Hop naar: | |