Ik heb nu een stukje code geschreven die er voor moet zorgen dat wanneer mensen een foto uploaden die van een te lage resolutie is, afgekeurd wordt. HEt probleem is dat mijn pagina uit een array van userfiles bestaat, dus er moet ook een array aan files gecheckt worden. Hoe doe ik dit nu in hemelsnaam?
Zo ver ben ik al:
<?
if ($_FILES['userfile']['tmp_name'][$i] != "") {
$dimensions = getimagesize($_FILES['userfile']['tmp_name'][$i]);
$breedte = $dimensions[0];
$hoogte = $dimensions[1];
if ($breedte < 1600) {
$resolutie[$i] = "{$_FILES['userfile']['name'][$i]} heeft een te lage resolutie! Verwacht geen perfecte afdruk!";
}
if ($hoogte < 1200) {
$resolutie[$i] = "{$_FILES['userfile']['name'][$i]} heeft een te lage resolutie! Verwacht geen perfecte afdruk!";
}
}
?>
wat ik dan niet snap, is dat die $toobig functie wel werkt. Die zou volgens jullie dan toch ook niet mogen werken? Leg me dat nou eens uit. Want volgens mij zit het probleem niet in de while. (ik post dit nu even, omdat ik nu niet bij het script kan, en om dat via ssh te doen is ook zoiets)
@mitch
zo ziet hij er ook uit, ik heb niet het hele script gepost, dat zou een beetje te lang worden
Laat eens een groter deel van het script zien dan, als je er echt alleen
<?
while($i < $getal) {
if ($_FILES['userfile']['size'][$i] > (1024*$maxsize) ) {
$toobig[$i] = "{$_FILES['userfile']['name'][$i]} is te groot! Hij mag niet groter zijn dan $maxsize kb. Kom persoonlijk langs met een CD-ROM.";
}
}
?>
als ik kijk naar wat je in je openingspost wil laten gebeuren. Zoals het er nu staat moet de breedte EN hoogte kleiner zijn dan 1200. Lijkt me dat dat een breedte OF lengte moet worden :)
Verder is de code voor mij te rommelig om aan te beginnen, sorry.
hij is rommelig omdat ik hem rechtstreeks via een ssh stream geplukt heb, ik heb het er het beste van proberen te maken hoor. Zeg nou zelf, hij is nu leesbaar.
dwz dat tabs, spaties, enters en andere opmaak verloren gaat. Dit heb ik via een tunnel en een tcplogger bijelkaar geflanst. Ik moet toch een keer aan remote management gaan denken. (xfree86+vnc of zo, maar dat is weer baaaad voor de serverload)
Ik ben net begonnen met overzetten in een editor met color highlighting en ben tabs toe gaan voegen. Je hebt een $i++ op een plek staat waar hij maar 1 keer langs komt.
Heb je wel al die site doorgelezen die ik al twee keer genoemd heb over foreach() ?
Daarmee kun je echt 10 keer simpeler door een array lopen.
ja, gedaan. Maar ik zoek een maatoplossing voor dit probleem. Anders moet ik 6(!) van dit soort scripts om gaan gooien, zo had ik hem goed aan de praat, totdat ik imagesize ging toevoegen.