Site check: rate or hate!
- als ik de datum 10-15-2011 invoer pakt hij hem dus niet (engelse datum). De zoekfunctie is dus wel slecht.
- Lay-out is mooi
- Ik mis een logo.
- Niet W3C valid
- 'waarom de vliegtickets.nl' -> 'Nu u uw vluchtgegevens heeft ingevuld, kunt u hierboven de beschikbare vliegticket aanbieders vergelijken om zo de goedkoopste prijs te vinden en meteen te boeken.' is dit zo, velden zijn bij mij leeg.
- Ik mis wat navigatie in de header.
- Als ik een zoekopdracht heb gedaan en ik klik op een vlieg aanbieder ga ik van jou website af. En dat is juist iets wat jij niet wilt.
- Waar is de social media toolbar. Like, +1 etc. Dit kan op zo een site niet ontbreken volgens mij.
- Waarom kan ik niet op bestemmingen en aanbieders klikken in de footer?
- Ik mis een disclaimer, algemene voorwaarden en privacy policy.
oftewel genoeg werk nog ;) opzich wel mooie site en snel bij mij in elk geval.
Code (php)
1
' or '1'='1
HTML is nog niet goed: http://validator.w3.org/check?uri=http%3A%2F%2Fwww.devliegtickets.nl%2F&charset=%28detect+automatically%29&doctype=Inline&group=0 .
Wat is nu de toegevoegde waarde van dit systeem? Dat wordt niet echt duidelijk.
Edit: en zorg ervoor dat je technische foutmeldingen niet aan de gebruiker laat zien:
Code (php)
1
Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in /home/deb12789n4/domains/devliegtickets.nl/public_html/index.php on line 98
Gewijzigd op 02/11/2011 20:38:26 door The Force
@Internet Verslaafde
Klopt, ik zal nog even Tahoma en Arial toevoegen aan de font family.
@Baz IJsseldoorn
Allereerst bedankt voor de uitgebreide review, erg nuttig! Je hebt een aantal dingen genoemd die ik zelf inderdaad nog wil toevoegen, en aantal dingen waar ik geen raad mee wist.
Dingen waar ik mij al bewust van was en weet wat ik er mee moet:
- De fouten zoals van mijn site af na het klikken op een aanbieder en geen links op de bestemmingen staan inderdaad ook al op mijn lijstje!
- Ik ben op dit moment bezig om jQuery toe te voegen voor de datepicker, stond al op mijn lijstje maar werkte nog niet helemaal zo als het moest.
- Ik wil van interne SQL naar externe XML, op deze manier kan ik betere gegevens tonen die meer up to date zijn. Hier wordt de site wel langzamer door.
- W3C valid ga ik aan werken nadat de functionaliteit 100% is.
- Logo/huisstijl ben ik nog voor aan het schetsen.
Waar ik nog mijn twijfels over had:
- Auto aanvullen stond ook nog op mijn lijstje, maar ik wist niet waar ik een database vandaan kon halen: dus bij deze bedankt voor de tip!
- Wat betreft disclaimer, privacy policy, algemene voorwaarden, social media, navigatie etc: ik zoek nog een mooie plek voor deze informatie, heb jij een idee?
Kortom, nog veel werk inderdaad. De website wou ik z.s.m. online om al inkomsten te generen, vandaar dat hij nog niet 100% is.
@The Force
De toegevoegde waarde is dat je op één plek met één klik de goedkoopste prijs kunt vinden van alle vliegticketaanbieders die er zijn.
EDIT:
Zie ook mijn andere topic: http://www.phphulp.nl/php/forum/topic/simplexml-xpath/80875/last/
Ik kom er namelijk niet helemaal uit met de XML haha.
Gewijzigd op 02/11/2011 20:57:34 door Bas Wagenmaker
Enige storende vind ik de links in de tweets. Als ik iets als bit.ly/vfpQPy tegenkom op mijn site, denk ik meteen aan een spammer.
Tobias Tobias op 02/11/2011 20:50:43:
Enige storende vind ik de links in de tweets. Als ik iets als bit.ly/vfpQPy tegenkom op mijn site, denk ik meteen aan een spammer.
Daar zal ik ook nooit op gaan klikken. Inderdaad veel te 'verdacht'.
The Force op 02/11/2011 20:35:01:
Waarom krijg ik resultaten als ik onderstaande code invul bij "Naar:"?
Code (php)
1
' or '1'='1
Dit is dus SQL-Injection.
Heb je een alternatief voor shorturls? Die dus niet direct naar mijn site gaan maar naar bijvoorbeeld cheaptickets.nl?
Toevoeging op 02/11/2011 21:08:40:
$-SanThe-
SQL injection meteen verholpen, kun je dit testen?
Toevoeging op 02/11/2011 22:09:36:
Datepicker ook toegevoegd!
Toevoeging:
Je kunt er natuurlijk ook een javascript-oplossing voor maken, dan is de link helemaal niet meer te zien
Gewijzigd op 03/11/2011 09:49:31 door Tobias Tobias
Automatisch aanvullen trouwens ook toegevoegd.
Mathias B op 05/11/2011 09:59:30:
..en het duurt ook lang om te laden.
bij mij 5 sec. per ingevoerde letter. Een gemiddelde bezoeker zal denken dat de boel vastgelopen is
Bas Wagenmaker op 08/11/2011 12:19:32:
Ik moet inderdaad nog wat aan de laadtijd doen, maar aangezien er tienduizenden vluchten en bijna tienduizend luchthavens worden vergeleken ben ik allang blij dat hij het binnen 2 seconden doet haha.
Goed kijken naar de explain plan van je query ("EXPLAIN SELECT name FROM gebruiker") en kijken waar hij lang over doet.
Als je nog geen indexes hebt kun je hier wel een duidelijke tutorial vinden:
http://phptuts.nl/view/32/