php orienté objet (oop)

Version imprimable

Bonjour à tous,

J'essaye de me frotter à l'orientée objet en PHP avec un petit programme et je cale depuis ! Le programme en question est un formulaire dont, la vérification est gérée par Jquery et j'aimerais , que , les enregistrements soient faites en base de donnée en PHP Orientée Objet. Vous trouverez en dessous mes différents fichiers . Merci d'un coup main !

index.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
 
 
    <form action ="../class/Validator.php" method ="POST" class ="form">
 
        <div class ="identif">
          <label for ="identif">Identif:</label>
          <span class =""> * </span>
          <input type ="text" name="identif" id="identif"  placeholder="Ex : alouette.mignonne"/>
          <span class = "error-message">error</span>
        </div><!-- endidentif -->
 
        <div class="mail">
            <label for="mail">Votre email :</label>
                 <span class=""> *</span>
                 <input type="email" name="mail" id="mail" placeholder="Ex: mignonne@hotmail.fr"/>
                 <span class = "error-message">error</span>
        </div><!-- endmail -->
 
        <div class="sender">
          <label for="envoi">
              <input type="submit" id="envoi" value="Continuer ..." />
          </label>
        </div>
      </form><!-- endForm -->

app.js

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
 
 
app = {
     /*
    * Fonction appelée au chargement du DOM
    */
    init: function() {
        console.info("app.init");
 
        // J'écoute le clic sur le bouton d'envoi...
        $("input[type=submit]").on("click", this.monSubmit);
 
    },
 
    /*
    * Handler du submit
    */
 
    monSubmit: function(e){
        console.info("app.monSubmit")
 
        // On arrête le chargment de la page
        e.preventDefault()
 
        // Appel de la fonction pour la vérif...
        app.stopChamp()
 
     },
 
     stopChamp: function(){
         console.info("app.stopChamp")
 
        if($("input[type=submit]")){ // Start test
 
          var error = []
          if($("#identif").val() == "" ){
            // On sélectionne le span qui à la class : error-message avec ".next"
            error["identif"] = $("#identif").next(".error-message").fadeIn(1000).text("Veuillez  bien renseigner le champ")
          }
          else{
            if(!$("#identif").val().match(/^[a-z]+$/i)){
              error["identif"] = $("#identif").next(".error-message").fadeIn(1000).text("identif pas valide")
            }
            else
              $("#identif").next(".error-message").fadeOut()
          }
 
 
          if($("#mail").val() == ""){
            error["mail"] = $("#mail").next(".error-message").fadeIn(1000).text("Veuillez  bien renseigner l'email")
          }
          else{
            if(!$("#mail").val().match(/^[a-z0-9._-]+@[a-z0-9._-]{2,}\.[a-z]{2,4}$/)){
              error["mail"] = $("#mail").next(".error-message").fadeIn(1000).text("Votre email n'est pas valide")
            }
            else
              $("#mail").next(".error-message").fadeOut()
          }
 
          if(!error["mail"] && !error["identif"]){
            var $this = $("input[type=submit]")
                $this.attr("disabled", "disabled") 
                    setTimeout(function(){
                      $this.attr("value", '<img src="../img/ajax-loader.png" alt="loading"/> Chargement ...')
                              .css({
                        opacity : 0.4,
                        cursor  : "not-allowed"
                      })
                    },300)
 
                    // Appel de la fonction qui fera la requête Ajax
                    app.reqAjax()
 
                    return false
          }
 
        }// End test
 
     },
     reqAjax : function(){
       console.info("app.reqAjax")
 
       // On récupère l'url et le type
		   //var url  = this.action
				 var url  = "class/Validator.php"
		   //var type = this.method
 
      // On récupère les infos (identif et mail) 
      // var data = $(this).serialize()
		  var data = $("form").serialize()
 
      // On fait la requête
      $.ajax({
        url: url,
        type: "POST",
        data: data,
        success: app.showSuccess  
      })
 
     },
 
     showSuccess: function(){ // se référer à http://dev.howtommy.net/?d=2011/09/16/04/00/00-comment-faire-apparaitre-un-beau-popup-html-sur-son-site-avec-jquery
       console.info("app.showSuccess")
 
       // On crée la div qui servira de griser le fond....
       var popup = $("<div>",{
         class: "test",
         css:{
         background: "rgba(0,0,0,.8)",
				 width: "100%",
				 height: "100%",
         position: "fixed",
				 top: 0,
				 left: 0,
         "z-index": 1
         }
       })
 
       // On crée le POPUP
       $(".recupPopup").css({
          "border-radius": "15px / 15px",
          position: "fixed",
          display: "none",
          left: "285px",
          top: "140px",
          "z-index": 2,
          padding: "10px",
          width: "50%",
          "background-color": "#EEEEEE",
          "font-size": "12px",
          "line-height": "16px",
          color:" #202020",
          border : "3px outset #555555"
        })
 
      // On ajoute la DIV (grisée) au body
      $("body").append(popup)
 
      // On l'a fait apparaitre après quelques sécondes...
      popup.fadeTo(600, 0.5, function () {
        // On fait apparaitre le POPUP ,après quelques sécondes...
        $(".recupPopup").fadeIn(3000)
      })
     }
 
}
 
/*
 * Chargement du DOM
 */
$(function(){
    app.init();
})

class/Validator.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
 
<?php
 
class Validator{
 
    private $data;
    private $identif;
    private $mail;
 
 
    public function __construct($data,$identif,$mail){
       $this->data     =  $data;
       $this->identif  =  $identif;
       $this->mail     =  $mail;
    }
 
 
    public function isRegister($identif, $mail){
    	 $req = App::getDataBase()->query("INSERT INTO starpa SET identif = :identif, mail = :mail");
         $req->execute();
    }
 
} // End class

class/DataBase.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
33
34
35
36
 
<?php
 
class DataBase{
 
  private $pdo;
  private $req = null;
 
  /*
  * Par defaut host = localhost pour n'est pas obligé de la préciser si,
  * la bdd est sur le même serveur  
  */
  public function __construct($login, $password, $databaseName, $host = "localhost"){
  	try {
      $this->pdo = new PDO("mysql:dbname=$databaseName;host=$host", $login, $password);
      $this->pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
      $this->pdo->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_OBJ);
    }
 
    catch(PDOException $e){
    	exit('MySQL Connect Error >> '.$e->getMessage());
    }
  }
 
  public function query($query, $params = false){
    if($params){
      $this->req = $this->pdo->prepare($query);
      $this->req->execute($params);
    }
//    else{
//        $this->req = $this->pdo->query($query);
//    }
    return $this->req;
  }
 
} // End class ...

class/App.php

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
<?php
 
class App{
    /*
    A chaque fois que je ferai appel à getDataBase, il initialisera une nouvelle, chose pas bien !
    Alors,pour conserver une connection tout le long de la connection, il faut :
    Créer une variable static : db et null au départ et mettre une condition(if)
    */
    static $db = null;
 
    static function getDataBase(){
      if(!self::$db){
        self::$db = new DataBase("root", "", "fullz");
      }
      return self::$db;
    }
}// End class

Certains me diront que je n'ai pas besoin de 3 class pour faire un enregistrement en base de donnée avec ce petit programme mais j'aimerais bien y arrivé et si possible, comment faire avec : spl_autoload_register ?
Au fait, je ne veux pas de retour de la base de donnée juste un enregistrement des données fournir par l'utilisateur ? Les vérifications sont gérées par Jquery.

Si y'a des questions et que mes dires sont mal expliqués, n'hésitez à me poser des questions !

Merci

24/09/2016, 08h18
badaze

Une chose.
Ce n'est pas parce que tu contrôles la validité des données avec jquery qu'il ne faut pas le faire avec php.
En effet d'une part on peut désactiver le javascript et d'autre part on peut prendre le code source de ta page, enlever les contrôles et envoyer des données qui seront traitées sans contrôle donc.

php orientée objet (oop)

Bonjour badaze,

Et merci de ton retour ! Alors il faut que je fasse obligatoirement le contrôle côté PHP et ça va marcher ? Parce que mes erreurs sont :

Code:

1
2
3
4
5
6
7
8
9
 
( ! ) Fatal error: in C:\wamp64\www\f-fullz\class\DataBase.php on line 30
( ! ) PDOException: in C:\wamp64\www\f-fullz\class\DataBase.php on line 30
Call Stack
#	Time	Memory	Function	Location
1	0.0008	245048	{main}( )	...\index.php:0
2	0.0029	267848	Validator->isRegister( )	...\index.php:8
3	0.0052	276296	DataBase->query( )	...\Validator.php:18
4	0.0052	276392	query ( )	...\DataBase.php:30

Et quand je mets la ligne en question en commentaire, j'ai ça :

Code:

1
2
3
4
5
6
 
( ! ) Fatal error: Call to a member function execute() on null in C:\wamp64\www\f-fullz\class\Validator.php on line 19
Call Stack
#	Time	Memory	Function	Location
1	0.0013	245048	{main}( )	...\index.php:0
2	0.0056	267096	Validator->isRegister( )	...\index.php:8

Merci !

08/10/2016, 23h48
mathieu

dans la méthode Validator->isRegister vous avez 2 paramètres que vous n'utilisez pas dans la requête MySQL

php orientée objet (oop)

Citation:

Envoyé par mathieu

dans la méthode Validator->isRegister vous avez 2 paramètres que vous n'utilisez pas dans la requête MySQL

Ma nouvelle class : Validator

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
<?php
 
class Validator{
 
    private $data;
 
    public function __construct($data){
       $this->data     =  $data;
    }
 
 
    public function isRegister($identif, $mail){
    	$req = App::getDataBase()->query("INSERT INTO starpa SET identif= ?,mail= ?");
        $req->execute([$this->data[$identif], $this->data[$mail]]);
    }
 
} // End class

Mais j'ai l'erreur suivante :

Code:

1
2
3
 
( ! ) Fatal error: in C:\wamp64\www\f-fullz\class\DataBase.php on line 30
( ! ) PDOException: in C:\wamp64\www\f-fullz\class\DataBase.php on line 30

20/10/2016, 11h47
Celira
ça pourrait être une bonne idée de capturer et d'exploiter les exceptions PDO avec un bloc try/catch autour de l'appel. Quelque chose du genre :
Code:

1 2 3 4 5 try { $this->req = $this->pdo->query($query); } catch (PDOException $e) { echo 'Erreur sur la requête '.$query.' : '.$e->getMessage(); }

Code:

1 2 3 4 public function isRegister($identif, $mail){ $req = App::getDataBase()->query("INSERT INTO starpa SET identif= ?,mail= ?"); $req->execute([$this->data[$identif], $this->data[$mail]]); }
Ta requete SQL est incorrecte. La syntaxe de INSERT INTO c'est : "INSERT INTO table (col1, col2,...) VALUES (val1, val2, ...)". Le SET c'est pour les requetes de type UPDATE.

Au passage tu n'as pas l'air d'avoir compris ce que t'a dis badaze.

Les contrôles javascript ne sont qu'une aide à la saisie qui permettent, dans une utilisation normale, d'informer plus rapidement le visiteur sur des erreurs de saisies. Mais cela ne valide rien du tout puisque les contrôles javascript - comme tout le code qui se trouve côté client - peuvent être modifiés/piratés. La validation réelle doit se faire impérativement côté serveur en php, c'est à dire que dans ton exemple tu dois refaire les contrôles sur le mail et sur le champ "identif" en php. C'est pas obligatoire mais c'est le seul contrôle qui soit réellement efficace. Si tu ne le fais pas, n'importe qui d'un peu formé au web (c'est à dire la grande majorité des membres de ce forum, sans même parler de vrais pirates) pourra rentrer n'importe quoi dans les champs de ta base de donnée.

Pour ton exemple tu pourrais faire une classe php validator avec deux fonctions, une pour le mail et une pour contrôler le champ "identif". A distinguer de l'enregistrement en bdd qui sera fait par la suite si les contrôles se passent bien. Pour dire qu'il vaut mieux bien différencier les parties de ton code et nommer précisément chaque étape par un nom approprié non équivoque. Si tu utilises le mot "validator" pour valider tout le processus de contrôle et d'enregistrement du formulaire, tu te prive d'utiliser ce mot pour désigner son sens premier dans ce contexte à savoir la validation des champs du formulaire. Tu t'apercevras avec la pratique que ce genre d'approximation est un handicap majeur pour la compréhension et l'organisation du code. Et d'ailleurs cela t'a amené à nommer "validator" un simple enregistrement en bdd qui ne contrôle rien du tout.

Un exemple plus classique de script 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 33 34 class validator { static public function mail($mail) { return filter_var($mail, FILTER_VALIDATE_EMAIL) !== false; } static public function identif($identif) { // return ton regex } } // Récupère les post de la requête ajax $mail = filter_input(INPUT_POST, 'mail'); $identif = filter_input(INPUT_POST, 'identif '); $erreur = []; // Validation des champs if(!validator::mail($mail)) $erreur[] = 'erreur mail'; if(!validator::mail($identif )) $erreur[] = 'erreur identifiant'; // Enregistrement en bdd if(count($erreur) == 0) { try { //requête enregistrement; } catch (PDOException $e) { $erreur[] = 'Erreur sur la requête '.$query.' : '.$e->getMessage(); } } // retour ajax echo count($erreur) > 0 ? json_encode(array('result', $erreur)) : json_encode(array('result', 'ok'));