Php Onverklaarbare false if

[Verwijderd]

Ik ben een beetje ten einde raad wat het probleem kan zijn met dit gevalletje...

Stukje code:

PHP:

public function setNewPassword($password, $repeatedPassword){
        if(empty($password))                return Admin::PASSWORD_NOT_FILLED;
        if(strlen($password) < 5)           return Admin::PASSWORD_TO_SHORT;

        FB::info($password.', '.$repeatedPassword);
        
        //if($password != $repeatedpassword)  return Admin::PASSWORD_INCORRECT_REPEATED;
        $this->password = md5($password);

        return True;
    }

$password en $repeatedPassword zijn hetzelfde... Controleer dit nog eens extra met FB::info...
Toch geeft hij return PASSWORD_INCORRECT_REPEATED

Als ik $password = $password doe geeft hij uiteraard wel True.

Hoe kan het dat deze twee net gelijk zijn terwijl ze daadwerkelijk hetzelfde ingegeven zijn in de fields

[djluc]

spaties? Doe eens een var_dump over beide apart?

[Guillome]

1 = is natuurlijk een asignment, let daar mee op.
Hoe roep je de functie aan?
Je hebt die controle gecomment he?
Anders gebruik je strcomp

"... deze twee net gelijk... " ? net of niet?

[ Voor 14% gewijzigd door Guillome op 12-04-2010 18:47 ]

[Verwijderd]

zijn php variabelen niet case sensitive?

code:

if($password != $repeatedPassword)  return Admin::PASSWORD_INCORRECT_REPEATED;

[Verwijderd]

Code comment was nog van een testje, uiteraard is hij nu weer uncomment.

Deze functie wordt door de AdminManager opgeroepen:
$admin = new Admin($id);
$vNewPassword = $admin->setNewPassword($in['password'], $in['repeatedPassword']);

if($vNewPassword !== True) return $vNewPassword;

Ga even op php.net strcomp zoeken en proberen, laat zometeen het resultaat weten

[EDIT]Nee variabelen zijn niet case-sensitive en zelfs al zouden ze zijn waren ze correct... helaas...[/code]

[ Voor 13% gewijzigd door Verwijderd op 12-04-2010 18:52 ]

[Guillome]

En wat djluk zegt?

[Verwijderd]

met strcomp werkt het wel... Al wil ik wel graag weten waarom if het niet doet...

Ga nu de vardump doen.

[Verwijderd]

Vardump resultaat: string(4) "test" string(4) "test"

Exact hetzelfde dus...

[ Voor 20% gewijzigd door Verwijderd op 12-04-2010 18:58 ]

[djluc]

strcmp doet valles, o.a. entities en locales verwerken waardoor dat niet een handige tool is om een 1 op 1 vergelijk te doen, je wilt gewoon 2 strings vergelijken en dat doe je met == of === als de types ook nog zeker gelijk zijn en je dat wilt checken.

[edit]Als dat gelijk is dan zou ik je aanroep checken.

[ Voor 10% gewijzigd door djluc op 12-04-2010 18:59 ]

[WouZz]

Verwijderd schreef op maandag 12 april 2010 @ 18:51:
[EDIT]Nee variabelen zijn niet case-sensitive en zelfs al zouden ze zijn waren ze correct... helaas...[/code]

wel dus

[Verwijderd]

Nu doet hij het ineens wel... Zeer raar... Volgens mij niets veranderd aan de code voor zover ik zie.

PHP:

public function setNewPassword($password, $repeatedPassword){
        if(empty($password))                return Admin::PASSWORD_NOT_FILLED;
        if(strlen($password) < 5)           return Admin::PASSWORD_TO_SHORT;
        
        FB::info($password.', '.$repeatedPassword);

        if($password != $repeatedPassword) return Admin::PASSWORD_INCORRECT_REPEATED;
        
        $this->password = md5($password);

        return True;
    }

Verder geen andere bestanden op gehad dus begrijp nu niet waarom het nu ineens wel werkt...

[Verwijderd]

case sensitive ...


originele post:
if($password != $repeatedpassword) return Admin::PASSWORD_INCORRECT_REPEATED;

werkende post:
if($password != $repeatedPassword) return Admin::PASSWORD_INCORRECT_REPEATED;

[ Voor 4% gewijzigd door Verwijderd op 12-04-2010 19:06 ]

[Voutloos]

$repeatedpassword bestond inderdaad niet.

Dit had je overigens kunnenmoeten zien op 1 van de volgende manieren:
1 - Een E_NOTICE over de variabele met typo
2 - Een indicatie in je IDE.

offtopic:
Offtopic feest, laat de vaste 42 posts maar komen: Hashen in een security context moet _altijd_ met salt.

[ Voor 22% gewijzigd door Voutloos op 12-04-2010 19:08 ]

[WouZz]

Tip: error_reporting(E_ALL)

[RobIII]

djluc schreef op maandag 12 april 2010 @ 18:58:
strcmp doet valles, o.a. entities en locales verwerken

Ik heb hier geen PHP bij de hand, maar stcmp('é','é') geeft dus 0 (aka equal) Dat lijkt me behoorlijk ongewenst gedrag (hoewel het vaak wel handig kan zijn, maar dan wil ik dat of specificeren middels een extra (optionele) parameter, of een andere functie daarvoor hebben, of gewoon lekker ouderwets zelf regelen voor ik strcmp aanroep).

[ Voor 34% gewijzigd door RobIII op 12-04-2010 19:18 ]

[HuHu]

WouZz schreef op maandag 12 april 2010 @ 19:07:
Tip: error_reporting(E_ALL)

Betere tip: error_reporting(E_ALL | E_STRICT)

[Voutloos]

Nog betere tip: error_reporting(-1)

Dan heb je alle errors, ongeacht nieuwe/andere constantes in de toekomst. En ja, er is geen constante voor deze magische -1, dat zou namelijk te logisch zijn.

[Guillome]

Nog betere tip: voutloos programmeren!

[ctrlaltdelbe]

Guillome schreef op maandag 12 april 2010 @ 20:19:
Nog betere tip: voutloos programmeren!

Dat wil ik je wel een keer zien doen

[Guillome]

PHP:

echo "Hello world;

[WouZz]

Guillome schreef op maandag 12 april 2010 @ 22:26:

PHP:

echo "Hello world;

Ik ben trots op je!

Parse error: syntax error, unexpected $end, expecting T_VARIABLE or T_DOLLAR_OPEN_CURLY_BRACES or T_CURLY_OPEN in /Users/wouter/hello.php on line 4

[4Real]

Tip van flip: Gooi altijd boven in je binnenkomst bestand het volgende:

PHP:

error_reporting(E_ALL);

Die helpt je een beetje met debuggen

[Guillome]

Hoe is dat nou mogelijk wouter? shit

[CH4OS]

Guillome schreef op maandag 12 april 2010 @ 22:47:
Hoe is dat nou mogelijk wouter? shit

Niet van de LOI...

[ctrlaltdelbe]

WouZz schreef op maandag 12 april 2010 @ 22:37:
[...]


Ik ben trots op je!


[...]

I rest my case

[Guillome]

Geen lol?

[MueR]

PHP:

return Admin::PASSWORD_TO_SHORT;

En zonder typo:

PHP:

return Admin::PASSWORD_TOO_SHORT;

[Guillome]

Beetje laat maar scherp!