Discussion :

[spirou]

Bonjour,

J'essaie de m'initier à la poo, mais il y a un truc que je ne comprends pas.

Voici le début d'une class que j'essaie de faire :

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
class formulaire
{
    function __construct($id, $action = true, $method = 'post', $class = false)
    {
        $class = ($class == isset($class)) ? 'class="' . $class . '"' : '';
        $action = ($action === true) ? 'action="' . $_SERVER['REQUEST_URI'] . '"' : 'action="' . $_SERVER['PHP_SELF'] . '"';
        $this -> form = "\n" . '<form id="' . $id . '" method="' . $method . '" ' . $class . '">';
        return $this -> form;
    }
 
    function input_text( $name, $texte, $class = null, $value = true, $maxlength = null )
    {
        $value = ($value === true) ? ' value="' . $_POST[$name] . '"' : '';
        $class = ($class !== null) ? ' class="' . $class . '"' : '';
        $maxlength = ($maxlength !== null) ? ' maxlength="' . $maxlength . '"' : '';
        $this -> text = "\n\t" . '<p><label for="' . $name . '">' . $texte . '</label>';
        $this -> text .= "\n\t" . '<input type="text" name="' . $name . '" id="' . $name . '"' . $class . $value . $maxlength . ' /></p>';
        return $this -> text;
    }
}

Bon, ça reste du bidouillage, mais au moins pour une 1ère tentative, ça marche

Je la déclare donc dans ma page appelante :

Code :

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

1
2
3
$news = new formulaire('news');
echo $news -> __construct('news');
echo $news -> input_text('titre', 'Titre de la news :');

Le truc qui me tourmente, c'est le echo $news -> __construct('news');

Pourquoi suis-je obligé de réutiliser cette fonction, alors qu'elle est déjà déclarée dans $news = new formulaire('news'); ?

Car si je fais ça :

Code :

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

1
2
3
$news = new formulaire('news');
$news .= $news -> input_text('titre', 'Titre de la news :');
echo $news;

Php me signale que la ressource est un objet.

Merci !

[ePoX]

Le truc qui me tourmente, c'est le echo $news -> __construct('news');

Moi aussi sa ma tourmente quand je voit cela

La méthode __construct est une méthode *magique* qui ne doit pas être appelé explicitement. Donc ce que tu as fais la c'est un peu étrange.
(Je sais c'est pas trop expliqué, mais prend le au pied de la lettre)

Car si je fais ça :

Code :
$news = new formulaire('news');
$news .= $news -> input_text('titre', 'Titre de la news :');
echo $news;

La tu touches à la surcharge des opérateurs. Je croit qu'en php on ne peut pas le faire.
Mais bon la tu demandes à PHP de concaténer une chaine de caractères et un objet. Ce qui n'à pas de sens. On concatene des chaines de caractères avec d'autre chaines.

Sinon d'une manière plus générale, décompose ton problème.
Tu souhaites faire une classe qui gère les formulaire, fais le en plusieurs sous parties.
Et puis tant que j'y suis tu n'utilises aucun attribut dans tes classes, je pense la que tu rates quelque chose de primordial.

Voila à quoi devrait plus ressembler ta classe, à mon sens (je suis sur qu'il y 'aura d'autres avis) :

class formulaire
{
public $id;
public $class;
public $name;
public $method;
public $action;

public function __construct()
{
$this->id = NULL;
$this->class = NULL;
$this->name = NULL;
ect...
}

public function __toString()
{
$retour = "";

$retour .= "<form ";
if( $this->id !== NULL)
$retour .= "id=\"".$this->id."\" ";
if( $this->name !== NULL)
$retour .= "id=\"".$this->name."\" ";
ect...

$retour .= "</form>";
return $retour;
}

}

//usage

$form = new formulaire()
$form->id = "toto";

echo $form;

Je n'ai volontairement pas parler du cas des champs textes, car j'aimerais te laisser y refléchir.

bbye

[spirou]

C'est pas gentil de se moquer

Je me doute que mon code est pourrav, mais c'est pas très facile à comprendre, pourtant j'ai un bouquin pour m'aider...

Je ne savais pas tout ce que tu viens de me dire, surtout pour le __conctruct(), __toString(), je ne connaissais pas, je viens de regarder la doc, c'est pas très explicite, mais je pense avoir compris avec ton code.

Bon, je vais essayer pour mes champs textes, je pense qu'il faut procéder de la même façon que tu viens de me fournir, (pas de paramètres pour ma fonction ??)

Merci beaucoup pour tes explications

[ePoX]

et bien pour ce qui est des paramètres dans le constructeur j'en mets assez peu souvent.

En fait je n'en mets que lorsque l'information est primordial à la vie l'objet.
Chose qui m'arrrive assez peu souvent, tellement peu souvent que je n'ai pas d'exemple...

Je me doute que mon code est pourrav, mais c'est pas très facile à comprendre, pourtant j'ai un bouquin pour m'aider...

C'est en forgeant que l'on devient forgeron

Bon, je vais essayer pour mes champs textes, je pense qu'il faut procéder de la même façon que tu viens de me fournir, (pas de paramètres pour ma fonction ??)

Et bien je pense que ces entités, les champs textes, devrait être des classes à part entière.
A toi après d'intégrer avec élégance l'ajout d'un champs à un objet formulaire.

[spirou]

Oui, mais si je fais une classe pour chaque input différent, cela ne sert plus à grand chose d'utiliser des classes, autant utiliser de simples fonctions ?

Il y a une limite en nombre de classes ?

J'ai d'autres petites questions

A quoi servent réellement les méthodes __get et __set ? Je n'arrive pas à bien comprendre à quoi elles servent

peut-on appeler une classe dans une autre ?

Par exemple, ma classe formulaire qui serait appelée dans une autre classe qui affiche la page :

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
class page
{
    public $entete;
    public $navigation;
    public $menu;
    public $contenu;
    public $pied;
    {
        public function __construct
        {
         // .....
        }
        // autres functions
        // .....
        return $display;  
}

Où $contenu contiendrait par exemple un appel à la classe formulaire, qui elle même contiendrait une autre classe réalisant les diverses requêtes.

Il faut utiliser l'héritage avec extends pour faire ça ?

Merci

[spirou]

Personne pour répondre à mes questions ?
Bon tant pis, laisser tomber...

J'en pose une autre à la place

En faisant comme dis ePoX, c'est à dire une classe pour chaque input différents, select, etc..

Comment pourrais-je les intégrer dans ma class formulaire, c'est à dire entre les balises form ?