Hallo,
Wie zou me kunnen helpen met mijn Wachtwoord verander script?
Hij werkt niet goed.

<?php
if (isset($_POST['ww']))
    {
        
        if (empty($_POST['o_ww']))
        {
            
            $error='Vul je wachtwoord in';
            
        }else{
            $wachtwoordoud = $_POST['o_ww'];
            $qQuery = $mysql->query("SELECT 1 FROM users WHERE id = '". $id ."' AND password = '" . $wachtwoordoud . "'");
            
            if ($mysql->num_rows($qQuery) == 0)
            {
                
                $error='Je wachtwoord klopt niet';
                
            }
            
        }
        
        if (empty($_POST['ww']))
        {
            
            $error='Vul een nieuw wachtwoord in';
            
        }else{
            
            if ($_POST['ww'] != $_POST['ww2'])
            {
                
                $error='De wachtwoorden zijn niet gelijk';
                
            }
            
        }else{
            $wachtwoord = $_POST['ww'];
            mysql_query("UPDATE users SET password = '" . $wachtwoord . "' WHERE id = '". $id ."'");
            
            echo '<strong>Je wachtwoord is veranderd!</strong>';
            
        }else{
            
            echo 'De volgende fouten zijn opgetreden:<br />';
            
            echo $error;
            
        }
        
    }
    ?>  

Wie zou even willen kijken?
Alvast bedankt.
Het script is sowieso niet veilig voor sql injection.
Maar wat gaat er precies fout? Wat staat er op je scherm?

Meer info aub...
er komt niks op mijn scherm, ik weet dat hij niet goed is daarom vraag ik iemand er naar te kijken.
Die tweede else is niet goed.
Ik vraag ook niks over Sql Injection...
Er komt niks op je scherm? Ook geen parse error?
Je else op regel 43 is het einde van een else op regel 37, dat klopt natuurlijk niet.

Roel van de Water op 15/08/2011 11:55:23

Er komt niks op je scherm? Ook geen parse error?
Je else op regel 43 is het einde van een else op regel 37, dat klopt natuurlijk niet.



Er komt misschien wel wat op het scherm, zoals ik al zei ik weet dat er iets fout is.

Je else op regel 43 is het einde van een else op regel 37, dat klopt natuurlijk niet.

Was ik ook al achter..

Als je om hulp vraagt, dan mag je ook best wat aardiger overkomen.
Ik was eigenlijk van plan om je een goede code te sturen, maar als je op zo'n manier reageert, dan doe ik daar de moeite niet eens meer voor :)
Roel van de Water op 15/08/2011 11:59:34

Als je om hulp vraagt, dan mag je ook best wat aardiger overkomen.
Ik was eigenlijk van plan om je een goede code te sturen, maar als je op zo'n manier reageert, dan doe ik daar de moeite niet eens meer voor :)


Sorry hoor, maar jij zegt dingen die ik allang weet.
ik vraag alleen wat er in plaats van die tweede else moet staan. begin jij over sql injection.
Sorry dat ik je waarschuw hoor.
In het vervolg zeg ik wel niks meer. Maar dan moet je niet gaan janken als je website overhoop gehaald is.
Roel van de Water op 15/08/2011 12:04:12

Sorry dat ik je waarschuw hoor.
In het vervolg zeg ik wel niks meer. Maar dan moet je niet gaan janken als je website overhoop gehaald is.

Je weet gewoon niet hoe je het moet oplossen, daarom ga je over iets anders beginnen.
Henke van der bunt op 15/08/2011 12:05:20

Je weet gewoon niet hoe je het moet oplossen, daarom ga je over iets anders beginnen.


Ik zou als ik jou was niet te veel domme en onaardige opmerkingen maken want dan is er helemaal niemand meer die jou wilt helpen.
Heb het al klaar. Als het zolang moet duren voordat ik een zinnig antwoord krijg.

Reageren