[PHP] Variabele buiten functie

[Sp33dFr34k]

OK, wellicht een stomme vraag, maar ik ben nog een noob wat betreft php dus hier gaat ie:

Ik heb een functie uploadPlaatje. Als bestand1 bestaat voert ie de functie uit met als parameter bestand1, voor 2 hetzelfde etc. Dat werkt allemaal, ze worden in de map gezet met een nieuwe bestandsnaam, dat is in dit geval auto met daarachter het eerste getal wat nog niet bestaat in die map. Deze bestandsnaam heet $file_name.

Nu wil ik dat de $file_name voor elk plaatje opgeslagen wordt, maar logischerwijs krijg ik alleen het laatste bestand geprint. Nu is mijn vraag; hoe kan ik ervoor zorgen dat ik alle bestandsnamen netjes op een rijtje krijg om ze vervolgens in de database te zetten?

[mithras]

Heb je problemen met de scope van functies? Ken je het begrip array nog niet? Zonder code valt er weinig te helpen

[Tommasso]

De bestandnamen in een array plaatsen lijkt mij het makkelijkste. daarna moet je ze invoeren in de tabel (je database)

[Alexji]

zoiets dus:

PHP:

$fileNames = array();

// na elke upload

$fileNames[] = $fileName;

//daarna uitlezen


foreach ($fileNames as $fileName){
//insert into DB
    echo $fileName;
}

[Sp33dFr34k]

Danku very much, ik ga het nu proberen.

[ Voor 44% gewijzigd door Sp33dFr34k op 08-11-2009 23:21 ]

[Sp33dFr34k]

Dacht ik al, net als met een variabele gooit ie de array leeg aan het begin van de functie. Ik zal wel erg dom aan het denken zijn, maargoed ik kom er gewoon ff niet uit...

Hieronder de code zoals ik 'm nu heb.

PHP:

<?
function uploadBestand($bestand) {
$fileNames = array();
//De groote van het bestand vergelijken met het maximum
if ($_FILES[$bestand]['type'] > 302400){
    exit("Je bestand " . $_FILES[$bestand]['size'] . " bytes, het maximum toegelaten is 302400 bytes");
} 

if($_FILES[$bestand]['type'] == "image/jpeg" || $_FILES[$bestand]['type'] == "image/pjpeg") {

    $x = strtolower(substr($_FILES[$bestand]['name'], -3));
    // De naam van de file veranderen in bv. image0
    $file_name = "auto";    
    $n = 0;
    // Check of file al bestaat, zo ja, filename verhogen 
    while (file_exists('autos/' . $file_name . $n . '.' . $x)) 
    { 
        $n = $n + 1; 
    } 
    $file_name = $file_name . $n; 

    move_uploaded_file($_FILES[$bestand]['tmp_name'],'autos/' . $file_name . '.' . $x);
    chmod ('autos/' . $file_name . '.' . $x, 0777);
    //kiezen
    $b = imagecreatefromjpeg('autos/' . $file_name . '.' . $x) or die ("Jpg probleempje");
    $bx = imagesx($b); // Bron breedte 
    $by = imagesy($b); // bron hoogte
    $lm = $b;
    $width=600;
    $temp=$bx/600;
    $height=$by/$temp;
    if ($bx > $width) {
        $lm1 = imagecreatetruecolor($width ,$height) or die ("Probleem met maken van plaatje jpg");
        imagecopyresampled($lm1, $lm, 0, 0, 0, 0, $width, $height, $bx, $by);
        Imagejpeg($lm1,'autos/' . $file_name . '.' . $x,80); //80 is jpg kwaliteit
        imageDestroy($lm1);
        imageDestroy($lm);
    }
    echo "Succesvol opgeslagen<br />";
    $file_name = $file_name . ".jpg";
    $fileNames[] = $file_name;
    } else {
    echo "";
}
print_r($fileNames);
}

if (isset($_FILES['bestand1'])) {
    uploadBestand(bestand1);
}
if (isset($_FILES['bestand2'])) {
    uploadBestand(bestand2);
}
if (isset($_FILES['bestand3'])) {
    uploadBestand(bestand3);
}
if (isset($_FILES['bestand4'])) {
    uploadBestand(bestand4);
}
?>
<form action="" enctype="multipart/form-data" method="post">
<input type="file" name="bestand1"><br>
<input type="file" name="bestand2"><br>
<input type="file" name="bestand3"><br>
<input type="file" name="bestand4">
<input type="submit" name="submit" value="Upload">
</form>

[Alexji]

Dit komt omdat je de array binnen de functie definieert. Dus dan wordt elke keer als je de functie aanroept de array opnieuw aangemaakt. Zoek maar eens op variable scopes.

ook lijkt me niet dat je met

PHP:

if ($_FILES[$bestand]['type'] > 302400){

de grootte van de file krijgt.

[Sp33dFr34k]

Kwam ik net ook achter, maar als ik de array buiten de functie declareer is de output hetzelfde..

De rest van het script is overigens gejat, probeer hem alleen zo te maken dat ik meerdere files kan uploaden ipv 1 tegelijk en dat ik de namen meteen in een regel van mijn database kan zetten...

[Alexji]

Wat is de output nu dan als je 3 bestanden upload?

[Sp33dFr34k]

Succesvol opgeslagen
Array ( [0] => auto0.jpg )
Succesvol opgeslagen
Array ( [0] => auto1.jpg )
Succesvol opgeslagen
Array ( [0] => auto2.jpg )

[sky-]

Buiten de functie kan je met :

PHP:

<?php
global $fileNames = array();
function uploadBestand($bestand) {
#$fileNames = array();
//De groote van het bestand vergelijken met het maximum
if ($_FILES[$bestand]['type'] > 302400){
    exit("Je bestand " . $_FILES[$bestand]['size'] . " bytes, het maximum toegelaten is 302400 bytes");
}?>

De array defineren.

[ Voor 3% gewijzigd door sky- op 08-11-2009 23:47 . Reden: Zin vergeten. ]

[Sp33dFr34k]

Dat had ik idd al geprobeerd, maar global arrays vindt ie blijkbaar niet zo leuk...

Parse error: parse error, unexpected '=', expecting ',' or ';' on line 2

[Alexji]

Volgens de PHP manual moet het ook zijn

PHP:

<?php

$fileNames = array();

function uploadBestand($bestand) {

global $fileNames;

//De groote van het bestand vergelijken met het maximum
if ($_FILES[$bestand]['type'] > 302400){
    exit("Je bestand " . $_FILES[$bestand]['size'] . " bytes, het maximum toegelaten is 302400 bytes");
}


//etc
$file_name = $file_name . ".jpg"; 
$fileNames[] = $file_name; 
?>

[ Voor 11% gewijzigd door Alexji op 08-11-2009 23:55 ]

[Patriot]

Dat komt omdat je de variabele buiten de functie moet declareren, en binnen de functie met het global-keyword aan moet geven dat hij die variabele uit de globale scope moet halen. Regel twee mag dus naar een positie binnen de functie (net na regel 3 bijvoorbeeld).

edit:
grom @ hierboven

[ Voor 5% gewijzigd door Patriot op 08-11-2009 23:55 ]

[Sp33dFr34k]

Fixed!

Buiten functie:

PHP:

$fileNames = array();

Dan in de functie:

PHP:

global $fileNames;
$fileNames[] = $file_name;

Krijg nu als output bij 2 plaatjes:

Array ( [0] => auto0.jpg [1] => auto1.jpg )

Woehoe, kijken of de rest ook gaat lukken

Dank voor jullie hulp allen!

EDIT: Haha, allemaal tegelijk ineens met de oplossing, nogmaals dank!

[ Voor 10% gewijzigd door Sp33dFr34k op 08-11-2009 23:57 ]

[NMe]

Misschien wat mosterd na de maaltijd, maar het woord "scope" werd al in de eerste reactie genoemd. Je hebt dat vervolgens nog 2-3 reacties lang genegeerd. Kun je de volgende keer asjeblieft wat inzet tonen door zelf ook te gaan zoeken op de termen die je op een presenteerblaadje krijgt?

[Sp33dFr34k]

Klopt NMe, je hebt helemaal gelijk, mijn excuses. Heb scope gegoogled en zag toen even niet waarom dat relevant was, maar uiteindelijk was de 1e post al meteen de oplossing achteraf gezien. Ik zal er de volgende keer beter op letten.

[mithras]

En daarnaast, al geef ik de hint scope, het gebruik van global zou ik toch vermijden. Je kan het ook (en imho beter) oplossen met dergelijke constructies:

PHP:

$list = array();
foreach ($files as $file) {
  $list[] = uploadBestand($file);
}

Of, als je de functie de stack wil laten controleren:

PHP:

$list = array();
foreach ($files as $file) {
  uploadBestandEnVoegToe($file, $list);
}

[Woy]

Idd, het gebruik van global zou ik ook zo veel mogelijk vermijden. Misschien wil je de functie later ook nog eens in een andere context gebruiken, en dan heeft de functie nog andere effecten dan dat je wil. Door gewoon expliciet de return value in een array te stoppen, ben je later veel flexibeler. Misschien wil je immers op een andere pagina wel 2 array's bij gaan houden voor verschillende bestands type's.

[MBV]

Verder wil ik nog de suggestie geven om je indenting op orde te brengen. Als je een nieuwe scope begint, moet je ook de indenting vergroten:

PHP:

function x() {
  $var = 5;
  if (iets()) {
    $var = 8;
  }
  return $var;
}

$var2 = x();
echo $var2;

Dat maakt het voor jouw, en voor ons, een stuk makkelijker om je code te lezen. Ik had in eerste instantie niet eens gezien dat je een functie declareerde bovenaan, omdat je daarna verder ging zonder indenting.