Site check: rate or hate!

Misschien een andere font? Calibri werkt niet op iedere computer

Waarom heb je niet standaard de vliegvelden en datum picker erin zitten. Datum picker is gratis en kan d.m.v. JqueryUI. Vliegvelden heeft de IATA een database van.

- 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.

Layout is aardig, implementatie wat minder. Automatisch aanvullen is essentieel. En escape je je data wel goed? Waarom krijg ik resultaten als ik onderstaande code invul bij "Naar:"?

' 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:

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

Bedankt voor de reacties, de site is inderdaad nog niet af vandaar dat ik hier om een review vroeg!

@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.

1e reactie: Mooi, overzichtelijk, in 1 blik duidelijk wat het doel van de site is.

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:"?

' or '1'='1

Dit is dus SQL-Injection.

@Tobias
Heb je een alternatief voor shorturls? Die dus niet direct naar mijn site gaan maar naar bijvoorbeeld cheaptickets.nl?

[size=xsmall]Toevoeging op 02/11/2011 21:08:40:[/size]

$-SanThe-
SQL injection meteen verholpen, kun je dit testen?

[size=xsmall]Toevoeging op 02/11/2011 22:09:36:[/size]

Datepicker ook toegevoegd!

Geef de shorturl gewoon een naam en zit de link in de href: <a href="bit.ly/vfpQPy">Laatste Tweet</a>

Toevoeging:
Je kunt er natuurlijk ook een javascript-oplossing voor maken, dan is de link helemaal niet meer te zien

Dat was inderdaad niet zo'n probleem geweest als de links niet direct van twitter af kwamen, dus ik kan er niks aan veranderen. Snap je?

Automatisch aanvullen trouwens ook toegevoegd.

Bij je automatisch aanvullen zou je beter een limiet zetten. Bij mij is de lijst zo lang dat je background zich herhaalt en het duurt ook lang om te laden.