[PHP] Functie schrijven

[SoeperKees]

Ik zit met het volgende.

Ik gebruik dit script al enige tijd, en deze zet ik in het desbetreffende php bestand. Maar nu heb ik laatst met een project van school, dit in veel meer bestanden moeten doen. Zo ontdek je een fout en kun je alle bestanden weer af gaan etc...jullie snappen het.

Nu wil ik van dit alles een functie maken zodat ik het maar 1 keer hoef aan te passen ipv 3-4 keer.

Ik heb kort geleden een afkap functie gemaakt voor stukjes tekst....maar meer ook niet. Hoe ik dit aan moet pakken zou ik niet weten.

Kunnen jullie mij in de juiste richting helpen?

PHP:

// Toevoegen

$types['L'] = array ( "image/pjpeg", "image/jpeg", "image/gif", "image/png", "image/xpng" ); // mime types die toegelaten zijn

if ($_GET['actie'] == "toevoegen" || $_POST['actie'] == "toevoegen")
{

        if ($_POST["bericht"])

            {
                        //de mappen opgeven
                        $map = "../fotos/"; //foto map
                        $thumb_map = "thumbs/"; //thumbnail map
                        if(in_array($_FILES['foto_bestand']['type'] , $types['L']))
                        {

                            if (isset($_FILES['foto_bestand']))
                            {
                                $bericht = $_POST['bericht'];
                                $datum = $_POST['datum'];

                                //Bestand een extentie geven, in dit gevalt .jpg
                                $extentie = ".jpg";

                                $bericht  = addslashes($bericht);
                                

                                //$new_foto_file = $bestands_naam . $extentie;
                                $new_foto_file = md5(rand() * time()) . $extentie;
                                $new_thumb_file = "thumb_". $new_foto_file;

                                //temp bestanden worden in string gezet
                                $fotosrc = $_FILES['foto_bestand']['tmp_name'];
                                $thumbnailsrc = $_FILES['foto_bestand']['tmp_name'];

                                //Hoogte en Breedte van de thumbnails
                                $thumb_y = 166;
                                $thumb_x = 166;

                                //Hoogte en Breedte van de grote foto's
                                //$user_hoogte = 450;
                                $user_breedte = 600;

                                //list voor de grote foto
                                list($breedte, $hoogte) = getimagesize($fotosrc);
                                
                                $foto_ratio = $hoogte / $breedte;
                                
                                if($foto_ratio > 1)
                                {
                                $breedte2 = $user_breedte;
                                $hoogte2 = $user_breedte * $foto_ratio;
                                }

                                else
                                {

                                $hoogte2 = $user_breedte;
                                $breedte2 = $user_breedte / $foto_ratio;                

                                }

                                //list voor de thumbnail foto
                                list($srcx_t, $srcy_t) = getimagesize($thumbnailsrc);

                                if($srcx_t <= $srcy_t)
                                {
                                $dstx_t = $thumb_x;
                                $dsty_t = round($srcy_t / ($srcx_t/$thumb_x));
                                }

                                else
                                {
                                $dstx_t = round($srcx_t / ($srcy_t/$thumb_y));
                                $dsty_t = $thumb_y;
                                }

                                $thumbx = $thumb_x;
                                $thumby = $thumb_y;

                                $image_thumbnail = imagecreatetruecolor($thumb_x, $thumb_y);
                                $image_foto = imagecreatetruecolor($breedte2, $hoogte2);

                                $image = imagecreatefromjpeg($fotosrc);
                                $image_t = imagecreatefromjpeg($thumbnailsrc);

                                imagecopyresampled ($image_thumbnail, $image_t, 0, 0, 0, 0, $thumbx, $thumby, $srcx_t, $srcy_t);
                                imagecopyresampled ($image_foto, $image, 0, 0, 0, 0, $breedte2, $hoogte2, $breedte, $hoogte);


                                //mappen en namen opgeven voor de foto's
                                $locatie_foto = $map . $new_foto_file;
                                $locatie_thumb = $map . $thumb_map .$new_thumb_file;


                                // Output
                                imagejpeg($image_thumbnail, $locatie_thumb, 80);
                                imagejpeg($image_foto, $locatie_foto, 100);

                                    //Query die de gegevens toevoegt in database
                                    $iFotoQ = mysql_query ("INSERT INTO fotoalbum (foto_url,
                                    thumb_url,
                                    bericht,
                                    datum) 
                                    VALUES ('".$new_foto_file."',
                                    '".$new_thumb_file."',
                                    '".$bericht."',
                                    '".$datum."')");

                                     if ($iFotoQ)
                                         {
                                            echo "<script type='text/javascript'>location.href='fotoalbum.php';</script>";
                                         }

                                     else
                                        {
                                            echo "<center>";
                                            echo"Hij doet het niet";
                                            echo "</center>";
                                        }


                            }


                        }
                        else
                        {
                            echo'<div>De extensie is niet toegelaten. Het MOET een foto zijn!</div>';
                        }


            }
                    echo "<div class=\"content2\">";
                    echo "<h2>Foto toevoegen</h2>";
                    $datum = date("Y-m-d H:i:s");

                    if ($_POST['submit'])
                    echo "<h2>Foto toevoegen</h2>";
                    echo "<form enctype=\"multipart/form-data\" actie=\"fotoalbum.php\" method=\"POST\">";
                    //echo "<input  class=\"nothing\" type=\"hidden\" name=\"MAX_FILE_SIZE\" value=\"4096000\" />";
                    echo "<input  class=\"nothing\" type=\"hidden\" name=\"actie\" value=\"add\">";

                    echo "<b>Korte beschrijving bij de foto:</b>";
                    echo "<textarea rows=\"2\" name=\"bericht\" cols=\"40\"></textarea> ";

                    echo "<input  class=\"nothing\" type=\"hidden\" size=\"19\" maxlength=\"19\" value=\"$datum\" name=\"datum\">";

                    echo "<b>Foto:</b>";
                    echo "<input name=\"foto_bestand\" type=\"file\"><br />  ";

                    echo "<input class=\"button\" type=\"submit\" value=\"Verstuur Foto\" />    ";

                    echo "</form>";
                    
                    echo "</div>";
}

[Voutloos]

Zet het in een lost bestand, zet er 'function wist_ik_het_maar() { ' en '}' omheen, include het waar je het nodig hebt en roep de functie aan?

[Verwijderd]

Leer progammeren. Dit is toch geen topic meer.

Je moet niet meteen een compleet werkende applicatie willen maken

Je zult honderden stukjes script moeten schrijven om te leren hoe je zoiets moet maken. Dat heeft iedereen hier gedaan, en dat kan jij ook doen, als je maar wilt.

[Verwijderd]

Voutloos schreef op donderdag 03 juli 2008 @ 21:24:
Zet het in een lost bestand, zet er 'function wist_ik_het_maar() { ' en '}' omheen, include het waar je het nodig hebt en roep de functie aan?

Dan kan je net zo goed deze code in een bestand zetten, zonder functie, en dat includen...want als ik het goed begrijp heeft de TS deze code dus in tig verschillende bestanden ge copy-paste atm


Als tip voor de TS: als je er een functie van wilt maken, dan moet je eerst de code eruit filteren die opzich niks met het af te handelen proces te maken heeft (zoals opmaak, html output etc).

Je wilt dus uiteindelijk een functie hebben met als output een boolean (resultaat; is het toevoegen gelukt) en als input een bericht, afbeelding, ID, etc... bijvoorbeeld.

[ Voor 31% gewijzigd door Verwijderd op 03-07-2008 21:30 ]

[Laurens-R]

Je opmaak mag ook een tikkie beter. Ik heb nu op sommige plekken nog het gevoel dat ik naar een rechte lijst zit te kijken, wat niet ten goede komt van de leesbaarheid.

Netjes inspringen na accolades dus, maar ook voorkomen dat je te ver inspringt.

Niets ernstigs, maar wel zo netjes!

[Creepy]

Sorry hoor, maar dit soort vragen gaan toch echt net even te ver. Een functie maken is echt de basis van programmeren. Als je dat al niet onder de knie hebt en zo aan je topicstart te zien wil je geen of erg weinig moeite doen om het zelf onder de knie te krijgen, dan houdt het voor hier in elk geval toch echt op.