Hey,

Ik heb een uploadscript gedownload. Werkt allemaal goed, maakt thumbnails aan enz. Nu wil ik de namen van de afbeeldingen opslaan in mijn database. Hoe pak ik de naam van een afbeelding na het uploaden?

Ik dacht het te doen met;

PHP Code:

{$_POST['uplimg1']} 


PHP Code:

{$_POST['uplimg2']} 


enz enz..

PHP Code:

############################### HTML FORM
    
        $form_img .= '
        <label>Image 1: </label> <input type="file" name="uplimg1"><br />
        <label>Image 2: </label> <input type="file" name="uplimg2"><br />
        <label>Image 3: </label> <input type="file" name="uplimg3"><br />
        <label>Image 4: </label> <input type="file" name="uplimg4"><br />';
    
    $htmo .= '
        <p>'.$feedback.'</p>
        <form method="post" enctype="multipart/form-data">
            '.$form_img.' <br />
            <input type="submit" value="Upload Afbeeldingen" style="margin-left: 50px;" />
        </form>
        ';    

    echo $htmo;
    
    $sql = "UPDATE aanbod SET foto1 = '{$_POST['uplimg1']}', foto2 = '{$_POST['uplimg2']}', foto3 = '{$_POST['uplimg3']}', foto4 = '{$_POST['uplimg4']}' WHERE id = '$id' ";
    echo $sql;
    //mysql_query($sql) or die(mysql_error()); 


Maar dat werkt niet. Hoe moet het wel?

PHP Code:

<?
echo $_FILES['uplimg1']['name'];
echo $_FILES['uplimg2']['name'];
echo $_FILES['uplimg3']['name'];
echo $_FILES['uplimg4']['name'];
?>

dus:

PHP Code:


$sql = "UPDATE aanbod SET foto1 = '".$_FILES['uplimg1']['name']."', foto2 = '".$_FILES['uplimg2']['name']."', foto3 = '".$_FILES['uplimg3']['name']."', foto4 = '".$_FILES['uplimg4']['name']."' WHERE id = '".$id."' "; 


Dat werkt :) super! Bedankt!

Ik zou ook even kijken naar je tabelstructuur, foto1, foto2, foto3, foto4 is niet echt netjes. Als er 1 foto is, heb je 3 lege velden en als je (later) meer foto's wilt toestaan, moet je je script weer aanpassen.

Waarom zou dat geen goede oplossing zijn als je altijd maar max. 10 foto's hebt? En lege velden in een record komen wel vaker voor toch? Niet alle velden zijn verplicht in te vullen..

Origineel gepost door Ton van Alebeek

Waarom zou dat geen goede oplossing zijn als je altijd maar max. 10 foto's hebt? En lege velden in een record komen wel vaker voor toch? Niet alle velden zijn verplicht in te vullen..

Als je max. 10 foto's mag uploaden, ga je niet 10 velden maken. Je maakt een aparte tabel "fotos" met 2 velden, in dit geval iets van aanbod_id en foto. Je kunt nu zoveel foto's als je maar wilt koppelen. Zo heb je geen nutteloze ruimte in de database en ben je veel flexibeler.

@ Ton: Dit is zeker waar, het is niet verplicht om alle velden in te vullen ;)
Maar dan verspil je dus wel een ton onbenutte ruimte :)

Het zal waarschijnlijk zo ook wel werken, de mannier van Peter is gewoon veel 'netter' en zal na verloop van tijd (hoe voller de database, hoe trager) ook beter werken!

Die manier is inderdaad netter. Maar veel records zullen er niet in de db komen, en dit is zo de snelste oplossing.
Bedankt voor jullie hulp.