Discussion :

[miya]

Bonjour,

J'ai créé sans aller très loin dans le développement, un formulaire qui permet de saisir un commentaire et l'affiche sur la page.

Suite au développement, plusieurs petit soucis 'minime' apparaissent.

1/ un decallage entre le label et le input text en sortie html
2/ en rafraichissant ma page, l'enregistrement se fait à nouveau.

De plus, j'aurai aimé savoir si je respecte bien l'ensemble du MVC et que le tout est codé dans les règles.

Un screenshot pour représenter le tout :



Le 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
 
class CommentaireForm extends Zend_Form{
	public function __construct($options=null){
		parent::__construct($options);
		$this->setName('ajouterCommentaire');
		$nom = new Zend_Form_Element_Text('Nom');
		$nom->setLabel('nom')
				->setRequired(true);		
		$email=new Zend_Form_Element_Text('Email');
		$email->setLabel('email')	
					->setRequired(True);
		$contenu= new Zend_Form_Element_Textarea('Contenu');
		$contenu->setLabel('commentaire')
					->setRequired(True);
		$submit = new Zend_Form_Element_Submit('envoyer');
		$submit->setLabel('envoyer');
		$this->addElements(array($nom,$email,$contenu,$submit));
	}
}

indexController.php

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
 
	function addcomAction(){
		$this->view->title="ajouter com";
		$formCom = new CommentaireForm();
		if($this->_request->isPost()){
			$formData = $this->_request->getPost();
			$formCom->populate($formData);
			$formData=$formCom->getValues();
			if($formCom->isValid($formData)){
				$com = new Commentaire();
				$commentaire = $com->createRow($formData);
				if($commentaire){
					$id=$commentaire->save();
				}
			}
		}
		$this->view->formCom=$formCom;
	}

addcom.phtml

Code :

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

1
2
3
4
5
 
<?php
		echo $this->action('affichercom','index');
		echo $this->formCom;
?>

Commentaire.php

Code :

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

1
2
3
4
5
6
7
8
9
10
 
class Commentaire extends Zend_Db_Table_Abstract {  
 
  protected $_name = 'commentaire';
  protected $_primary = 'Id';
 
  public function getCommentaireAll(){
		return $this->fetchAll();
  }
}

[vg33]

1) Si tu ne veux pas de réenregistrement avec actualiser, tu dois faire une redirection après enregistrement des données, même si c'est vers la même page.
2) J'imagine que c'est un test, mais tu dois impérativement tester tes données dans le form en utilisant des filters et des valideurs. Exemple : si tu rentres des espaces dans l'un des champs requis, il doit être considéré comme vide, ce qui n'est pas le cas dans ton exemple.
3) $formCom->populate($formData); n'est pas utile à cet endroit. Il me parait plus clair de l'insérer en cas d'échec de validation.
4) Sinon, le MVC me semble bien respecté dans ce code.

[miya]

1) Si tu ne veux pas de réenregistrement avec actualiser, tu dois faire une redirection après enregistrement des données, même si c'est vers la même page.
2) J'imagine que c'est un test, mais tu dois impérativement tester tes données dans le form en utilisant des filters et des valideurs. Exemple : si tu rentres des espaces dans l'un des champs requis, il doit être considéré comme vide, ce qui n'est pas le cas dans ton exemple.
3) $formCom->populate($formData); n'est pas utile à cet endroit. Il me parait plus clair de l'insérer en cas d'échec de validation.
4) Sinon, le MVC me semble bien respecté dans ce code.

Merci, en plus j'ai compris toutes tes remarques

3/ tu abordes une remarque sur laquel je voulais poser une question. J'ai cherché, mais j'ai moyennement compris. A quoi sert exactement cette ligne? :

Code :

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

1
2
 
$formCom->populate($formData);

Je me dis qu'elle sélectionne les données envoyé en post et que cela l'insère dans le modèle de formulaire créer?

[vg33]

populate() complète en bloc toutes les values de l'objet Zend_Form, typiquement avec les données POST ou GET. C'est utile pour réafficher un formulaire avec des erreurs.
Une précision : à ma connaissance, populate() ne complète pas les values de champs indiqués en disabled. Il faut faire un setValue() pour chacun de ces champs.

[miya]

Merci vg33

[Eusebe]

Et pour info, il me semble que populate est automatiquement appelé par isValid. Il est donc inutile de l'appeler explicitement si on a précédemment vérifié la validité du formulaire