file Downloader

ziet er goed uit:)

Ziet er niet goed uit!!!

Je gebruikt met file() een voorgedefinieerde php functie. Niet doen dus, misschien dat het prima werkt hoor, maar zorg dat je hiervoor niet de naam van een voorgedefinieerde php functie gebruikt.

Verder ziet het er prima uit hoor :)

Hoezo is het fout om een voorgedefinieerde functienaam te pakken. In PHP is een functie en een klasse bij mijn weten totaal iets anders (in tegenstelling tot Javascript) en kan het geen kwaad. Dus, vertel eens, wat is jouw motief om het af te raden?

Wat ik alleen nog kan aanraden is fpassthru in plaats van file_get_contents. Schijnt minder geheugen te gebruiken en daarmee sneller te zijn.

Ik zeg ook niet dat het fout is, maar het is netter (vind ik) om daar een naam voor te pakken zodat je je niet kan vergissen. Deze class kan je bijvoorbeeld net zo goed get_File noemen, hij gaat toch die file voor je "ophalen". Dus hij omschrijft de Class ook nog beter met een andere naam.

Ik snap dat het prima werkt zo, dus het is geen groot probleem, (kwam denk ik wel anders over he :S) maar voor de duidelijkheid van je code kan het wel eens handig zijn.

Je hebt helemaal gelijk Robert, nooit bij nagedacht, is nu ook ge-update.
Het is ook niet duidelijk om dat te doen.
@Jelmer; bedankt voor de tip, ook ge-update!

Sorry, ik had het fout, ik haal fpassthru & readfile door elkaar. readfile is nog net even iets beter in dit geval dan fpassthru. Let wel dat beide functies niet het bestand zelf inladen, maar als het ware 'doorsturen'. Dus $content zal niet de echte inhoud bevatten. strlen zou je dan dus ook moeten vervangen door bijvoorbeeld filesize.

De header Content-Length hoef je niet eens te sturen :S Doe je dat ooit? Ooit in je PHP scripts? Nee, en toch komt-ie altijd in de response headers terecht!
Automatisch dus. Content-Length kan je dus weglaten, is weer wat minder process.
Content-Transfer-Encoding voegt ook ernstig weinig toe (waar heb je deze headers vandaan man? Op php.net staat een letterlijk voorbeeld.).
Heel die cache control lijkt me ook niet nodig (misschien wel niet slim, want twee keer hetzelfde bestand downloaden is zonde, 1 keer uit cache halen wel zo efficient).
Leuke class, maar beetje overdreven denk ik... Het kan zonder class in 5 regels ofzo... Denk dat je er beter een functie van kan maken. En dan met readfile() omdat die geen file handler nodig heeft.

Mooi script hoor ;)