[PHP] Database connectie - Wát doe ik fout?

[Asitis]

Ik ben een aardige n00b in het zelf-schrijven van PHP applicaties, maar je moet ergens beginnen.
Daarom probeer ik eerst een database connectie te maken, en een inlog scriptje (opdracht van school).

Ik volg de instructies van school, maar ik heb het idee dat die ook niet kloppen (variabelen die dan 'password', 'passwoord' of 'wachtwoord' heten - alles door elkaar..). Maargoed, dit heb ik tot nu toe:

PHP:

<?php
if (isset($_POST['f_username']) && isset($_POST['f_pass']))
{
    $db = mysql_connect("localhost","school_ict","fhict");
    mysql_select_db("school_ict") or die(mysql_error());
    $query = "  SELECT * FROM klant
                WHERE email= '$email'
                AND wachtwoord= '$wachtwoord' 
                ";
                
    $result = mysql_query($query) or die(mysql_error());
    $record = mysql_fetch_assoc($result);
    
    // Username en password uit record halen
    $email          = $record['email'];
    $wachtwoord     = $record['wachtwoord'];
    
    // Geldigheid inloggegevens controleren
    if ($_POST['f_username'] == $email && $_POST['f_pass'] == $wachtwoord)
    {
        echo ("Yes!");
    }
    else 
    {
        echo ("Gegevens worden niet geldig bevonden.");
    }
    mysql_close($db);
}
?>

en dat met een simpel html formpje (user + pass + submit). De database gegevens en de database kloppen allemaal - Ik krijg steeds als ik probeer in te loggen de echo "Gegevens worden niet geldig bevonden.", terwijl ik zeker weet dat de gegevens kloppen.
Ik denk dus dat de fout zit in het definiëren van de variabelen, maar ik zie door de bomen het bos niet meer.

En ik heb meer vertrouwen in jullie dan in de docente van school

[Snake]

Je query is fout, op regel 6. Plaatst de juiste variablen daar in.

[BastiaanN]

kijk even naar je query. Je gebruikt $email en $wachtwoord terwijl je gegevens via de $_POST variabelen binnenkomen, lijkt me gewoon een kwestie van de verkeerde vars gebruikt? deze definieer je pas daaronder en gebruik je om de records mee op te halen.

/edit spuit 11

[ Voor 4% gewijzigd door BastiaanN op 16-09-2008 17:44 ]

[Duroth]

PHP:

$db = mysql_connect("localhost","school_ict","fhict");

Met dit object doe je vervolgens niks meer.

Deze moet je dan nog even meegeven aan de select_db functie:

PHP:

mysql_select_db("school_ict",$db)

Ook lijken $email en $wachtwoord niet gedefiniëerd te zijn (in je query).

Daarnaast zie ik nog tal van andere schoonheidsfoutjes (die IF/ELSE lijkt me niet echt nuttig, aangezien het record dat je query opvraagt al controleert of de ingevoerde gegevens overeen komen)

[Jaap-Jan]

En vergeet vooral niet de user outinput te escapen.

[ Voor 7% gewijzigd door Jaap-Jan op 16-09-2008 18:54 . Reden: output escapen :? ]

[BastiaanN]

Duroth schreef op dinsdag 16 september 2008 @ 17:45:

PHP:

$db = mysql_connect("localhost","school_ict","fhict");

Met dit object doe je vervolgens niks meer.

Deze moet je dan nog even meegeven aan de select_db functie:

eigenlijk mag hij gewoon "$db =" weglaten. indien je niets opgeeft wordt gewoon de laatste connectie gebruikt, dus dat zal het probleem niet zijn

[Asitis]

Eeeeehm oke
Dat gedoe van de database heb ik dus van de opdracht overgenomen.. Het is toch de bedoeling dat ingevoerde gegevens worden gecontroleerd met die van de database, dus moet die connectie toch blijven staan?

[ppx17]

Connectie blijft ook staan, alleen het object waar je deze in plaatst en vervolgens niet meer gebruikt mag je weg laten, direct mysql_connect doen dus, niet eerst $db=.

Verder is het gewoon kwestie van de verkeerde variabelen gebruiken..

[Asitis]

Oke.. De variabelen zijn dus fout (op regel 6) - ik klink vast stom, maar wat is er dan mis mee?

[AtleX]

Op die plek bestaan $email en $wachtwoord nog helemaal niet. Tip, zet je error reporting eens op een fatsoenlijk level, E_ALL bijvoorbeeld.

[Snake]

PHP:

    $query = "  SELECT * FROM klant
                WHERE email= '$email'
                AND wachtwoord= '$wachtwoord' 
                ";

->

PHP:

    $query = "  SELECT * FROM klant
                WHERE email= '" . mysql_real_escape_string  ($_POST['f_username']) . "'
                AND wachtwoord= '" . mysql_real_escape_string  ($_POST['f_pass']) . "' 
                ";

[Asitis]

Aaah! Thanks Het werkt! Opdracht erop na geslagen, en dit kon eigenlijk nooit de bedoeling zijn (ondertussen uitgevonden dat het geleverde voorbeeld ookal niet werkte.. wat een "school" hé.)

Maar bedankt in ieder geval, ik weet dat ik niet te nauw meer op 't boek moet letten
Jullie zullen vast nog meer van me horen

[Jaap-Jan]

Asitis schreef op dinsdag 16 september 2008 @ 18:46:
Aaah! Thanks Het werkt! Opdracht erop na geslagen, en dit kon eigenlijk nooit de bedoeling zijn (ondertussen uitgevonden dat het geleverde voorbeeld ookal niet werkte.. wat een "school" hé.)

Maar bedankt in ieder geval, ik weet dat ik niet te nauw meer op 't boek moet letten
Jullie zullen vast nog meer van me horen

Welk boek? Dan weet ik in ieder geval wat ik zeker níet moet hebben.

[Asitis]

Jaap-Jan schreef op dinsdag 16 september 2008 @ 18:55:
[...]
Welk boek? Dan weet ik in ieder geval wat ik zeker níet moet hebben.

Don't worry, het is door een 'leraar' op school zelf geschreven. Om het maar even aan te tonen; de opleiding is ICT Media design, en dit is de website van de man die onze lessen schrijft...

[Maghiel]

Asitis schreef op dinsdag 16 september 2008 @ 19:32:
[...]

Don't worry, het is door een 'leraar' op school zelf geschreven. Om het maar even aan te tonen; de opleiding is ICT Media design, en dit is de website van de man die onze lessen schrijft...

Ziet eruit alsof hij zelf terug naar school mag

[Michali]

Klinkt me echt te bekend in de oren. Leraren die er geen drol van snappen.

[Snake]

Asitis schreef op dinsdag 16 september 2008 @ 19:32:
[...]

Don't worry, het is door een 'leraar' op school zelf geschreven. Om het maar even aan te tonen; de opleiding is ICT Media design, en dit is de website van de man die onze lessen schrijft...

En wat is hij er fier op he

Snap niet waarom er uberhaupt les wordt gegeven in frames maarja. (zelfs hier bij informatica, hogeschool, 2de jaar :x )

[ Voor 5% gewijzigd door Snake op 16-09-2008 20:37 ]

[Kettrick]

BastiaanN schreef op dinsdag 16 september 2008 @ 17:44:
kijk even naar je query. Je gebruikt $email en $wachtwoord terwijl je gegevens via de $_POST variabelen binnenkomen, lijkt me gewoon een kwestie van de verkeerde vars gebruikt? deze definieer je pas daaronder en gebruik je om de records mee op te halen.

/edit spuit 11

Waarschijnlijk is het "boek" geschreven in de tijd dat register_globals nog standaard aan stond
Ik op zoek gaan naar een boek wat recenter is, er zijn behoorlijk wat dingen veranderd de afgelopen jaren

[Creepy]

In dit geval had je met simpel debuggen (de query even afdrukken ) vrij snel de fout gevonden.

offtopic:
Maar gezien je leraar met een website uit het jaar 0 en een php boek uit het jaar 1 snap ik dat debuggen je niet is aangeleerd.....pff

Ik hoop eigenlijk dat je een geintje maakt en dat je dit zo niet aangeleerd krijgt.. variabelen direct in een query dumpen zonder concatenation en welke vorm van escaping en input controle dan ook

[ Voor 29% gewijzigd door Creepy op 16-09-2008 20:52 ]

[Maghiel]

Creepy schreef op dinsdag 16 september 2008 @ 20:49:
In dit geval had je met simpel debuggen (de query even afdrukken ) vrij snel de fout gevonden.

offtopic:
Maar gezien je leraar met een website uit het jaar 0 en een php boek uit het jaar 1 snap ik dat debuggen je niet is aangeleerd.....pff

Ik hoop eigenlijk dat je een geintje maakt en dat je dit zo niet aangeleerd krijgt.. variabelen direct in een query dumpen zonder concatenation en welke vorm van escaping en input controle dan ook

Je wilt niet weten hoe ze op sommige scholen lopen te klooien.
Op de school(waar ik gelukkig mee klaar ben nou ) waar ik zat, was er een vak waar wat simpels in ActionScript gemaakt moest worden, maar de leraar kon zelf ook geen hulp bieden, hij kwam er niet uit. Nou ja toen heb ik toch nog wat centjes kunnen verdienen door wat functies voor een 10tje te verkopen aan klasgenoten

[Sebazzz]

Belachelijk Het is dat je een papiertje moet hebben maar anders kan ik net zo goed niet naar de hogeschool gaan

Maghiel schreef op dinsdag 16 september 2008 @ 20:55:
[...]
Nou ja toen heb ik toch nog wat centjes kunnen verdienen door wat functies voor een 10tje te verkopen aan klasgenoten

Is dat niet... gemeen of soort machts/kennismisbruik?

[Asitis]

Creepy schreef op dinsdag 16 september 2008 @ 20:49:
In dit geval had je met simpel debuggen (de query even afdrukken ) vrij snel de fout gevonden.

offtopic:
Maar gezien je leraar met een website uit het jaar 0 en een php boek uit het jaar 1 snap ik dat debuggen je niet is aangeleerd.....pff

Ik hoop eigenlijk dat je een geintje maakt en dat je dit zo niet aangeleerd krijgt.. variabelen direct in een query dumpen zonder concatenation en welke vorm van escaping en input controle dan ook

Helaas maak ik geen grapje.. Bij ons op school staan deze lessen dan ook bekend als "legendarisch slecht". Maar ergens is het misschien wel goed, we komen zelf de fouten tegen en zoeken zelf wel uit hoe het dan wél hoort. Maarja, dat is niet echt het doel van een school

[Creepy]

Zolang je zelf opzoek gaat hoe het wel moet dan is er niet zoveel aan de hand (daar leer je alleen maar van ) Er zijn echter zat mensen die zoiets hebben van "ja maar het werkt toch" en die mensen op de totaal verkeerde manier zaken aanleren.........

offtopic:
Ik hoop dat je leraar dit topic ziet en toch eens gaat nadenken over wat ie aan het doen is.....

[ Voor 17% gewijzigd door Creepy op 16-09-2008 21:05 ]

[Cartman!]

Creepy, en dat is dan ook het verschil tussen de mensen die later wel een goede baan krijgen en de mensen die na hun "ICT"-opleiding broodjes gaan verkopen.

Het is wel extreem jammer dat je 1500e collegegeld mag betalen voor zo'n prutsdocent. Helaas heb ik t zelf ook meegemaakt, gaf me maar een hoog cijfer omdat ie niet kon beoordelen of t goed was... hij wist namelijk niet wat sessions waren

[RobIII]

Sebazzz schreef op dinsdag 16 september 2008 @ 21:00:
Is dat niet... gemeen of soort machts/kennismisbruik?

Nee, dat is nou zaken doen Welcome in the real world
Ik heb iets dat jij graag wil hebben? Dokken. Jij kunt iets dat ik niet kan? Dokken. Op wat open source hippies na wordt er met software gewoon, net als met brood, kleding of auto's gewoon grof geld verdiend. En terecht.

[Creepy]

Cartman! schreef op dinsdag 16 september 2008 @ 21:26:
Creepy, en dat is dan ook het verschil tussen de mensen die later wel een goede baan krijgen en de mensen die na hun "ICT"-opleiding broodjes gaan verkopen lopen te kloten zodat een fatsoenlijk iemand dat gepruts weer kan opruimen.

Echt... hoe vaak je mensen tegenkomt die toch echt als software ontwikkelaar o.i.d. aan het werk zijn maar niet in staat zijn om de meeste simpele dingen voor elkaar te krijgen omdat hun IDE/ontwikkelingomgeving/taal er niet een kant en klare oplossing voor heeft...

Ik zou er bijna zelf les van gaan geven.... bijna dan he

Is dat niet... gemeen of soort machts/kennismisbruik?

Slim juist
Die kennis had de rest zelf ook op kunnen doen....

[ Voor 8% gewijzigd door Creepy op 16-09-2008 21:43 ]

[RobIII]

Creepy schreef op dinsdag 16 september 2008 @ 21:39:

Ik zou er bijna zelf les van gaan geven.... bijna dan he

Ik zou bijna moderator worden in een devschuur...oh...wacht...

[whoami]

RobIII schreef op dinsdag 16 september 2008 @ 21:41:
[...]

Ik zou bijna moderator worden in een devschuur...oh...wacht...

dat wil ik later als ik groot ben ook.

[curry684]

Asitis schreef op dinsdag 16 september 2008 @ 19:32:
[...]

Don't worry, het is door een 'leraar' op school zelf geschreven. Om het maar even aan te tonen; de opleiding is ICT Media design, en dit is de website van de man die onze lessen schrijft...

* curry684 kruipt weer op stoel.

Ter verdediging moet ik aanvoeren dat ik recent een IMD'er op sollicitatie heb gehad die wel degelijk heel mooi spul had gebouwd en er ook echt verstand van had. Dhr. van Cooten is iig niet 100% representatief voor de studenten

whoami schreef op dinsdag 16 september 2008 @ 23:12:
[...]

dat wil ik later als ik groot ben ook.

Ik ben al zo groot dat ik het gewoon niet meer ben

[ Voor 15% gewijzigd door curry684 op 17-09-2008 00:08 ]

[GlowMouse]

Asitis schreef op dinsdag 16 september 2008 @ 19:32:
[...]

Don't worry, het is door een 'leraar' op school zelf geschreven. Om het maar even aan te tonen; de opleiding is ICT Media design, en dit is de website van de man die onze lessen schrijft...

Dit plaatje is nog wel het ergste:
Het is wel een privésite en dat linkerdeelvenster is in 2004 online gezet. In 4 jaar kun je veel leren, dus zoveel zegt die pagina echt niet over die man.

[Asitis]

GlowMouse schreef op woensdag 17 september 2008 @ 00:14:
[...]

Dit plaatje is nog wel het ergste: [afbeelding]
Het is wel een privésite en dat linkerdeelvenster is in 2004 online gezet. In 4 jaar kun je veel leren, dus zoveel zegt die pagina echt niet over die man.

Nou, laat ik je vertellen dat de lessen en voorbeelden (recent gemaakte!) nog geen steek moderner zijn en allemaal worden gegeven met het excuus "Ja.. Het is aan jullie om het beter te doen!"

[Creepy]

curry684 schreef op woensdag 17 september 2008 @ 00:08:
[...]
Ik ben al zo groot dat ik het gewoon niet meer ben

Stiekum wil je gewoon terug he?
@GlowMouse: maar die pagina helpt wel mee aan beeldvorming Dat combineren we met de verhalen van van Asitis en er onstaat toch een aardig beeld. Tuurlijk, geheel eenzijdig maar toch.... en dat moet je dan nog combineren met een klas vol pubers die nergens zin in hebben en de attentiespan hebben van een goudvis.. probeer dan maar eens zaken goed aan te leren

Er wordt wat mij betreft aan verschillende kanten vaak te licht gedacht over "programmeren" (en dan bedoel ik eigenlijk software ontwikkeling in z'n geheel). Vaak aan de kant van diegenen die het wel willen kunnen maar helaas ook vaak bij docenten e.d.

Dhr. van Cooten is iig niet 100% representatief voor de studenten

Gelukkig heb je altijd studenten die wel een stapje harder lopen en wat meer werk verzetten.

[ Voor 72% gewijzigd door Creepy op 17-09-2008 12:28 ]

[Maghiel]

Sebazzz schreef op dinsdag 16 september 2008 @ 21:00:
Is dat niet... gemeen of soort machts/kennismisbruik?

Nee, ik heb er toch tijd en moeite ingestoken, dat had de rest ook kunnen doen(hebben sommigen ook gedaan). Ik ga natuurlijk niet de school halen voor hun

RobIII schreef op dinsdag 16 september 2008 @ 21:30:
[...]


Nee, dat is nou zaken doen Welcome in the real world
Ik heb iets dat jij graag wil hebben? Dokken. Jij kunt iets dat ik niet kan? Dokken. Op wat open source hippies na wordt er met software gewoon, net als met brood, kleding of auto's gewoon grof geld verdiend. En terecht.

Nou ja, hoeft niet met alles. Maar wat ik zei, het was tijd en moeite van mij. En ik vind een 10tje dan vrij schappelijk

Creepy schreef op dinsdag 16 september 2008 @ 21:39:
[...]

Echt... hoe vaak je mensen tegenkomt die toch echt als software ontwikkelaar o.i.d. aan het werk zijn maar niet in staat zijn om de meeste simpele dingen voor elkaar te krijgen omdat hun IDE/ontwikkelingomgeving/taal er niet een kant en klare oplossing voor heeft...

Ik zou er bijna zelf les van gaan geven.... bijna dan he

[...]

Slim juist
Die kennis had de rest zelf ook op kunnen doen....

precies

[ Voor 8% gewijzigd door Maghiel op 17-09-2008 12:56 ]

[Crazybyte]

Als ik zo over het topic heen lees, kan ik er niet aan ontkomen dat wij op dezelfde school zitten. Ik herkende het eigenlijk al bij het wachtwoord voor de connectie met je database.

Ik heb me in het verleden al vaak genoeg mateloos geërgerd aan dat wat de studenten hier wordt aangeleerd. Ben zelf een ouderejaars en heb al vaker een IMD-er geholpen met zijn/haar opdrachten, maar dat wat ze je aanleren is zo vaak zo hopeloos verouderd, dat het gewoon beter is om de uitwerking links te laten liggen en zelf maar opzoek te gaan naar een correcte uitwerking.

Maar goed, mocht je in de toekomst eens hulp nodig hebben, loop eens 3.51a binnen.