Discussion :

[L'AMI Yves]

Voici mon code :

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
            $query="INSERT INTO Appels VALUES(NULL, NOW(), " . $Ut["Numero"] . ", " . $_SESSION["Val3"] . ", '" . $_SESSION["IP"] . "', 'RAS')";
            mysqli_query($conn,"LOCK TABLES Appels WRITE") or die(mysqli_error($conn));        //Astuces trouvées sur le NET
            mysqli_query($conn,"SET AUTOCOMMIT = 0") or die(mysqli_error($conn));              //pour être sûr de récupérer
            mysqli_query($conn,$query) or die(mysqli_error($conn));
            $result = mysqli_query($conn,"SELECT LAST_INSERT_ID()") or die(mysqli_error($conn));
            mysqli_query($conn,"COMMIT") or die(mysqli_error($conn));                          //le bon numéro...
            mysqli_query($conn,"UNLOCK TABLES") or die(mysqli_error($conn));                   //en bloquant un autre utilisateur
            $numero=mysqli_fetch_row($result); 
            mysqli_free_result($result);
           //Enregistrement de la nouvelle IP dans la table "Lieux"
            $query="UPDATE Lieux SET IP = '" . $_SESSION['IP'] . "', Appel = " . $numero[0] . " WHERE Numero = " . $_SESSION['Val3'] . " AND Utilisateur = ". $_SESSION['Val1'];
var_dump($query);
            mysqli_query($conn,$query) or die(mysqli_error($conn));
            $number=mysqli_affected_rows($conn);
            var_dump($number);

Et l'affichage obtenu :

Code :

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

1
2
string(84) "UPDATE Lieux SET IP = '192.168.1.51', Appel = 7 WHERE Numero = 2 AND Utilisateur = 1"
int(1)

J'ai bien vérifié les ";".

Je vois avec phpMyAdmin que la table "Appels" répond normalement.
Elle a pour premier item 'Numero' qui est auto_increment. Il faut donc être sûr qu'il n'y a pas un autre
"auto_incrément" qui passe oar là par hasard. Ai-je bien compris l'astuce trouvée sur le net ?

La fonction "affected_rows" me dit que 1 enregistrement de "Lieux" est affecté, ce que j'attendais.

Et pourtant phpMyAdmin me montre que la table "Lieux" n'est pas mise à jour...

Merci de m'aider encore une fois.

AMIcalement.

[Nb]

Bonjour,
Je pense que ton update n est pas commit.

[L'AMI Yves]

Ah, ça doit bien être ça. Voila ce que c'est que de pomper des bouts de code qu'on ne comprend pas.

J'ai ajouté une ligne après la lecture du numéro d'enregistrement auto_incrément, et ça marche.

Merci !

AMIcalement.

[escartefigue]

Je vois avec phpMyAdmin que la table "Appels" répond normalement.
Elle a pour premier item 'Numero' qui est auto_increment. Il faut donc être sûr qu'il n'y a pas un autre
"auto_incrément" qui passe oar là par hasard. Ai-je bien compris l'astuce trouvée sur le net ?

L'auto_increment est affecté pour un thread, il n'y a aucun risque de récupérer la valeur d'un autre thread par la fonction LAST_INSERT_ID() !

J'ai ajouté une ligne après la lecture du numéro d'enregistrement auto_incrément, et ça marche.

Il est essentiel de comprendre la notion de transaction, de commit et de rollback. Il ne faut pas coder au hasard en repompant des bouts de code sans comprendre ces fondamentaux !

[Artemus24]

Salut à tous.

Quelques remarques :

a) il est préférable d'indiquer l'ordre des colonnes dans un INSERT, ce que vous ne faites pas.

b) au lieu de mettre NULL dans votre INSERT, il est préférable d'indiquer dans la déclaration de la table pour cette colonne "default NULL".
Et bien sûr de ne pas référencer cette colonne dans votre INSERT. Le NULL sera automatiquement affecté si aucune valeur n'est présente.

c) il est préférable d'indiquer dans le fichier "my.ini" (ou my.cnt) le mode transactionnel que vous utiliser.
Le "set autocommit=0" n'a pas lieu d'être dans votre script.

d) une transaction commence par un "start transaction" et se termine soit par un "commit", soit par un "rollback". Où sont-ils ?

e) on peut en effet récupérer la dernière valeur de votre identifiant "id".
Il me semble que cela pourrait se résoudre par un déclencheur, ce qui éviterait de surcharger inutilement votre script.
N'ayant pas plus d'information à ce sujet, je ne peux pas me prononcer sur la validité de ce que je viens de dire.

f) pourquoi utilisez vous mysqli qui est ancien et non pas pdo ?

@+