suppression

Version imprimable

Bonjour aux forum,

En rapport direct avec le sujet de Julien pauli: http://g-rossolini.developpez.com/tu...ge=modele#LV-I

j'ouvre ce post pour traiter d'un point récurrent lors du développement d'un back office. Il est construit très simplement :

|-article
|--- ajout
|--- modification
|--- suppresion
|-utilisateur
|--- ajout
|--- modification
|--- suppresion
etc etc

Prenons l'exemple d'un article :
http://img413.imageshack.us/img413/1776/backoff.jpg

Toutes ces fonctions seront 'quasiment' les mêmes pour chacune des pages du back off, dés lors je souhaite utiliser une même fonction dans mon controller, se différencie par l'appel du lien.

Prenons l'exemple de la fonction suppression :

AdminController.php
Code:

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 .. public function deleteAction() { $this->view->title = "Supprimer un article"; $id = $this->_request->getParam('id',0); $mod = $this->_request->getParam('mod'); $this->view->mod = $mod; if ($id > 0) { switch($mod) { case 'accueil': $accueil = new Accueil(); if($accueil){ $where = 'idAccueil = ' . $id; $accueil->delete($where); $this->view->message = "L'article a été correctement supprimé"; $this->_redirect('/admin/index'); }else{ $this->view->message = "Erreur de suppression"; } break; case 'utilisateur': break; case 'article': break; } } }
Que j'appel ainsi :

Code:

echo $this->url(array("controller"=>"admin","action"=>"delete","mod"=>"accueil","id"=>$cle->idAccueil));

Mais, avec cette facon de coder, je vais me retrouver avec des centaines et centaines (redondantes) de lignes dans mon controller.
- Alors, dois je créer une aide d'action ?
- De plus, lorsque je click sur mon lien suppression, je fais appel à une fonction de AdminController, que je suis obligé de déclarer en deleteAction, donc je créé une vue action.phtml, qui ne contient rien. Moi je souhaiterai uniquement une fonction delete() ?

Bonjour,

Il me semble que ton action 'delete' fait beaucoup de choses qui auraient plus leur place dans le modèle...

Pourquoi ne pas créer par exemple dans ta classe 'Accueil' une méthode 'delete($id)' (et faire de même pour tes autres tables) ?
Cette méthode pourrait réaliser la suppression et lancer une exception en cas de problème...

Ton action pourrait alors être réduite à quelques lignes (non testées) :
Code:

1 2 3 4 5 6 7 $model = ucfirst($mod); // il serait largement préférable de valider puis filtrer le paramètre, mais c'est une autre histoire... ;) try { $table = new $model(); $table->delete($id); } catch... // traitements différents en fonction du type d'exception rencontrée

Bonjour,

Merci pour la réponse.

J'ai suivi vos conseils, voici comment pourrait se décomposer ma nouvelle fonction :
AdminController.php

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
...
    public function deleteAction()
    {
        $this->view->title = "Supprimer";
		$id = $this->_request->getParam('id',0);
		$mod = $this->_request->getParam('mod');
   	 	//a valider & filtrer 
   	 	$module = ucfirst($mod); 
		try
		{
		    $table = new $module();
		    $table->deleteOne($id);
		    $this->view->message = "L'article a été correctement supprimé"; 
		    $this->_redirect('/admin/'. strtolower($module) );
		}
		catch(Exception $e){
			$this->view->message = "Erreur de suppression";
		}
    }

Accueil.php

Code:

1
2
3
4
5
 
...
  public function deleteOne($id){
  	return $this->delete('idAccueil = ' . $id);
  }

Bonjour aux forums,

Sur le même modèle que le code de Eusebe, présenté ci dessus, j'ai développé une fonction 'update' :

AdminController.php

Code:

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
 
...
public function editAction()
	{
        $this->view->title = "Editer un article";
		$id = $this->_request->getParam('id',0);
		$mod = $this->_request->getParam('mod');
		$module = ucfirst($mod);
		$formulaire = ucfirst($mod) . "Form";
		try
		{
			$form = new $formulaire();
			$table = new $module();	
			//modification en bd
			if ($this->_request->isPost()) {
				$formData = $this->_request->getPost();
			    if ($form->isValid($formData)) {
			    	$table->updataOne($id, $formData);
			    	$this->view->message = "UPDATE"; 
		    		$this->_redirect('/admin/'. strtolower($module) );
			    }
			}else{
				//insertion des donnees dans le formulaire
				$acc = $table->getAccueil($id);
				$form->populate($acc->toArray());
			}  
		}
		catch(Exception $e){
			$this->view->message = "Erreur";
		}
		$this->view->form = $form;
    }

Accueil.php

Code:

1
2
3
4
5
6
7
8
9
10
11
 
...
  public function getAccueil($id){
	return $this->find($id)->current();
  }
  public function updateOne($id, $data){
  	$acc = $this->find($id)->current();
  	$acc->titre = $data[0];
  	$acc->resume = $data[1];
  	return $acc->save();
  }

Mais, il me reste petit soucis

Le 1er, ma fonction : updataOne($id, $formData); recoit le tableau, hors je lui passe le tableau du $form qui à la forme suivante :

Code:

1
2
3
4
5
6
Array
(
    [titre] => test
    [resume] => test
    [envoyer] => envoyer
)

ce que je ne trouve pas très propre, car il y a le parametre envoyer dans le tableau, meme si je ne l'utiliserai pas

Mon deuxieme soucis se situe du côté du model. Ma fonction :

Code:

1
2
3
4
5
6
7
 
  public function updateOne($id, $data){
  	$acc = $this->find($id)->current();
  	$acc->titre = $data[0];
  	$acc->resume = $data[1];
  	return $acc->save();
  }

n'est pas automatisé

Merci pour tous conseils