Script check

  • Halle leden,


    Ik ben beginner php programeur en heb een vraagje of ik zo goed begin?


    Ik maak een profiel script, dat je leden kan bekijken.


    Hij is nog niet helemaal af!


    Mischien eventuele tips zijn altijd welkom.


    user.php


    Alvast bedankt.

  • Engelse commentaar is juist goed.
    Eigenlijk hooort iedereen gelijk te beginnen met engels commentaar.
    Maar ik geef toe dat ik daar zelf nooit zin in heb :P..


    Verder zal je het volgende kunnen doen:
    mysql_query("SELECT * FROM `Users` WHERE login='".$getuser."'");


    Je ziet hier boven dat je SELECT alles FROM users.


    Terwijl het veel beter is als je het volgende doet:
    SELECT login, family,rank,information FROM Users Where login = '".mysql_real_escape_string($_GET['user'])."'");


    Als je goed kijkt heb ik die * weg gehaald en vervangen zodat hij niet heel je tabel users gaat laden (extra load).
    En ik heb de `` weg gehaald bij het stukje Users, dit omdat het onnodig is..


    Verder waarom is de tabel Users de u uberhaubt met hoofdletter?
    het is makkelijker werken als het kleine letter is maar ja iedereen eigen styl.. en dan moet je de tabel naam ook aanpassen dus ja :p..


    Ook zie je dat ik mysql_real_escape_string heb toegepast zodat user er veilig in komt..
    Eigenlijk zou die nog veiliger kunnen door niet mysql_real_escape_string te gebruiken maar is_almun die checkt of je wel alphabetische tekens gebruikt en niet van die %&((^!@! ..


    Waarom gebruik je hier het volgende:

    PHP
    if ($family == '') {


    En niet met een empty functie om te kijken of die leeg is?

    PHP
    if (empty($family)) {


    Ik hoop dat je nu wel verder kan;)


    O ja ps wat voor script is het?
    Want anders zie ik nog een verbeter puntje, heel misschien.

    School
    Tweedejaars Informatica Student @ Hogeschool Rotterdam.


    Webtalen
    - PHP ( OOP )
    - SQL
    - JavaScript i.c.m. jQuery framework
    - (x)HTML
    - CSS
    - Actionscript
    - XML


    Programmeertalen
    - Java
    - C#
    - Ruby i.c.m. on Rails framework

Participate now!

Heb je nog geen account? Registreer je nu en word deel van onze community!