Meer dan 1 foto uploaden

$count verhogen?

Een $_GET ongecontroleerd in een path zetten is niet echt veilig.
En wat gebeurt er als je de volgende keer weer 4 foto's upload?

- SanThe - op 15/10/2013 14:57:02

$count verhogen?

Een $_GET ongecontroleerd in een path zetten is niet echt veilig.
En wat gebeurt er als je de volgende keer weer 4 foto's upload?

Ik verwijder eerst een foto voordat er een nieuwe zal worden toegevoegd met hetzelfde nummer. Twee dezelfde nummers zullen dus niet voorkomen. Dus de bestanden zullen altijd 1,2,3 of 4 heten.

Dus het bestand moet de waarde van de loop krijgen $i
Maar hoe doe ik dit?

Dit zou ongeveer moeten zijn wat je vraagt.

<?php
if ($_SERVER['REQUEST_METHOD'] === 'POST' && !empty($_FILES['userfile'])) {
  $uploads_dir = 'uploads/';
  $success = 0;
  $fail = 0;
  foreach($_FILES['userfile']['name'] as $key=>$file) {
    if(empty($_FILES['userfile']['tmp_name'][$key])) {
      // de gebruiker liet de file leeg
      continue;
    }
    // extensie zoeken.  zie http://php.net/manual/en/function.pathinfo.php , voorbeeld drie
    list($dirname, $basename, $extension, $filename) = array_values(pathinfo($file));
    if (in_array(strtolower($extension), array('jpg', 'jpeg', 'gif', 'png', 'bmp'))) {
      // geldige extensie
      $newfile = $uploads_dir . ($key + 1) . "." . $extension;
      $tmp_name = $_FILES['userfile']['tmp_name'][$key];
      if(move_uploaded_file($tmp_name, $newfile)) {
        $success++;
        
      }
      else {
        $fail++;
        echo "Fout foto's<br>";
      }
    }
    else {
      $fail++;
      echo "Invalid Extension<br>";
    }
  }
  if (!$fail) {
    echo 'alles gelukt!';
  }
}
?>
<form method="POST" enctype="multipart/form-data">
  <input name="userfile[]" type="file" ><br>
  <input name="userfile[]" type="file" ><br>
  <input name="userfile[]" type="file" ><br>
  <input name="userfile[]" type="file" ><br>
  <input type="submit" value="Upload all" >
</form>

Merk dus een aantal dingen op:
- jouw manier om extensies te vinden, is niet correct. Een extensie is niet per definitie drie karakters lang.
Pathinfo regelt dat allemaal goed.

- Je hoeft die $i niet in het formulier te steken. eventueel juist als een label.
Als je in een formulier zo-iets zet: name="userfile[]" kan je de $key zien, zoals ik toon in de foreach lus (lijn 6).
In lijn 15 gebruik ik dan die ($key + 1)

Thx. opgelost(-;

Nog 1 vraag. Ik wil de $extensie zo instellen dat er bijvoorbeeld ook .png of andere extensies kijkt of die bestaan. Is dit mogelijk?

$extensie = .jpg || .png

if (file_exists("foto.$extensie")) {

}