functie binnen functie

[Niellles]

Het volgende is wat ik in feite wil:

function CreateHash($password)
{
    $password = sha1(md5($password."sinterklaas"));
}

function NewUser($voornaam,$tussenvoegsels,$achternaam,$straat,$huisnr,$postcode,$email,$telefoon,$instrument)
{
    $wachtwoord = substr(md5($voornaam.$telefoon.rand()),0,5);
    $query = "INSERT INTO accounts (wachtwoord, voornaam, tussenvoegsels, achternaam, straat, huisnr, postcode, email, telefoon, instrument, access) VALUES ('".CreateHash($wachtwoord)."','$voornaam','$tussenvoegsels','$achternaam','$straat','$huisnr','$postcode','$email','$telefoon','$instrument','0')";
    mysql_query($query);
}

Op die manier werkt het niet... op de volgende wel:

function NewUser($voornaam,$tussenvoegsels,$achternaam,$straat,$huisnr,$postcode,$email,$telefoon,$instrument)
{
    $wachtwoord = substr(md5($voornaam.$telefoon.rand()),0,5);
    $wachtwoord = sha1(md5($wachtwoord."sinterklaas"));
    $query = "INSERT INTO accounts (wachtwoord, voornaam, tussenvoegsels, achternaam, straat, huisnr, postcode, email, telefoon, instrument, access) VALUES ('$wachtwoord','$voornaam','$tussenvoegsels','$achternaam','$straat','$huisnr','$postcode','$email','$telefoon','$instrument','0')";
    mysql_query($query);
}

Ik vermoed dat in het eerste stukje code het probleem zit in het feit dat ik een zelfgemaakte functie aanroep binnen een andere zelfgemaakte functie. Is dat zo? Wat doe ik daar aan?

Iemand die mij hiermee uit de brand kan helpen?
Opzich werkt het, maar ik vind het niet helemaal net.

Niels

 

[phobia]

haal die functie eens uit de query, zo

function CreateHash($password)
{
    $password = sha1(md5($password."sinterklaas"));
}
 
function NewUser($voornaam,$tussenvoegsels,$achternaam,$straat,$huisnr,$postcode,$email,$telefoon,$instrument)
{
    $wachtwoord = CreateHash(substr(md5($voornaam.$telefoon.rand()),0,5));
    $query = "INSERT INTO accounts (wachtwoord, voornaam, tussenvoegsels, achternaam, straat, huisnr, postcode, email, telefoon, instrument, access) VALUES ('$wachtwoord','$voornaam','$tussenvoegsels','$achternaam','$straat','$huisnr','$postcode','$email','$telefoon','$instrument','0')";
    mysql_query($query);
}

 

[SvU]

Maakt niks uit, de fout zit in de functie CreateHash.

function CreateHash($password)
{
$password = sha1(md5($password."sinterklaas"));
return $password;
}

Zo werkt ie wel.

Overigens doe je niks aan SQL Injection, lees je in in de functie mysql_real_escape_string().

 

[phobia]

ach ja tuurlijk hij returnde nix!!!
stom helemaal overheen gelezen!

 

[Niellles]

ach ja tuurlijk hij returnde nix!!!
stom helemaal overheen gelezen!

Kan gebeuren, had ik namelijk ook . (stom :s)
Bedankt!

mysql_real_escape_string gebruik ik altijd (zodra de functie werkt...), toch bedankt dat je me er even op attent maakt!