Discussion :

[dancom5]

Bonjour la communauté.

A l'appel de ma fonction insertIP() une inscription se fait pour stocker la date et le IP dans la bdd.
A la 6e ligne, c'est pour afficher le dernier ID mais je veux pouvoir l'afficher que si j'en ai besoin.
Il se pourrait bien que j'aie besoin de ce ID pour une commande select qui sera dans une autre
fonction pour une requête.

Code :

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

1
2
3
4
5
6
7
8
 
function insertIP(){
    global $dbConn;
    $query = $dbConn->prepare("INSERT INTO flood(ip, ladate) VALUES(:ip, NOW())");
    $query->bindvalue(':ip', detectIP());
    $query->execute();
    return $dbConn->lastInsertId();
}

Ma question est en rapport avec la syntaxe à utiliser dans ma fonction et à son utilisation.
Comment?

[badaze]

J'avoue ne pas comprendre le problème.

Code php :

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

1
2
 
$dernierID = insertIP();

[dancom5]

Principalement ma fonction enregistre le IP et la date dans la base de données.
En appelant insertIP(), c'est ce que ça va faire.

Cependant, si je souhaite que le dernier ID soit affiché qui confirme l'enregistrement,
c'est le rôle de ce bout de code mis tout de suite après la requête pour faire cela :

Code :

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

$dbConn->lastInsertId();

Comme j'ai mis à la suite de la requête Insert, il va s'afficher dès que je vais appeler
ma fonction insertIP() et aussi insérer l'enregistrement.

Ce que je souhaite, faire, c'est que ça soit optionnel l'affichage du dernier ID inséré.
Mais que je puisse l'afficher. J'avais penser à l'appel via un argument comme ceci:
insertIP(afficher)
(en exemple pour illuster ma demande.)

NOTE: je vais exprimer autrement:
Si j'indique quelque chose en argument de la fonction, ça va afficher le retour de lastInsertId
si non ne l'affiche pas.

J'avoue ne pas comprendre le problème.

Code php :

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

1
2
 
$dernierID = insertIP();

[badaze]

Mettre un echo ou print dans la fonction ne me semble pas très propre. En effet tu peux faire tes inserts à des "endroits" dans ton script où tu n'as pas forcément quelque chose à afficher. De plus avec cette façon tu ne sépares pas l'affichage de la manipulation des données.

[dancom5]

La seule chose que j'ai trouvé par hasard par des tests, c'est ceci d'ajouter un echo:

Code :

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

echo insertIP();

Mais, il peut arriver qu'on veuille ajouter des commandes dans une fonction sans vouloir
les utiliser tout de suite comme c'est le cas avec ma fonction. Il se peut bien que je souhaite
seulement effectuer l'enregistrement sans afficher le dernier ID entré.

ci-haut, ça va afficher le dernier ID après avoir effectué l'enregistrement.

Je ne considère pas ça comme une solution viable puisqu'il doit surement exister une autre
solution pour les autres commandes.

On parle souvent d'argument facultatif mais dans ma fonction, c'est une commande qui est
facultative pas un argument à la fonction.

[dancom5]

Il serait alors souhaitable d'afficher le dernier ID en dehors de la fonction. Mais comment quand c'est LastInsertId?

[badaze]

Tu sauvegardes la valeur.

Code php :

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

1
2
$dernierID = insertIP();

[dancom5]

Au finale, ça ne fonctionne pas.
J'arrive à enregistrer le IP et la date, mais pas afficher le dernierID.

Peut-être que ça peut vous indiquer si je mets la source d'essai:

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
//---------------------------------------------
// enregister le IP flood
function insertIP(){
  global $dbConn;
    $query = $dbConn->prepare("INSERT INTO flood(ip, ladate) VALUES(:ip, NOW())");
    $query->bindvalue(':ip', detectIP());
    $query->execute();
    return $dbConn->lastInsertId();
}
 
include_once('config.php');
insertIP();
 
$dernierID = insertIP();
 
echo $dernierID;

[xdelatour]

Et en supprimant la ligne 12 ?

Cette ligne appelle la fonction mais sa valeur de retour est ignorée, si le echo affiche quelque chose, ce devrait être l'id créé par l'appel fait à la ligne 14 et pas celui créé par l'appel de la ligne 12.

[dancom5]

Bonjour xdelatour; entre les lignes 11 et 16, si j'écris la ligne 14, ça exécute la requête sans afficher le ID.

En passant, l'insertion de la date via NOW(), j'ai remarqué que ce n'était pas nécessaire de l'ajouter avec INSERT
puisque dans ma base de donnée, j'ai une commande SQL de mis pour la date directement dans le champ
via DATE.

Pour les fonctions PHP comme je voulais faire, je vais m'attarder à faire de quoi un peu plus «portable».
Si quelqu'un a une idée où je pourrais plus avoir de l'information la dessus.

Et en supprimant la ligne 12 ?

Cette ligne appelle la fonction mais sa valeur de retour est ignorée, si le echo affiche quelque chose, ce devrait être l'id créé par l'appel fait à la ligne 14 et pas celui créé par l'appel de la ligne 12.

[badaze]

Au risque de me répéter afficher le dernier id dans la fonction qui fait l'insert est une fausse bonne idée.

[gene69]

Salut
Badaze a totalement raison. Tu as encore un niveau ou tu ne t'apperçois pas qu'il existe deux types de fonctions/méthodes. Il y a celles qui travaillent (calculent, manipulent,...) et celle qui affichent.

Apres faire plusieurs appel à lastInsertID() ça augmente le risque qu'une autre transaction ait fini une autre transaction et donc changé cet id (et là si ya des connexions persistantes...). Moi je suis partisant de la plus grande méfiance avec cette méthode. On l'appelle le plus vite possible apres l'insert et on stoque le retours dans une variable php.

Vous remarquez que le manuel ne garantie pas que cette fonction est idempotente

[dancom5]

Code :

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

1
2
3
4
 
$test = insertIP();
insertIP();
echo insertIP();

Les 3 va effectuer l'enregistrement.
Le dernier va l'afficher en plus.

Ça va être embêtant si je veux conserver un peu plus loin la variable $test ou faire un echo sur insertIP sans effectuer l'enregistrement.

Comme dans le cas suivant:

J'insert le IP et la date et un peu plus loin, je récupère le ID qui a fait l'objet de l'enregistrement.

[badaze]

Je ne comprends pas ton souci. Tu sauvegardes le résultat de insertIP() dans une variable et tu utilises cette variable quand tu en as besoin si tu en as besoin. Si tu en fais un echo tu auras toujours le dernier id inséré sauf si tu le fais avant le premier appel à inserIP().

[xdelatour]

Est-ce-que le code suivant répond à la problématique ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
function insertIP($flag=null){
    global $dbConn;
    $query = $dbConn->prepare("INSERT INTO flood(ip, ladate) VALUES(:ip, NOW())");
    $query->bindvalue(':ip', detectIP());
    $query->execute();
    if ($flag == "afficher")  
                 echo $dbConn->lastInsertId(); 
    return $dbConn->lastInsertId();
}
 
insertIP();
insertIP("afficher");

Par contre, l'insertion se fait à chaque appel de la fonction, dans tous les cas et l'ID créé est renvoyé, seul l'affichage dépendra de la présence ou non de l'argument lors de l'appel.

[dancom5]

Oui je comprends.

Ayant vu les fonctions, je regarde maintenant du côté des objets pour tenter de faire mieux dans la portée
des requêtes. ici. Mais, je maitrise pas totalement le sujet un coup extentié. Je marque résolu.