Inbouwen layout

Gestart doorTwany14 reacties74 weergaven

15 berichten · pagina 1 van 1

Twany

Lid · Topicstarter

Berichten
1.863
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Hallo jongens/meisjes,

Ik heb een code die ik in wel bouwen in een layout.
Dit is de code van inloggen.php:

[COLOR= #0000BB]<?php[/COLOR]
[URL="http://www.php.net/include"]include[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]"conf.inc.php"[/COLOR][COLOR= #007700]);[/COLOR] [COLOR= #FF8000]// Bekijkt database en formulier informatie[/COLOR]
[URL="http://www.php.net/session_start"]session_start[/URL][COLOR= #007700]();[/COLOR] [COLOR= #FF8000]// Start de sessie[/COLOR]
[URL="http://www.php.net/if"]if[/URL] [COLOR= #007700]([/COLOR][COLOR= #0000BB]$_SESSION[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]logged[/COLOR][COLOR= #007700]] ==[/COLOR] [COLOR= #0000BB]1[/COLOR][COLOR= #007700]) {[/COLOR] [COLOR= #FF8000]// Gebruiker is al ingelogd[/COLOR]
    [URL="http://www.php.net/header"]header[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]"Location: index.php"[/COLOR][COLOR= #007700]);[/COLOR] [COLOR= #FF8000]// Gaat naar de voorpagina[/COLOR]
    [URL="http://www.php.net/exit"]exit[/URL][COLOR= #007700]();[/COLOR] [COLOR= #FF8000]// Stopt de rest van het script[/COLOR]
[COLOR= #007700]}[/COLOR] [URL="http://www.php.net/else"]else[/URL] [COLOR= #007700]{[/COLOR]
    [URL="http://www.php.net/if"]if[/URL] [COLOR= #007700](![/COLOR][URL="http://www.php.net/isset"]isset[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]submit[/COLOR][COLOR= #007700]])) {[/COLOR] [COLOR= #FF8000]// Als het formulier niet is verzonden[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"<form action="inloggen.php" method="POST">"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"<table>"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"<tr>"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"<td colspan="2">Log in:</td>"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"</tr>"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"<tr>"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"<td width="50%">Gebruikersnaam:</td><td width="50%"><input name="username" size="18" type="text" />"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"</tr>"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"<tr>"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"<td width="50%">Wachtwoord:</td><td width="50%"><input name="password" size="18" type="password" />"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"</tr>"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"<tr>"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"<td colspan="2"><input type="submit" name="submit" value="submit"</td>"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"</tr>"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"</table>"[/COLOR][COLOR= #007700];[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"</form>"[/COLOR][COLOR= #007700];
    }[/COLOR] [URL="http://www.php.net/else"]else[/URL] [COLOR= #007700]{[/COLOR]
        [COLOR= #0000BB]$username[/COLOR] [COLOR= #007700]=[/COLOR] form[COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]username[/COLOR][COLOR= #007700]]);[/COLOR]
        [COLOR= #0000BB]$password[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/md5"]md5[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]password[/COLOR][COLOR= #007700]]);[/COLOR] [COLOR= #FF8000]// Encrypt het wachtwoord[/COLOR]
 
        [COLOR= #0000BB]$q[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/mysql_query"]mysql_query[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]"SELECT * FROM `users` WHERE username = $username AND password = $password"[/COLOR][COLOR= #007700])[/COLOR] or [URL="http://www.php.net/die"]die[/URL] [COLOR= #007700]([/COLOR][URL="http://www.php.net/mysql_error"]mysql_error[/URL][COLOR= #007700]());[/COLOR] [COLOR= #FF8000]// mySQL query[/COLOR]
        [COLOR= #0000BB]$r[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/mysql_num_rows"]mysql_num_rows[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$q[/COLOR][COLOR= #007700]);[/COLOR] [COLOR= #FF8000]// Kijkt of er iets is in de database [/COLOR]
 
        [URL="http://www.php.net/if"]if[/URL] [COLOR= #007700]([/COLOR][COLOR= #0000BB]$r[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]1[/COLOR][COLOR= #007700]) {[/COLOR] [COLOR= #FF8000]// Er is iets in de database, de gebruikersnaam en wachtwoord komen overeen[/COLOR]
            [COLOR= #0000BB]$_SESSION[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]logged[/COLOR][COLOR= #007700]] =[/COLOR] [COLOR= #0000BB]1[/COLOR][COLOR= #007700];[/COLOR] [COLOR= #FF8000]// Stelt de sessie in[/COLOR]
            [URL="http://www.php.net/header"]header[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]"Location: index.php"[/COLOR][COLOR= #007700]);[/COLOR] [COLOR= #FF8000]// Terug naar voorpagina[/COLOR]
            [URL="http://www.php.net/exit"]exit[/URL][COLOR= #007700]();[/COLOR] [COLOR= #FF8000]// Stopt de rest van het script[/COLOR]
        [COLOR= #007700]}[/COLOR] [URL="http://www.php.net/else"]else[/URL] [COLOR= #007700]{[/COLOR] [COLOR= #FF8000]// Foute gebruikersnaam/wachtwoord[/COLOR]
            [URL="http://www.php.net/exit"]exit[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]"Foute gebruikersnaam/wachtwoord"[/COLOR][COLOR= #007700]);[/COLOR] [COLOR= #FF8000]// Stopt het script met een error[/COLOR]
        [COLOR= #007700]}
    }
}[/COLOR]
[URL="http://www.php.net/mysql_close"]mysql_close[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$db_connect[/COLOR][COLOR= #007700]);[/COLOR] [COLOR= #FF8000]// Sluit de connectie[/COLOR]
?>


Hoe kan ik die inbouwen in layout? Ik heb al op google gekeken maar kon niks helpend vinden...
0
Berichten
1.871
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Boven echo "";

het stuk na de content div inc. wink
0
Berichten
10.421
Reputatie
4
Punten
0,00
Lid sinds
Laatst gezien
Kan je meteen weggooien, het is voor 80% fout gebouwd of verouderd....
0
Twany

Lid · Topicstarter

Berichten
1.863
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Kun jij het vernieuwen dan...
0
Berichten
93.699
Reputatie
0
Punten
1,27
Lid sinds
Laatst gezien
Zoveel mogenlijk html uit de php houden. Die gehele form moet uit de echo en uit de php tags?

Exit en die moeten allemaal weg. Gewoon een if else afhandeling is goed
0
Berichten
10.421
Reputatie
4
Punten
0,00
Lid sinds
Laatst gezien
Tvl schreef:
Kun jij het vernieuwen dan...
Ik kan wel tips geven:
- Gebruik if($_SERVER[REQUEST_METHOD] == "POST") { om te controleren of een formulier is verstuurd.
- Plaats grote lappen HTML het liefst buiten PHP.
- Ik zie mogelijke SQL-injection. Ik zie niet wat je form() functie doet, maar gezien de naam ervan, geeft het mij weinig duidelijkheid wat die functie wel doet. Het hernoemen kan zeker geen kwaad
- Variabelen horen buiten quotes.
- Ik mis foutafhandeling op je mysql_query() (en dan geen laffe die() techniek)
- een script gaat nooit dood als hij fout loopt. Dus je exit() bij je gebruikersnaam/login-controle kan weg.

toevoeging op 27 juni 2012, 10:58:32:
Sevvlor schreef:
Zoveel mogenlijk html uit de php houden. Die gehele form moet uit de echo en uit de php tags?
Exit en die moeten allemaal weg. Gewoon een if else afhandeling is goed

Nee, bij een redirectheaders is het zelfs aan te raden.
0
Twany

Lid · Topicstarter

Berichten
1.863
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Dat is dus het probleem, deze code heb ik van internet afgehaald en weet dus niet hoe ik em moet verbeteren bloos...
0
Berichten
11.263
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Tvl schreef:
Dat is dus het probleem, deze code heb ik van internet afgehaald en weet dus niet hoe ik em moet verbeteren bloos...
Dan wil ik je alleen een tip geven:

Ik begon ook projectjes door gratis PHP codes te nemen van internet, maar dan kan je geen fouten oplossen en dus je project kan voor 75% niet meer te redden zijn.
Nu ben ik begonnen met het leren van basis PHP via een tut die je in een maand voldoende kent, en dan ben je al een hoop verder, dus ik raad je aan eerst de basis te leren.

Als je die basis kent, kan je toch al een paar fouten die Aarclay opnoemde verbeteren wink
0
Berichten
10.421
Reputatie
4
Punten
0,00
Lid sinds
Laatst gezien
Tvl schreef:
Dat is dus het probleem, deze code heb ik van internet afgehaald en weet dus niet hoe ik em moet verbeteren bloos...
Dat zeg ik toch.......??????????
0
Twany

Lid · Topicstarter

Berichten
1.863
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Huh confused?
0
Berichten
10.421
Reputatie
4
Punten
0,00
Lid sinds
Laatst gezien
Pssst: Post #6 in dit topic....
0
Twany

Lid · Topicstarter

Berichten
1.863
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Heb je dan voor mij een NEDERLANDSE tutorial...?
0
Twany

Lid · Topicstarter

Berichten
1.863
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Dankje smile
0
Berichten
5.957
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien

Jouw reactie

Deel je idee, stel een vraag of help een ander verder.

Editor laden…

Praat mee met de One2xs-community.

Inloggen om te reageren