Is dit een duidelijk php script?

Overzicht Reageren

Sponsored by: Vacatures door Monsterboard

Ocirina Ocirina

Ocirina Ocirina

18/01/2011 22:39:49
Quote Anchor link
Wat zou hier anders moeten kwa uiterlijk? De werking is wel duidelijk, niet veilig etc. maar daar gaat het niet om.

Quote:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
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
<?php
if($_SERVER['REQUEST_METHOD'] == 'POST') // Controle of er op de knop is geklikt.
{
    if(!empty($_POST['naam']) AND ($_POST['password'])) // Kijken of de gegevens BEIDE zijn ingevoerd.
    {
        if($_POST['naam'] == 'willem' AND md5 ($_POST['password']) == 'md5 gecodeerde pass.')
        {

            echo 'U bent succesvol ingelogd!';
        }

        else
        {
            echo 'Helaas bent u niet ingelogd.';
        }
    }

    else
    {
        echo "De ingevoerde gegevens zijn niet juist ingevoerd.";  // Melding foute gegevens.
    }
}

else
{
    header('Refresh: 2; url=index.php');  // Als er niet geklikt is wordt je doorverwezen naar de index.php pagina met een gepaste melding.
    echo 'U bevind zich op een niet geautoriseerde pagina, u wordt doorverwezen.';
}


?>
 
PHP hulp

PHP hulp

13/01/2025 01:07:47
 

18/01/2011 22:44:28
Quote Anchor link
Bijna goed.
Regel vier....
 
Ocirina Ocirina

Ocirina Ocirina

18/01/2011 22:45:37
Quote Anchor link
isset?
Gewijzigd op 18/01/2011 22:46:17 door Ocirina Ocirina
 

18/01/2011 22:47:57
Quote Anchor link
Kijk eens goed.
 
Vincent Huisman

Vincent Huisman

18/01/2011 22:49:05
Quote Anchor link
AND ($_POST['password'])

dit zal dus altijd goed zijn
 
Jelmer -

Jelmer -

18/01/2011 22:49:44
Quote Anchor link
empty is een functie die je aanroept met $_POST['naam'] als argument. Dat gaat goed. Bij $_POST['password'] roep je geen functie aan.
 
Ocirina Ocirina

Ocirina Ocirina

18/01/2011 22:49:52
Quote Anchor link
AND of OR? Omg:p ik ziet het echt niet?

Toevoeging op 18/01/2011 22:51:08:

Moet her OR zijn? Als ik dat in me script had ingevoerd werkte het juist niet.
Dank jullie!
 
Vincent Huisman

Vincent Huisman

18/01/2011 22:51:21
Quote Anchor link
van AND zou ik OR maken en je checkt nu niet of post wachtwoord is gevuld maar alleen of hij bestaat
 

18/01/2011 22:51:54
Quote Anchor link
Vincent Huisman op 18/01/2011 22:49:05:
AND ($_POST['password'])

dit zal dus altijd goed zijn


Nee.
Als password leeg is zal php dit type casten naar false.
Zie Jelmer's reactie voor antwoord.
(Logica...)

Vincent Huisman op 18/01/2011 22:51:21:
van AND zou ik OR maken en je checkt nu niet of post wachtwoord is gevuld maar alleen of hij bestaat

NEEN!
Je moet wel en gebruiken.
EN de gebruikersnaam EN het password moet ingevuld zijn. Dan pas is het nuttig om verdere controles te doen.
Gewijzigd op 18/01/2011 22:54:07 door
 
Ocirina Ocirina

Ocirina Ocirina

18/01/2011 22:53:19
Quote Anchor link
Oke, ik begrijp wat ik verkeerd doe. Maar wat is dan de verbetering, nog een If/else erin stoppen? Of kan ik het bij dezelfde functies houden?
Gewijzigd op 18/01/2011 22:53:40 door Ocirina Ocirina
 

18/01/2011 22:56:01
Quote Anchor link
Ocirina of Time op 18/01/2011 22:53:19:
Oke, ik begrijp wat ik verkeerd doe. Maar wat is dan de verbetering, nog een If/else erin stoppen? Of kan ik het bij dezelfde functies houden?


Alsjeblieft. Luister niet naar Vincent, dat is alleen maar onzin wat hij heeft gezegd. Het hele script is correct op het dingetje na waar Jelmer het over heeft...
 
Ocirina Ocirina

Ocirina Ocirina

18/01/2011 22:57:57
Quote Anchor link
Nee, dat doe ik ook niet. Maar dan moet ik alleen nog een !empty functie erbij zetten, dus zo:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
if(!empty($_POST['naam']) AND !empty($_POST['password']))
 

18/01/2011 22:58:49
Quote Anchor link
Inderdaad.
 
Ocirina Ocirina

Ocirina Ocirina

18/01/2011 23:02:28
Quote Anchor link
Dankjewel! Ik weet niet of het je verder interesseert, maar vind het wel tof hier zo.
Ook vind ik het fijn dat jij (bijna) altijd helpt (karl dus). Ook omdat je een goede kennis hebt van php. In ieder geval bedankt allemaal!
Gewijzigd op 18/01/2011 23:03:06 door Ocirina Ocirina
 

18/01/2011 23:08:26
Quote Anchor link
Ocirina of Time op 18/01/2011 23:02:28:
Dankjewel! Ik weet niet of het je verder interesseert, maar vind het wel tof hier zo.
Ook vind ik het fijn dat jij (bijna) altijd helpt (karl dus). Ook omdat je een goede kennis hebt van php. In ieder geval bedankt allemaal!


Ik ben blij dat je zo'n net script hebt gemaakt.
 
Ocirina Ocirina

Ocirina Ocirina

18/01/2011 23:10:03
Quote Anchor link
Haha ;) dank u.
 



Overzicht Reageren

 
 

Om de gebruiksvriendelijkheid van onze website en diensten te optimaliseren maken wij gebruik van cookies. Deze cookies gebruiken wij voor functionaliteiten, analytische gegevens en marketing doeleinden. U vindt meer informatie in onze privacy statement.