Discussion :

[wattaroo]

Bonjour,
lors d'un exercice, j'ai crée une classe avec un constructeur et sa méthode. Lors de la création de l'objet, le fonction renvoi bien le texte avec les paramètres.
Mais quand j'ai voulu ajouter une autre classe pour introduire un formulaire, ça ne me donne plus le résultat, ça me vire de la page pour revenir à la racine du serveur. Le formulaire s'affiche mais le traitement ne s'effectue pas.

J'ai du me gourer dans l'appel de la méthode getformu(), mais je ne vois pas comment la corriger.
Est-ce parce que je l'appelle comme-ci elle avait un constructeur ??

Merci pour votre aide.

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
<?php
	$nom = $_POST['nom'];
	$departement = $_POST['departement'];
 
	class formu{
		public function getformu(){
			echo '<form method=\'POST\' action="<? $_SERVER["PHP_SELF"]?><br/>';
			echo '<fieldset><br/>';
			echo '<legend>Associer le département à une ville</legend><br/>';
			echo '<label>Taper le nom de la ville</label><input type=\'text\' name=\'nom\'/><br/>';
			echo '<label>Taper le nom du département<label><input type=\'text\' name=\'departement\'/><br/>';
			echo '<input type=\'submit\' name=\'envoi\' value=\'Envoi\'/><br/>';
			echo '</fieldset><br/>';
			echo '</form>';
 
		}
	}
	class maville{
		public $nom;
		public $departement;
		public function __construct($nom,$departement){
			$this->nom = $nom;
			$this->departement = $departement;
		}
		public function getinfo(){
			$texte = 'La ville de '.$this->nom.' est dans le département : '.$this->departement.'<br/>';
			return $texte;
		}
	}
	$formu1 = new formu();
	echo $formu1->getformu();
	$ville1 = new maville($nom,$departement);
	echo $ville1->getinfo();
?>

[Niki59]

Bonjour wattaroo,

J'ai du me gourer dans l'appel de la méthode getformu(), mais je ne vois pas comment la corriger.

Dans ta méthode getformu de ta classe formu tu fais un echo de ton formulaire :

Code :

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

1
2
3
4
		public function getformu(){
			echo '<form method=\'POST\' action="<? $_SERVER["PHP_SELF"]?><br/>';
			echo '<fieldset><br/>';
// etc....

or lorsque tu appel cette méthode depuis ton objet tu fais aussi un echo :

Code :

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

echo $formu1->getformu();

Je pense que c'est le "double echo" qui fait foirer ton programme.
Essaye de remplacer ta méthode par ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public function getformu(){
 
    $form = '<form method=\'POST\' action="<? $_SERVER["PHP_SELF"]?><br/>';
    $form .= '<fieldset><br/>';
    $form .=  '<legend>Associer le département à une ville</legend><br/>';
    $form .=  '<label>Taper le nom de la ville</label><input type=\'text\' name=\'nom\'/><br/>';
    $form .=  '<label>Taper le nom du département<label><input type=\'text\' name=\'departement\'/><br/>';
    $form .=  '<input type=\'submit\' name=\'envoi\' value=\'Envoi\'/><br/>';
    $form .=  '</fieldset><br/>';
    $form .=  '</form>';
 
    return $form;
 
}

Bonne journée

[wattaroo]

Merci Niki59 pour ton aide.

Effectivement, le double "echo" ne doit pas faciliter les choses.

Mais j'ai corrigé mon code avec tes recommandations et je n'obtiens pas un meilleur résultat. Ca plante toujours et me renvoi sur la racine du serveur.

Donc, il doit s'agir d'autre chose.

Si quelqu'un à une autre idée. Merci.

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
<?php
	class formu{
		public function getformu(){
			$form = '<form method=\'POST\' action="<? $_SERVER["PHP_SELF"]?>';
			$form .='<fieldset>';
			$form .='<legend>Associer le départment à une ville</legend>';
			$form .='<label>Taper le nom de la ville</label><input type=\'text\' name=\'nom\'/><br/>';
			$form .='<label>Taper le nom du département<label><input type=\'text\' name=\'departement\'/><br/>';
			$form .='<input type=\'submit\' name=\'envoi\' value=\'Envoi\'/>';
			$form .='</fieldset>';
			$form .='</form>';
			return $form;
		}
	}
 
	$nom = $_POST['nom'];
	$departement = $_POST['departement'];
 
	class maville{
		public $nom;
		public $departement;
		public function __construct($nom,$departement){
			$this->nom = $nom;
			$this->departement = $departement;
		}
		public function getinfo(){
			$texte = 'La ville de '.$this->nom.' est dans le département : '.$this->departement.'<br/>';
			return $texte;
		}
	}
	$form = new formu();
	echo $form->getformu();
 
	$ville1 = new maville($nom,$departement);
	echo $ville1->getinfo();
?>

[RunCodePhp]

Salut

Il y a un petit truc à retenir, et à exploiter, c'est de vérifier le HTML généré (par Php) au niveau de ton navigateur : click droit -> code source de la page
Ensuite, analyser le code pour repérer les éventuels erreurs.

Le problème du HTML (et Css, Javascript, ... tout ce qui est coté client) c'est que s'il y a une erreur, il y a rien qui nous l'averti.
A moins d'utiliser des outils comme installer un plugin Firebug dans FireFox.

Code :

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

$form = '<form method=\'POST\' action="<? $_SERVER["PHP_SELF"]?>';

Ce code Php va générer ce code HTML :
<form method='POST' action="<? $_SERVER["PHP_SELF"]?>

Il faudrait faire plutôt comme ceci :

Code :

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

1
2
 
$form = '<form method="POST" action="'.$_SERVER["PHP_SELF"].'">';

Cette ligne c'est en faite déjà du Php, il n'est plus utile de remettre des <?php ?>, ça sera interprété comme une chaine.
Il faut concaténer.


Il y a un <label> non fermé aussi :

Code :

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

<label>Taper le nom du département<label> <- pas fermé

On le tape avec quoi le département, un marteau, une masse ?
Aussi, on ne peu vraiment pas taper un prénom de département ?

Ne faudrait il pas dire : Veuillez saisir un département ?

[Herode]

Je ne sais pas quel est le contexte de l'appel mais à la lecture du code, je vois quelques soucis potentiels :

Code :

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

$machin = $_POST['toto']

ne peut donner de résultats prévisibles que si $_POST['toto'] a été initialisé. Sinon, quand tu utilises $machin par la suite, tu auras des soucis.

Par exemple :

Code :

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

1
2
3
4
5
$nom = $_POST['nom'];
$departement = $_POST['departement'];
...
$ville1 = new maville($nom,$departement);
echo $ville1->getinfo();

Si $nom et $departement ne sont pas définis, $ville1->getInfo() devrait planter. Tu peux tracer en mettant un die(__LINE__) en divers endroits de ton code. Tant que tu vois ton "die", c'est que ça plante plus loin et tu peux le déplacer pour trouver la ligne fautive.

[wattaroo]

Merci à vous deux d'avoir pris le temps de me répondre.

RunCodePhp, j'ai effectué ta modif sur "l'action" et effectivement les balises php faisaient double emploi. Ca marche sans problème depuis. Merci pour tes conseils.

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
<?php
	class formu{
		public function getformu(){
		$form = '<form method="POST" action="'.$_SERVER["PHP_SELF"].'">';
		$form .= '<fieldset>';
		$form .= '<legend>Associer le départment à une ville</legend>';
		$form .= '<label>Saisir le nom de la ville</label><input type=\'text\' name=\'nom\'/><br/>';
		$form .= '<label>Saisir le nom du département</label><input type=\'text\' name=\'departement\'/><br/>';
		$form .= '<input type=\'submit\' name=\'envoi\' value=\'Envoi\'/>';
		$form .= '</fieldset>';
		$form .= '</form>';
		return $form;
		}
	}
 
	$nom = $_POST['nom'];
	$departement = $_POST['departement'];
 
	class maville{
		public $nom;
		public $departement;
		public function __construct($nom,$departement){
			$this->nom = $nom;
			$this->departement = $departement;
		}
		public function getinfo(){
			$texte = 'La ville de '.$this->nom.' est dans le département : '.$this->departement.'<br/>';
			return $texte;
		}
	}
	$formu1 = new formu();
	echo $formu1->getformu();
	$ville1 = new maville($nom,$departement);
	echo $ville1->getinfo();
 
?>

Herode, je comprends bien ta remarque et suis d'accord avec toi, mais en l'occurrence, $nom et $departement sont définis, car le code récupère bien les saisies du formulaire.
Je vais quand même suivre tes conseils et voir comment m'améliorer avec le "die(__LINE__)"

Merci à vous.