meerdere foto's uploaden

mitchellve · 24 Januari 2015

Ik zit met een probleem in mijn script.
Ik wou mijn php kennis verbeteren. en een script maken voor meerdere foto's te uploaden in 1 keer. Het script werkt niet volledig juist.
Het probleem zit hem als ik foto's selecteer. zal hij deze via de tmp map naar de juist locatie brengen. maar in de database gebeurt er iets raar... Normaal gezien moet hij altijd het pad opslagen in 1 rij. (uniek ID) maar hij slaagt alle paden op van de foto's in meerdere rijen.
bv:
foto 1 geupload pad => upload/foto1.png
foto 2 geupload pad => upload/foto2.png upload/foto1.png
en zou gaat dit maar voor tot het maximum aantal char word bereikt.
Dit is mijn code:

PHP

<?php
if (isset($_POST['submit'])) {
 $j = 0; //Variable for indexing uploaded image 


 $target_path = "upload/album/"; //Declaring Path for uploaded images
 for ($i = 0; $i < count($_FILES['file']['name']); $i++) 
 {
    


 
 //loop to get individual element from the array


 $validextensions = array("jpeg", "jpg", "png", "JPG"); //Extensions which are allowed
 $ext = explode('.', basename($_FILES['file']['name'][$i]));//explode file name from dot(.)
 $file_extension = end($ext); //store extensions in the variable


 $target_path = $target_path . md5(uniqid()) . "." . $ext[count($ext) - 1];//set the target path with a new name of image
 $j = $j + 1;//increment the number of uploaded images according to the files in array 


 if (($_FILES["file"]["size"][$i] < 1000000000000000) && in_array($file_extension, $validextensions)) 
 {
 if (move_uploaded_file($_FILES['file']['tmp_name'][$i], $target_path)) 
 {                                                                       //if file moved to uploads folder
 echo $j. ').<span id="noerror">foto uploaded is gelukt!.</span><br/>';
 $album = $dbh->prepare("insert into albumfoto(naam,album) values (?,?)");
 $naam_album=$_POST['albums'];
 $nam = "<a href='" .$target_path . "'class='top_up'><img class='album_foto' id='P" .substr($_FILES["file"]["name"][$i],0,strpos($_FILES["file"]["name"][$i],".")) ."'  src='" . $target_path . "' width='110' height='90'></img></a>";
 $album->bindParam(1,$nam);
 $album->bindParam(2, $naam_album); 
 $album->execute();
 }
 else 
 {                                   //if file was not moved.
 echo $j. ').<span id="error">probeer het nog eens. blijf dit zich voordoen contacteer webdesigner.</span><br/> ';
 }
 } 
 else 
 {//if file size and file type     was incorrect.
 echo $j. ').<span id="error">***een te grote foto of verkeerde extensie! alleen fotos ***</span><br/>';
 }
 }
 }
?>

Toon Meer

ziet iemand de fout?

groetjes,
Mitchell

T.Nijborg · 25 Januari 2015

Een aantal vragen/punten die ik niet helemaal snap

Het lijkt erop alsof je een volledige style/img tag wil opslaan in de database, dat is sowieso al niet slim. Hou alle style, html tags etc zoveel mogelijk uit de php code en database. In dit geval sla gewoon op waar de foto staat.
Ik zie geen html form of iets in die richting. Hier kan uiteraard ook wat mis zijn.
Ik zie ook dat je de maximale bestand grote heel hoog hebt staan. Dit heeft met z'n groot nummer niet echt zin om te controleren.
Code
```
$target_path = $target_path . md5(uniqid()) . "." . $ext[count($ext) - 1];
```
Geen idee waarom die - 1 maar die is hier niet nodig volgens mij?! Kan dit je probleem zijn?

Ik hoop dat je eruit kan komen

Gr,
Tim

mitchellve · 25 Januari 2015

@mentimjojo

Ik heb dit gedaan omdat anders waar de foto's worden uitgelezen er langer over doen als ik dit in mijn css & javacript file doe.

PHP

<form enctype="multipart/form-data" action="albummaken.php" method="post">


 <div id="filediv">
 <?php
if (isset($_POST['submit'])) {
 $j = 0; //Variable for indexing uploaded image 


 $target_path = "upload/album/"; //Declaring Path for uploaded images
 for ($i = 0; $i < count($_FILES['file']['name']); $i++) 
 {
    


 
 //loop to get individual element from the array


 $validextensions = array("jpeg", "jpg", "png", "JPG"); //Extensions which are allowed
 $ext = explode('.', basename($_FILES['file']['name'][$i]));//explode file name from dot(.)
 $file_extension = end($ext); //store extensions in the variable


 $target_path = $target_path . md5(uniqid()) . "." . $ext[count($ext) - 1];//set the target path with a new name of image
 $j = $j + 1;//increment the number of uploaded images according to the files in array 


 if (($_FILES["file"]["size"][$i] < 1000000000000000) && in_array($file_extension, $validextensions)) 
 {
 if (move_uploaded_file($_FILES['file']['tmp_name'][$i], $target_path)) 
 {                                                                       //if file moved to uploads folder
 echo $j. ').<span id="noerror">foto uploaded is gelukt!.</span><br/>';
 $album = $dbh->prepare("insert into albumfoto(naam,album) values (?,?)");
 $naam_album=$_POST['albums'];
 $nam = "<a href='" .$target_path . "'class='top_up'><img class='album_foto' id='P" .substr($_FILES["file"]["name"][$i],0,strpos($_FILES["file"]["name"][$i],".")) ."'  src='" . $target_path . "' width='110' height='90'></img></a>";
 $album->bindParam(1,$nam);
 $album->bindParam(2, $naam_album); 
 $album->execute();
 }
 else 
 {                                   //if file was not moved.
 echo $j. ').<span id="error">probeer het nog eens. blijf dit zich voordoen contacteer webdesigner.</span><br/> ';
 }
 } 
 else 
 {//if file size and file type     was incorrect.
 echo $j. ').<span id="error">***een te grote foto of verkeerde extensie! alleen fotos ***</span><br/>';
 }
 }
 }
?>
  <h2>Foto's uploaden in album</h2>
  <h5> Kies je album waar de foto's naartoe moeten </h5>
 
   <select name="albums"> 
<?php
    $album =  $dbh->query("SELECT naam FROM album");
    foreach($album as $var){
        echo "<option name='album_naam' value='" . $var['naam'] . "'>";
        echo $var["naam"];
        echo "</option>";
   }
?>
 </select>
  <br>
  <br>
 <input name="file[]" type="file" multiple id="file"/></div>
 <br/>
 <input type="submit" value="Upload de foto's" name="submit" id="upload" class="upload"/>
 </form>

Toon Meer

Ik heb dit gedaan om te testen dat daar de fout lag... dus dat mag eigenlijk genegeerd worden
als ik deze -1 wegdoen lukt het helemaal niet

hij zegt dat het probleem op deze lijn zit:

Code

if (move_uploaded_file($_FILES['file']['tmp_name'][$i], $target_path))

ik zie toch niks fout bij deze lijn...

heb een foto toegevoegd wat er in mijn database komt te staan. je kan zien dat hij meerdere foto's in 1 rij opslaat.

groetjes,
Mitchell

T.Nijborg · 25 Januari 2015

Ja dat vermoed ik ook, om een of andere reden zet hij de variable $target_path niet opnieuw. Probeer is als je data is opgeslagen, binnen de for loop even unset($target_path) te doen. Op deze manier, wanneer de for loop rij 2 gaat uitvoeren is $target_path leeg.

mitchellve · 25 Januari 2015

@mentimjojo
Na mijn PDO?

T.Nijborg · 25 Januari 2015

@mitchellve Na dat je er klaar mee bent, dus het beste is aan het einde van je for loop

mitchellve · 25 Januari 2015

Beste,

Oke ik ga dit vanavond doen. heb morgen examen.

bedankt!,
Mitchell

mitchellve · 25 Januari 2015

@mentimjojo
de unset doet het. heb wel even de declaratie van target_path terug in mijn for loop gestoken.
Hier is mijn code zodat andere mensen het kunnen gebruiken:

PHP

<form enctype="multipart/form-data" action="albummaken.php" method="post">


 <div id="filediv">
 <?php
if (isset($_POST['submit'])) {
 $j = 0; //Variable for indexing uploaded image 


 for ($i = 0; $i < count($_FILES['file']['name']); $i++) 
 {
  $target_path = "upload/album/"; //Declaring Path for uploaded images   


 
 //loop to get individual element from the array


 $validextensions = array("jpeg", "jpg", "png", "JPG"); //Extensions which are allowed
 $ext = explode('.', basename($_FILES['file']['name'][$i]));//explode file name from dot(.)
 $file_extension = end($ext); //store extensions in the variable


 $target_path = $target_path . md5(uniqid()) . "." . $ext[count($ext) - 1];//set the target path with a new name of image
 $j = $j + 1;//increment the number of uploaded images according to the files in array 


 if (($_FILES["file"]["size"][$i] < 1000000000000000) && in_array($file_extension, $validextensions)) 
 {
 if (move_uploaded_file($_FILES['file']['tmp_name'][$i], $target_path)) 
 {                                                                       //if file moved to uploads folder
 echo $j. ').<span id="noerror">foto uploaded is gelukt!.</span><br/>';
 $album = $dbh->prepare("insert into albumfoto(naam,album) values (?,?)");
 $naam_album=$_POST['albums'];
 $nam = "<a href='" .$target_path . "'class='top_up'><img class='album_foto' id='P" .substr($_FILES["file"]["name"][$i],0,strpos($_FILES["file"]["name"][$i],".")) ."'  src='" . $target_path . "' width='110' height='90'></img></a>";
 $album->bindParam(1,$nam);
 $album->bindParam(2, $naam_album); 
 $album->execute();
 }
 else 
 {                                   //if file was not moved.
 echo $j. ').<span id="error">probeer het nog eens. blijf dit zich voordoen contacteer webdesigner.</span><br/> ';
 }
 } 
 else 
 {//if file size and file type     was incorrect.
 echo $j. ').<span id="error">***een te grote foto of verkeerde extensie! alleen fotos ***</span><br/>';
 }
 unset($target_path);
 }
 }
?>
  <h2>Foto's uploaden in album</h2>
  <h5> Kies je album waar de foto's naartoe moeten </h5>
 
   <select name="albums"> 
<?php
    $album =  $dbh->query("SELECT naam FROM album");
    foreach($album as $var){
        echo "<option name='album_naam' value='" . $var['naam'] . "'>";
        echo $var["naam"];
        echo "</option>";
   }
?>
 </select>
  <br>
  <br>
 <input name="file[]" type="file" multiple id="file"/></div>
 <br/>
 <input type="submit" value="Upload de foto's" name="submit" id="upload" class="upload"/>
 </form>

Toon Meer

T.Nijborg · 26 Januari 2015

@mitchellve Top! Dat betekent dus dat je variabele niet opnieuw gezet werd maar dat de tekst erbij werd gezet. unset() kun je in princiepe overal voor gebruiken!

mitchellve · 26 Januari 2015

@mentimjojo weeral iets bijgeleerd vandaag

T.Nijborg · 26 Januari 2015

@mitchellve Mooizo ik snap alleen niet waarom hij de variabele niet opnieuw zetten maar er bij gaat schrijven. Ik weet dat het volgende zet.

Code

$variabele = "foto/";

Dit zou de variebele moeten zetten, dus als je dat op nieuw doet in een for loop verwijderd hij de huidige waarde en zet hij de nieuwe, tenzij je het volgende doet:

Code

$variabele += "foto/"

Dit schrijft er bij om het moment dat je dit in een for loop stopt. Maarja het werkt!

mitchellve · 26 Januari 2015

@mentimjojo
Ik vind het ook vreemd wat deze for loop doet.
Maar ja het belangrijkste is dat het werkt

groetjes,
Mitchell

DNS update Directadmin

PC probleem

Let op Vimexx klanten

wie kan mij helpen met Security aanscherpen voor nmijn website

Ik ben op zoek naar een goede hosting

App bouwer gezocht

4G met USB Dongle - windows 11 Chrome of Firefox - browser probleem

Hosting en websites op maat

Xenforo buckaroo plugin

300+ Nieuwe domeinnamen Maart 2024

Actieve crime

meerdere foto's uploaden

Participate now!

ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

Het Belang van Standaardisatie

De Revolutionaire Rol van IT in Alzheimeronderzoek

Functioneel ontwerp

Access Control List implementatie in PHP/MySQL - deel 1/2

Access Control List implementatie in PHP/MySQL - deel 2/2

Delen