|
Huidige score: |
|
80 |
|
|
Gepost op:
06 juli 2006 - 19:22 |
|
|
|
PHP ver gevorderde
|
Waarom werk je nog met $HTTP_SESSION_VARS ? $_SESSION is beter.
Ook werk je met mysql_fetch_object(), ookal denk ik dat mysql_fetch_assoc() beter is (of mysql_fetch_array() is ook goed).
Voor de rest zie ik qua functie en variabelegebruik niet meteen "fouten".
Wat je wel vaak anders doet is indenteren.
Soms indenteer je de accolades mee, soms niet, dan zet je de eerste accolade achter het haakje nog,... |
|
|
|
Gepost op:
06 juli 2006 - 19:23 |
|
|
|
PHP gevorderde
|
Kun je mss ook zeggen waar ik dat doe? |
|
|
|
Gepost op:
06 juli 2006 - 20:14 |
|
|
|
PHP expert
|
er zit een horizontale lijn in je layout.....(zie maar in FF)
voor de rest wel netjes |
|
|
|
Gepost op:
06 juli 2006 - 20:26 |
|
|
|
Ouwe rakker
|
config.php
In je config heb je ook een functie staan, geef functies een eigen file.
Verder maak je hier ook al verbinding, ik persoonlijk gebruik de config.php alleen om variabelen aan te maken en hier start-waarden aan te geven.
registreer.php
In je query maak je opeens gebruik van $pass, die variabele maak je voor de rest nergens aan.
inloggen.php
Hier controleer je het volgende:
!IsSet($pass)
Waar haal je de $pass nu vandaan? Deze is ook niet gedeclareert en zal dus altijd falen. Verder maak je daar gebruik van exit wat niet netjes is, vang dit soort dingen gewoon af en schotel de gebruiker een fatsoenlijke melding voor.
Je login formuliertje ziet er heel slordig uit. Je form decleration staat midden in een tabel en er is niet ge-indent.
uitloggen.php
In plaats van door je session array te loopen en te unsetten kan je ook gebruik maken van PHP.net: session_unset. Verder is het niet slim om output naar de browser te zenden met echo voordat je een header() zend. Dit vang je in andere files wel weer af.
nieuws.php
Je maakt gebruik van mysql_fetch_object. Zoals hierboven ook al eerder is vermeld is het beter dat je gebruik maakt van mysql_fetch_assoc().
<?php
echo "".$obj->titel." gepost op ";
echo date("d m Y",$obj->datum);
echo" door ".$obj->poster."";
?>
<?php echo "".$obj->titel." gepost op "; echo" door ".$obj->poster.""; ?>
Dit is natuurlijk gewoon zonde, je kan het beter zo doen (ik werk hem nu nog even uit met object).
<?php
echo $obj->title . ' gepost op ' . date("d m Y", $obj->datum) . ' door '. $obj->poster;
?>
<?php echo $obj->title . ' gepost op ' . date("d m Y", $obj->datum) . ' door '. $obj->poster; ?>
Je maakt hier nergens gebruik van stripslashes() of enige andere vorm van beveiliging om te voorkomen dat men HTML uitvoert op je website. Ik mag er dan dus vanuit gaan dat dat HTML enzo afgevangen wordt op het moment dat alles in de db gepropt wordt (ook al is dat niet aan te raden, altijd achteraf pas dit soort dingen toepassen).
<?php echo "".$obj->id.""; ?>
<?php echo "".$obj->id.""; ?>
Als je server short tags ondersteund kan je ook gewoon dit doen:
De " zijn sowieso niet nodig aan het begin en het einde.
Los eerst dit maar eens op, dan kijken we daarna wel verder |
|
|
|
Gepost op:
06 juli 2006 - 22:14 |
|
|
|
PHP gevorderde
|
@Prx: Ty voor de uitgebreide reactie.Ik zal het bekijken/veranderen. |
|
|
|
Gepost op:
07 juli 2006 - 00:43 |
|
|
|
HTML beginner
|
Mooie site. Alleen je portfolio vind ik een beetje verwarrend. Dat mag gerust een beetje duidelijker. Dit kan je doen door:
-links te leggen naar je gemaakte sites
-onderscheid te maken tussen je titeltjes(url, wat, uitleg) en de uitleg hierbij.
-tabellen gebruiken
Daarnaast vind ik het gebruikte lettertype niet al te duidelijk en redelijk klein. Je krijgt een 8
Voor de rest chapeau!!!!;-) |
|
|
|
Gepost op:
07 juli 2006 - 17:46 |
|
|
|
PHP gevorderde
|
|
|
|
Gepost op:
07 juli 2006 - 17:57 |
|
|
|
PHP gevorderde
|
|
|
Posten van een reactie is niet mogelijk. |
Bekijk: |
|
|
|