PHP gevorderde |
|
Plaatscode: 6329
Fouten:
- Een query 2 keer aanroepen (voor gebruikersnaam en wachtwoord apart) terwijl dit ook in 1 query kan.
- in de Query's WHERE clausule had je een vaste vergelijking staan, en geen variabele.
- Je script kon MySql injections bevatten (nu niet).
- mysql_fetch_assoc/row kan alleen in een while lues gebruikt worden.
Onduidelijkheden:
- waarom roep je nog een keer alle gegevens op over deze persoon, maar dan uit de tabel leden ipv gebruikers?
(Ik vermoed een geval van slecht normaliseren)
Daarnaast moet je bij een foutieve inlogpoging, nooit aangeven WAT er fout is (gebnaam of ww) maar altijd dat de combinatie verkeerd is.
Dit maakt het voor hackers moeilijker om juiste combinaties te bepalen. |