Discussion :

[laurentSc]

Bonjour,

la notion de clé primaire existe en SQL mais rien d'équivalent en PHP/POO. Néanmoins, il faut pouvoir, lorsqu'on instancie un nouvel objet de la classe, savoir s'il faut faire un INSERT ou un UPDATE de la ou des table(s) SQL correspondante(s). Il est donc nécessaire de savoir si les propriétés de ce nouvel objet sont déjà présentes en bdd. Je me demande donc s'il faut prévoir une propriété "identifiant" qui permettrait d'identifier l'objet et donc de vérifier si son "alter ego" SQL est déjà en bdd ou non. Quel est votre avis ?

[grunk]

Les problématique SQL n'ont pas être géré en PHP.

Savoir si tu dois insérer ou mettre à jour c'est un problème SQL qui se résout via SQL en faisant une requête du type INSERT ON DUPLICATE KEY.
Tant que tu as tous les champs nécessaire dans ton objet c'est le moteur de BDD qui décide tout seul en fonction des contraintes que tu as appliqué

[laurentSc]

OK,
c'est sûrement possible mais j'ai pas trouvé : je voudrais pouvoir compter le nombre de insert et le nombre de update. Comment obtenir cette information avec un insert on duplicate key ?

[grunk]

Pas possible à ma connaissance.

Si tu veux absolument le gérer manuellement, il suffit de regarder si ton objet à le champs de clé primaire rempli (en général l'id). Si il est rempli c'est que l'objet vient de mysql et qu'il faut donc faire un update , sinon c'est qu'il à été crée "à la main" et qu'il faut l'insérer.

[laurentSc]

J'ai du mal à comprendre. Cela dit, dans une autre discussion (https://www.developpez.net/forums/d2.../#post11749159), j'avais donné un exemple qu'il faudrait utiliser pour préciser. J'ai mis à jour cette discussion-là, et je clos celle-ci.

[grunk]

En reprenant plus ou moins ton exemple , admettons que tu es la classe suivante :

Code :

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

1
2
3
4
5
class Company
{
	private ?int $id = null;
	private string $label = '';
}

Id représentant l'id unique de ton élément dans ta bdd (en général un auto incrément).

Si tu veux créer une Company tu vas seulement remplir le label car tu n'es pas capable via le code de déterminer quel est l'id à utiliser.

Code :

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

$c = new Company('my super new company');

Donc si l'id est à null => c'est un insert que tu devras faire et la bdd se chargera de définir l'id via son autoincrément.

Au contraire si l'id est différent de null ca veux dire que tu as peuplé ton objet avec des infos qui viennent de la bdd et qu'il faudra donc faire un update.

[laurentSc]

Merci pour ta réponse mais malheureusement, j'ai encore du mal :

Id représentant l'id unique de ton élément dans ta bdd (en général un auto incrément).

Selon moi, l'auto increment n'existe que en SQL, et là, on est dans une classe PHP...

Au contraire si l'id est différent de null

Comment l'id peut-il être non nul s'il n'est pas géré par le constructeur de la classe ?

ca veux dire que tu as peuplé ton objet avec des infos qui viennent de la bdd

Là, je suis largué car pour moi, on va peupler l'objet avec la chaîne de caractères 'my super new company', donc ça ne vient pas de la bdd

[grunk]

En partant d'une table comme celle ci :

Code sql :

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

1
2
3
4
5
CREATE TABLE company(
   id INT UNSIGNED AUTO_INCREMENT,
   label VARCHAR(30) NOT NULL,
   PRIMARY KEY(id)
);

On créer la classe associée :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
class Company
{
	private ?int $id = null;
	private string $label = '';
 
	public function save() {
		if($id != null) {
			//UPDATE company SET ....
		} else {
			// INSERT INTO
		}
	}
}

Jusque là on est d'accord ?
Note que pour bien faire la fonction save ne devrait pas appartenir à l'objet company mais à un autre objet qui gère les interraction avec la bdd , mais pour la simplification on acceptera l'exemple ainsi.

Selon moi, l'auto increment n'existe que en SQL, et là, on est dans une classe PHP...

C'est bien la problématique que j'évoque. En PHP tu ne peux pas définir cet id , c'est à la bdd de le faire. Donc si tu veux ajouter une company , tu vas créer un objet php avec un id à null car tu ne peux pas le définir.
Dans ta logique métier , tu sauras donc que si l'id de ton objet php est null c'est qu'il doit être crée dans la base.

Comment l'id peut-il être non nul s'il n'est pas géré par le constructeur de la classe ?

C'est à toi de décider comment tu veux le gérer. Soit par un second contructeur , soit par un setter , peut importe ca n'a pas de lien avec ta problématique.

C'est juste que ton objet peut être créer de 2 facons différentes , soit avec des données de la bdd , auquel cas l'objet sera complet (avec un id) Soit avec une instanciation manuelle et là l'id sera null.

Code :

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

1
2
3
4
5
6
7
8
9
// Création via un formulaire par exemple
$c = new Company($_POST['company']); // L'id sera null car non défini
$c->save(); // On va faire un insert car pas d'id donc nouvelle entrée dans la base
 
//Création suite à récup des données via bdd
$datas = //SELECT ....;
$c2 = new Company($datas['label'],$datas['id']);
// modification sur $c2 ...
$c2->save(); // On va faire un update car id != null

[laurentSc]

Jusque là on est d'accord ?

Même pas car j'aurais écrit $this->id au lieu de $id...

C'est juste que ton objet peut être créer de 2 facons différentes , soit avec des données de la bdd , auquel cas l'objet sera complet (avec un id) Soit avec une instanciation manuelle et là l'id sera null.

On va faire un insert car pas d'id donc nouvelle entrée dans la base

Et si on fait une instanciation manuelle, mais que la valeur qu'on a choisie pour peupler l'objet PHP est déjà dans la bdd, selon ce que tu dis, on va décider de faire un INSERT alors qu'il aurait fallu faire un UPDATE...

[grunk]

Même pas car j'aurais écrit $this->id au lieu de $id...

Oui coquille de ma part.

Et si on fait une instanciation manuelle, mais que la valeur qu'on a choisie pour peupler l'objet PHP est déjà dans la bdd, selon ce que tu dis, on va décider de faire un INSERT alors qu'il aurait fallu faire un UPDATE...

C'est de la logique métier. Il n'ya pas de raison de créer manuellement un objet qui existe déjà dans la base.
Dans le cas d'un formulaire , on aura par exemple un refus à la soumission disant que l'entrée existe déjà.

[laurentSc]

C'est de la logique métier. Il n'ya pas de raison de créer manuellement un objet qui existe déjà dans la base.
Dans le cas d'un formulaire , on aura par exemple un refus à la soumission disant que l'entrée existe déjà.

Pas si simple. Pour illustrer mon propos, je reprends ton exemple de classe en le complétant :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<?php
class Company
{
    private ?int $id = null;
    private string $label = '';
    private string $address;
 
    public function save() {
        if($this->id != null) {
            //UPDATE company SET ....
        } else {
            // INSERT INTO
        }
    }
}

Code sql :

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

1
2
3
4
5
6
CREATE TABLE company(
   id INT UNSIGNED AUTO_INCREMENT,
   label VARCHAR(30) NOT NULL,
   address VARCHAR(100),
   PRIMARY KEY(id)
);

J'ai rajouté une propriété $address dans la classe PHP et une colonne address dans la table SQL. Supposons donc qu'on ait à traiter manuellement les données (company1, address1) ; que company1 soit déjà en bdd mais que la colonne SQL address ait une valeur différente de address1. Alors, les données reçues ne doivent pas être rejetées et il faut faire un UPDATE. Il me semble logique de créer un objet Company mais le code prévu (ci-dessus) ne va pas vu que l'id sera nul...Comment traiterais-tu ce cas ?

[Nightfall]

Non Grunk a raison. Généralement pour l'utilisateur de ton appli soit il demande de créer une nouvelle fiche, soit il demande d'en modifier une existante (et donc il faut qu'il la retrouve d'abord dans une liste, un tableau, etc..). Et a priori dans le contrôleur du MVC ce n'est pas la même action. Ce n'est pas on rentre des données et le php se débrouille pour savoir s'il faut ajouter ou modifier. Donc s'il demande de créer une nouvelle société et que le nom existe déjà on affiche un message d'erreur, on modifie pas l'existante.

Ensuite pour mieux comprendre, je pense qu'il faut que tu distingues la partie technique et la partie métier. La partie technique c'est ce qui gère la récupération des entités dans la bdd et la sauvegarde, donc c'est le framework ou l'ORM, et la partie métier c'est la partie spécifique à l'application, comme le fait que tu ne veuilles pas avoir 2 sociétés avec le même nom. Parce que la partie technique on l'écrit une seule fois (cf l'héritage), on ne recommence pas pour chaque entité !

Donc pour la partie technique : Grunk t'a tout expliqué mais je te la refais de façon plus générale :

Un objet sait s'il est un nouvel objet à enregistrer ou s'il est un objet récupéré en base. Par exemple, une propriété isNew() indique si l'objet est un nouvel objet.

Pour créer : on instancie l'objet, on remplit les propriétés avec les valeurs reçues par post et on fait un save(). L'objet est nouveau donc le save() fait un insert. Si la clé primaire est auto-increment, le framework récupère la valeur de la clé primaire de l'objet enregistré (voir comment faire en fonction du sgbd) et la met dans la propriété de la classe. L'objet est maintenant dans un état enregistré, si on refait un save(), ce sera un update.

Pour modifier : on recherche l'objet à partir de la valeur de la clé primaire, par exemple:

Code :

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

1
2
3
4
5
6
7
8
$company = Company::findById($_POST['id']);
if(!$company) {
    //findById a renvoyé null donc objet non retrouvé, on affiche une erreur
}
else {
    //récupération des champs dans $_POST
    $company->save();
}

Là l'objet sait qu'il est un objet enregistré, donc le save() fait un update.

Ca c'est le principe général que tu retrouveras partout (si ce n'est, comme dit plus haut, que de nos jours ce ne sont plus les classes des entités qui se sauvegardent elles-mêmes mais c'est une classe repository qui récupère et enregistre les objets).

Maintenant pour ne pas avoir 2 sociétés avec le même nom, c'est à l'utilisateur de cette partie technique de se débrouiller Soit tu mets le champ label en clé primaire et tu enlèves l'auto-incrément, soit tu mets un index unique sur le champ label et tu gardes l'auto-incrément (perso je préfère cette méthode). Dans les 2 cas le save() plantera (lancement d'une exception) et tu affiches un message d'erreur. Si besoin, tu peux vérifier avant de faire le save() si une société avec le même nom existe déjà, si tu ne peux pas identifier le type de problème lors du save().

Tu pourrais regarder comment on utilise les bdd avec Symfony (Doctrine) ou Yii par exemple.

[Séb.]

OK,
c'est sûrement possible mais j'ai pas trouvé : je voudrais pouvoir compter le nombre de insert et le nombre de update. Comment obtenir cette information avec un insert on duplicate key ?

Avec PDO::rowCount() tu auras 1 ou 2 selon les cas. Doc MySQL :

For INSERT ... ON DUPLICATE KEY UPDATE statements, the affected-rows value per row is 1 if the row is inserted as a new row, 2 if an existing row is updated, and 0 if an existing row is set to its current values. If you specify the CLIENT_FOUND_ROWS flag, the affected-rows value is 1 (not 0) if an existing row is set to its current values.

https://dev.mysql.com/doc/refman/8.0...tion_row-count

[laurentSc]

Ce n'est pas on rentre des données et le php se débrouille pour savoir s'il faut ajouter ou modifier.

Ben, dans mon cas, si ! L'idée de l'application est la suivante : des données ont déjà été insérées dans une bdd. On fait un export au format CSV ; mon appli lit ce fichier CSV et va alimenter une nouvelle bdd avec les données contenues dans le CSV. Donc, les données reçues, on ne sait pas s'il faut ajouter ou modifier (ou rien faire). J'ai prévu des méthodes seekId et isUpdateRequired et on va passer leur retour à la méthode save. Donc, en fonction des paramètres reçus, la méthode sait s'il faut faire un INSERT ou un UPDATE.

Voici tel que je crée la table SQL :

Code sql :

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

1
2
3
4
5
6
CREATE TABLE CO_company(
   CO_ident INT UNSIGNED AUTO_INCREMENT,
   CO_company VARCHAR(30) NOT NULL,
   PRIMARY KEY(CO_ident),
   UNIQUE(CO_company)
);

Merci pour le lien, Séb.

[grunk]

Donc dans ton cas un INSERT ON DUPLICATE KEY est tout indiqué.

Sinon ca veux dire qu'il faut que pour chaque ligne du fichier tu vérifie dans la base si elle doit être modifié ou non. C'est juste impensable si tu as bcp de ligne.

[Séb.]

Je plussoie le INSERT ON DUPLICATE KEY.

La syntaxe est un peu moche (on écrit 2 fois la même chose dans la même requête) :

Code SQL :

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

1
2
INSERT INTO table SET id = 1234, c1 = 234, c2 = 345
ON DUPLICATE KEY UPDATE c1 = 234, c2 = 345

Mais ça évite les vérifs systématiques "manuelles" et de charger le serveur inutilement.

[laurentSc]

OK, je regarde puis je reviendrai.

[laurentSc]

Bonjour,

même si j'ai mis un certain temps à revenir, j'ai pas lâché l'affaire...

Le INSERT ON DUPLICATE KEY est au point mais j'ai du mal avec PDO::rowCount()...

Code php :

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
 public function save() {  
        $ppp = new PDOPlusPlus(); //classe géniale de Rawsrc ; un enrobage de PDO
 
            $sql = <<<sql
INSERT INTO pl_platform (PL_platform,PL_platformowner,PL_publisher) VALUES (
     {$ppp($this->PL_platform)}, {$ppp($this->PL_platformowner)}, {$ppp($this->PL_publisher)}
)
ON DUPLICATE KEY UPDATE 
PL_platformowner={$ppp($this->PL_platformowner)},
PL_publisher={$ppp($this->PL_publisher)}
sql;
            $id = $ppp->insert($sql);
            $pdo = $ppp->PDO();  //récupération du PDOstatement
          /*  $sql="SELECT * FROM pl_platform"; //ces 3 lignes servent à vérifier que $pdo est OK : c'est le cas
            $stm=$pdo->query($sql);
            $res=$stm->fetchAll();*/
            $nb=$pdo->rowCount(); /* j'ai aussi essayé $pdo::rowCount() et PDOPlusPlus::rowCount() */ /* 
Fatal error: Uncaught Error: Call to undefined method PDO::rowCount() in 
  C:\projets\ticket_rawsrc\model\license\table\PL_platform.php:135 Stack trace: #0 
  C:\projets\ticket_rawsrc\model\Debug.php(24): ticket_rawsrc\model\license\table\PL_platform->save() #1 
  C:\projets\ticket_rawsrc\index.php(106): ticket_rawsrc\model\Debug->invoke(Array) #2 {main} thrown in C:\projets\ticket_rawsrc\model\license\table\PL_platform.php on line 135 */
 
 
            return $id;
 
    }

WTF ?

[laurentSc]

Comme pas moyen d'utiliser rowCount(), j'adopte une méthode marteau-pilon :

Code php :

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
$ppp = new PDOPlusPlus(); //classe géniale de Rawsrc ; un enrobage de PDO
 
        $nb_lines_before = ($ppp->select('SELECT count(*) AS count FROM pl_platform')[0]['count']);
 
            $sql = <<<sql
INSERT INTO pl_platform (PL_platform,PL_platformowner,PL_publisher) VALUES (
     {$ppp($this->PL_platform)}, {$ppp($this->PL_platformowner)}, {$ppp($this->PL_publisher)}
)
ON DUPLICATE KEY UPDATE 
PL_platformowner={$ppp($this->PL_platformowner)},
PL_publisher={$ppp($this->PL_publisher)}
sql;
            $id = $ppp->execute($sql);
 
        $nb_lines_after = ($ppp->select('SELECT count(*) AS count FROM pl_platform')[0]['count']);
        if ($nb_lines_after == $nb_lines_before)
            DispMsg::DispMsg("update");
        else
            DispMsg::DispMsg("insert");

[Loralina]

Bonjour,

j'ai du mal avec PDO::rowCount()...

C'est plutôt PDOStatement::rowCount qui existe.

$pdo = $ppp->PDO(); //récupération du PDOstatement

Le commentaire est sans doute erroné.

Comme pas moyen d'utiliser rowCount(), j'adopte une méthode marteau-pilon :

Bof !
J'ai cherché la classe de rawsrc que tu utilises : la méthode execute conviendra peut-être. Essayer var_dump($ppp->execute($sql));.