Discussion :

[roleca]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
if(count($resu_etu) > 0) {
 
	    //sauvegarde l'image en PNG	
            $image = base64_decode(substr($_POST['photo'], strpos($_POST['photo'], ",")+1));
            $fp = fopen(PHOTOS_DIR.$resu_etu[0]["numEtudiant"].'.png', 'wb');
            fwrite($fp, $image);
            fclose($fp);
	    //ouvre le PNG
    	    $imagepng = imagecreatefrompng(PHOTO_DIR.$result_etu[0]["numEtudiant"].'.png');
	    //convertit le PNG en JPG
	    imagejpeg($imagepng, PHOTO_DIR.$result_etu[0]["numEtudiant"].'.jpg', 100);
	    //détruit le PNG
            unlink(PHOTOS_DIR.$resu_etu[0]["numEtudiant"].'.png');
 
            echo "0";
    }
 
    else {
        $oDatabase->connectionSMS();
        $result_etu = $oDatabase->fetchDb($query);
        if(count($result_etu) > 0) { // => j'ai trouvé quelque chose
 
            //sauvegarde l'image en PNG 
            $image = base64_decode(substr($_POST['photo'], strpos($_POST['photo'], ",")+1));
            $fp = fopen(PHOTOS_DIR.$result_etu[0]["numEtudiant"].'.png', 'wb');
            fwrite($fp, $image);
            fclose($fp);
            //ouvre le PNG
            $imagepng = imagecreatefrompng(PHOTO_DIR.$resu_etu[0]["numEtudiant"].'.png');
            //convertit le PNG en JPG
            imagejpeg($imagepng, PHOTO_DIR.$resu_etu[0]["numEtudiant"].'.jpg', 100);
            //détruit le PNG
            unlink(PHOTOS_DIR.$result_etu[0]["numEtudiant"].'.png');
 
            echo "0";            
        }
        else {
            echo "1";
        }
    }

EDIT MODERATION : VEUILLEZ PRENDRE CONNAISSANCE DE CECI

[Spartacusply]

Manque toujours un truc, allez je vais te donner un indice :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
function testPolitesse($message) {
    if ($message contient 'bonjour' AND $message contient 'merci') {
        return UneRéponseALaQuestion;
    } else {
        return 'Sois poli on verra après';
    }
}

[Celira]

Allez, les gars, soyez gentils avec le nouveau

Bon, pour répondre à la question d'origine : on ne peut pas "simplifier" un code juste comme ça. Le truc évident, c'est que tu as 2 fois le même bloc de code. Tu peux donc le mutualiser dans une fonction.
A part ça, il faut savoir ce que le code doit faire, est-ce qu'il le fait effectivement, pourquoi vouloir le "simplifier", et d'ailleurs que veux-tu dire par "simplifier".

[roleca]

Meric celira

En fait on m'a demandé de creer une fonction a partir de ce code !

[Dendrite]

C'est ta première fonction ? C'est émouvant.
Ou bien c'est un exercice pédagogique ?

Oops, j'enlève ma solution si c'est le second cas !
Une piste :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
function fromPngToJpg($image){
   /*
   Blablabla, toutes tes instructions qui jouent avec le paramètre...
 
 
   */
   //et en dernière ligne, ça retournera true si tout se passe bien...
   return true;
}
//Ce qui te permettra de la convoquer ainsi
if(fromPngToJpg($png)){
   echo 'super, la photo est stockée en jpg sur le serveur.';
}

[Celira]

Un peu de docs sur les fonctions :
PHP - Manual : les fonctions
PHP : Le tutoriel pour grands débutants pressés - V-2. Les fonctions (tiens, tiens, dendrite, ça t'a rappelle pas quelque chose )

[Dendrite]

Elle est nulle la meuf qu'a fait le tuto, elle parle même pas des return true, c'est pourtant bien pratique.

[roleca]

c'est un petit exercice qu on m'a donné

Merci beaucoup c'est comme ca :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
   function fromPngToJpg($image){
        if(count($resu_etu) > 0) 
        {
            $image = base64_decode(substr($_POST['photo'], strpos($_POST['photo'], ",")+1));
            $fp = fopen(PHOTOS_DIR.$result_etu[0]["numEtudiant"].'.png', 'wb');
            fwrite($fp, $image);
            fclose($fp);
 
            $imagepng = imagecreatefrompng(PHOTO_DIR.$result_etu[0]["numEtudiant"].'.png');
            imagejpeg($imagepng, PHOTO_DIR.$resu_etu[0]["numEtudiant"].'.jpg', 100);
            unlink(PHOTOS_DIR.$result_etu[0]["numEtudiant"].'.png');
 
   return true;
}
 
if(fromPngToJpg($png)){
   echo ' photo est stockée en jpg sur le serveur.';
}

[Celira]

La meilleure façon d'apprendre c'est d'essayer, d'échouer, de corriger l'erreur et de réessayer. (Comme vu dans une signature : Si tu ne te plantes jamais, comment veux-tu pousser ?)
Mais bon, là, je peux déjà te dire que tu as une accolade non fermée et qu'il te manque des paramètres. Tu as lu les tutos ?

@Dendrite : t'avais pas une mise à jour dans les tuyaux ?
(Dixit Cel qui se dit régulièrement qu'elle va réécrire le tuto php-mysql et ne le fait jamais...)

[Dendrite]

roleca, tu manipules dans ta fonction des variables que tu ne lui as pas passées en paramètre...
Forcément, ça va coincer.
Un $_POST par définition, c'est pas dans une fonction.
Bref, lis un tuto.

@celira, c'est la prise en main de l'outil qui me rebute, pas le rédactionnel. Si tu peux me filer un lien par mp vers un truc qui explique comment prendre le truc en main...

[Celira]

En fait, $_POST c'est le seul truc qui va marcher dans la fonction, vu que c'est une super-globale (même si c'est pas une bonne idée de l'utiliser comme ça)
A lire : PHP.net - La portée des variables