Discussion :

[RinaBK]

Bonjour tout le monde,

Lorsque un membre poste un message sur la messagerie de mon site, mon code envoi un email aux autres membres afin de leur prévenir qu'un nouveau message a été envoyé. Dans cet email, les membres peuvent cliquer sur un lien qui doit les diriger directement depuis le nouveau message envoyé dans la messagerie. Pour cela, j'ai besoin du dernier numéro ID du dernier message afin que les membres soit directement redirigé sur ce dernier message. Donc, j'ai utilisé la fonction mysqli_insert_id() qui devrait normalement me retourner le numéro ID (AUTO_INCREMENT) qui me servira dans le lien dans l'email. Mais voilà que je peux faire tout ce que je veux, mysqli_insert_id() me retourne continuellement 0. Pourtant je n'ai pas d'autres requètes avant mon INSERT, alors je vous avoue que je ne comprends pas trop où est le problème.

Code :

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

1
2
3
4
if (mySqli_query(CON_DB,'INSERT INTO message VALUES ("AUTO_INCREMENT", '.$id_user.', CURRENT_TIMESTAMP, "'.$msg.'")')) {
	$dernierMsg = mysqli_insert_id(CON_DB);
	echo $dernierMsg;
}

Quelqu'un peut m'aider à comprendre ce qui ne va pas?

[mathieu]

après chaque requête INSERT, voyez-vous bien le nouvel enregistrement dans la table ? avec un identifiant différent des autres ?

[RinaBK]

Oui, les enregistrements fonctionnent super !
On peut voir les messages que j'ai envoyé comme test ICI au bas de la page.
Tout fonctionne super bien, sauf mysqli_insert_id()

[Paraffine]

Tout fonctionne super bien, sauf mysqli_insert_id()

Et si tu passes par une variable pour ta connexion, plutôt que par une constante ?

[RinaBK]

Et si tu passes par une variable pour ta connexion, plutôt que par une constante ?

Oh ! Désolée... En effet pour l'exemple j'ai mis une consente, mais en réalité c'est une fonction. Effectivement, pour mon exemple j'aurais du mettre : cddb()

[MaitrePylos]

Bonsoir,
On est bien sur que id est primary key dans la table ?

[RinaBK]

Oui, voici ma table :

Code SQL :

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

1
2
3
4
5
6
7
8
CREATE TABLE XXX_tutosmsg (
	id BIGINT UNSIGNED NOT NULL AUTO_INCREMENT,
	id_tuto BIGINT UNSIGNED NOT NULL,
	id_mbr BIGINT UNSIGNED NOT NULL,
	edate TIMESTAMP ON UPDATE CURRENT_TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP,
	msg TEXT NOT NULL,
	PRIMARY KEY (id)
);

[Paraffine]

Oh ! Désolée... En effet pour l'exemple j'ai mis une constante, mais en réalité c'est une fonction. Effectivement, pour mon exemple j'aurais du mettre : cddb()

Puisque l'insertion fonctionne, que la colonne est bien une clé primaire avec un auto-incrément, j'ai l'impression qu'il ne reste plus que cette connexion à se mettre sous la dent ; quel est le code de ta fonction ?

[RinaBK]

Puisque l'insertion fonctionne, que la colonne est bien une clé primaire avec un auto-incrément, j'ai l'impression qu'il ne reste plus que cette connexion à se mettre sous la dent ; quel est le code de ta fonction ?

Une fonction des plus basique. La voici :

Code PHP :

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

1
2
3
4
5
6
7
8
function cddb() {
	$host = 'localhost';
	$user = 'root';
	$pass = '';
	$base = 'gwanda';
	$dbc = mySqli_connect($host,$user,$pass,$base);
	return $dbc;
}

Je précise juste que je ne me connecte pas en localhost mais via mon serveur en ligne.

[Paraffine]

Bizarre, cela devrait bien réutiliser la connexion existante.
As-tu essayé en stockant la connexion dans une variable pour ne pas refaire appel à la fonction entre les deux instructions ?
Au cas où le mysqli_connect réinitialiserait le dernier id...

[MaitrePylos]

s ?
Au cas où le mysqli_connect réinitialiserait le dernier id...

Je plussoie, si il a relecture à chaque fois, la connexion se ferme et se rouvre, donc on a surement deux connexions différentes.
Comme le dit @Paraffine, essaye en mettant une fois la connexion dans une variable, ou réutilise la classe PDO que tu as mise en place dans un autre post.

[RinaBK]

Effectivement, en mettant la connexion dans une variable mysqli_insert_id() renvoi le dernier ID inséré
Mais là, je ne comprends pas trop... C'est simplement le fait de passer ma connexion via ma fonction qui pose ce problème??
Bon, de toute façon, vais devoir réviser la totalité de mes pages pour passer en mode objet Mais tout de même, c'est assez intriguant de voir qu'une simple fonction de rien du tout peu empêcher le bon fonctionnement d'une fonction PHP Je suis sur les fesses pour rester polie

En tout cas, merci à vous deux pour m'avoir aidé à trouver la cause de ce problème

[Paraffine]

(...) s'il y a relecture à chaque fois, la connexion se ferme et se rouvre, donc on a surement deux connexions différentes.

Ouiiii, j'avais confondu avec mysql_connect, en plus, pour le coup de la réutilisation de la connexion ! Comme mysqli_connect est l'équivalent de mysqli::__construct() on est chocolat pour le dernier id...

[MaitrePylos]

Ouiiii, j'avais confondu avec mysql_connect, en plus, pour le coup de la réutilisation de la connexion ! Comme mysqli_connect est l'équivalent de mysqli::__construct() on est chocolat pour le dernier id...

@RinaBk pour que tu comprennes.

Quand tu mets le résultat de ta fonction cddb() dans une variable et que tu ne touches plus à cette variable, alors le contenu ne bouge plus.
Dans ton cas tu as une connexion à la db.

Si à chaque fois que tu as besoin de ta db, tu réutilises ta fonction cddb(), alors tu crées d'office une nouvelle connexion (ce qui n'est pas grave en soi), mais du coup, tu perds la connexion précédente et toutes ses infos, notamment le dernier id , puisque celle-ci sera réinitialisée à chaque nouvelle connexion.

Comprends-tu le principe ?

[RinaBK]

@RinaBk pour que tu comprennes.

Quand tu mets le résultat de ta fonction cddb() dans une variable et que tu ne touches plus à cette variable, alors le contenu ne bouge plus.
Dans ton cas tu as une connexion à la db.

Si à chaque fois que tu as besoin de ta db, tu réutilises ta fonction cddb(), alors tu crées d'office une nouvelle connexion (ce qui n'est pas grave en soi), mais du coup, tu perds la connexion précédente et toutes ses infos, notamment le dernier id , puisque celle-ci sera réinitialisée à chaque nouvelle connexion.

Comprends-tu le principe ?

Effectivement, je comprends mieux maintenant, merci pour cette belle explication !
Donc, il aurait été préférable de mettre, par exemple, l'appel à ma fonction sur le haut de ma page $ddb = cddb(); et d'utiliser pour chaque requête la variable $ddb. Comme cela, j'utiliserais la même connexion et j'éviterais donc ce genre de problème.

[MaitrePylos]

Effectivement, je comprends mieux maintenant, merci pour cette belle explication !
Donc, il aurait été préférable de mettre, par exemple, l'appel à ma fonction sur le haut de ma page $ddb = cddb(); et d'utiliser pour chaque requête la variable $ddb. Comme cela, j'utiliserais la même connexion et j'éviterais donc ce genre de problème.