Discussion :

[nicolas66]

Bonjour,

Je suis actuellement en train de développer une série de classes pour faciliter la manipulation d'images en PHP 5 et je suis confronté à un petit problème de conception.

Je possède une classe Image abstraite avec plusieurs classes qui en dérivent (ImageGIF, ImageJPEG, etc). Chaque classe dérivée permet donc de charger et de sauvegarder une image dans le même format. Le problème c'est que je me suis aperçu que l'on pouvait créer une image jpeg et la sauvegarder dans un autre format. Les possibilités sont donc étendues alors que l'architecture de mes classes est limitée.

Une des solutions serait de créer une fonction de sauvegarde dans la classe Image et de faire des 'Si .. Alors .. Sinon' pour tester l'extension du nom de fichier passé en paramètre et réaliser le traitement souhaité. Cette solution n'est pas très élégante et n'exploite pas la puissance de la P.O.O. Auriez-vous d'autres idées ? Merci d'avance


Nico.

[ska_root]

Salut,

peux-tu nous poster ton code : les classes en question et le procédural instanciant ces objets, afin qu'on puisse y voir plus clair ?

merci

[jeff_!]

sinon peut etre un objet transversal qui se charge du transcriptage

[nicolas66]

Voici le code des différentes classes :

La classe image

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
41
42
43
44
45
46
47
 
<?php
abstract class Image
{
	// Variables members
	protected $m_Filename;
	protected $m_ImageHandler;
 
 
	// Methods
	public function __construct()
	{
		$this->m_Filename     = "";
		$this->m_ImageHandler = "";
	}
 
	public abstract function Create( $Filename );
 
	public function GetWidth()
	{
		$infos = @getimagesize($this->m_Filename);
 
		if( $infos )
			return $infos[0];
 
		return -1;
	}
 
	public function GetHeight()
	{
		$infos = @getimagesize($this->m_Filename);
 
		if( $infos )
			return $infos[1];
 
		return -1;
	}
 
	public abstract function GetType();
	public abstract function Show();
 
	public function __destruct()
	{
		@imagedestroy($this->m_ImageHandler);
	}
}
?>

La classe ImageJPEG

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
 
<?php
require_once('image.class.php5');
 
class ImageJPEG extends Image
{
	// Methods
	public function Create( $Filename )
	{
		$this->m_ImageHandler = @imagecreatefromjpeg($Filename);
 
		if( $this->m_ImageHandler )
		{
			$this->m_Filename = $Filename;
			return true;
		}
 
		return false;
	}
 
	public function GetType()
	{
		return "JPEG";
	}
 
	public function Show()
	{
		if( $this->m_ImageHandler )
		{
			header("Content-type: image/jpeg");
			@imagejpeg($this->m_ImageHandler);
		}
	}
}
?>

Le fichier de test

Code :

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

1
2
3
4
5
6
7
8
 
<?php
require_once('image_jpeg.class.php5');
 
$image = new ImageJPEG();
$image->Create('pics/bsd_daemon.jpg');
$image->Show();
?>

[nicolas66]

Sinon tu determines l'extension et avec un attern factory tu instance la bonne classe. Cette classe doit implementer une methode sauvegarder (definie dans une interface) que tu appelle sur la classe ressortant de la factory.

c clair ?

Je ne vois pas trop comment appliquer le design pattern factory à cet exemple. Pourrais-tu me dire comment tu t'y prendrai ? Merci

[jeff_!]

par exemple dans ta class abstraite image tu definie la methode

Code :

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

abstract public function transcrypte( $type , $ressource )

cette methode appelle la bonne class et cree(grace a la ressource) l'objet voulue et le retourne

je crois que c'est l'idee maintenant ca fait longtemps que je n'ai pas joué avec les images

Code :

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

1
2
3
$jpg = new JPG('monimage.jpg)
$gif = $jpg->transcrypte('gif', $jpg);
$gif->save('monimage.gif');

[stunti]

en fonction du nom de l'image avec un petit preg_match tu recupere l'extension.

ensuite dans une methode statique :

C un peu a la limite du pseudo code.

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
 
<?
abstract class ImageFactory {
     static function getInstance($filename) {
		if(preg_match('#^(.*)\.(.*)?]$#',$filename, $match)) {
			$classname = {'Image'.$match[2]};
			return new $classname;
		}
		else {
			throw new Exception('Type de fichier inconu');
		}
     }
}
 
 
$class_origine = ImageFactory::getInstance('toto.jpg');
$class_destination = new ImageGif();
 
$class_destination->save($class_origine->getImage());
 
 
?>

J'ai pas teste, fait ca de tete.
C surement a adapter en fonction des besoins, mais ca devrait servir de base.

[stunti]

Sinon tu determines l'extension et avec un attern factory tu instance la bonne classe. Cette classe doit implementer une methode sauvegarder (definie dans une interface) que tu appelle sur la classe ressortant de la factory.

c clair ?