Discussion :

[beegees]

Salut tout le monde,

J'insère une nouvelle ligne dans ma table avec des valeurs vides.

Je récupère l'ID le plus important.

Et j'aimerais faire un update de la nouvelle ligne en affectant le dernier ID + 1 à l'ID de la nouvelle ligne.

J'ai ce code qui ne fonctionne bien sûr pas :

Code :

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

1
2
3
4
5
6
7
8
9
$sql = "INSERT INTO ".$_SESSION['LaTable']." VALUES ()"; 
$execution=mysql_query($sql, $link)or die($query . " - " . mysql_error());
 
//Je prends la valeur du dernier ID 
$sql = 'SELECT ID FROM '. $_SESSION['LaTable']. ' ORDER BY ID DESC LIMIT 0,1';
$execution=mysql_query($sql, $link)or die($query . " - " . mysql_error());
 
$IncrementationID = 'UPDATE '.$_SESSION['LaTable'].' SET ID VALUES ('.$sql+1.') WHERE ID = ""' ; 
$execution=mysql_query($IncrementationID, $link)or die($query . " - " . mysql_error());

Je ne peux PAS utiliser l'auto-incrémentation car ce n'est PAS une clé primaire.

Sauriez-vous m'aider svp ?

Merci d'avance.

beegees

[Grepsd]

Déjà:

Code :

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

1
2
3
4
5
//Je prends la valeur du dernier ID 
$sql = 'SELECT ID FROM '. $_SESSION['LaTable']. ' ORDER BY ID DESC LIMIT 0,1';
$execution=mysql_query($sql, $link)or die($query . " - " . mysql_error());
 
$IncrementationID = 'UPDATE '.$_SESSION['LaTable'].' SET ID VALUES ('.$sql+1.') WHERE ID = ""' ;

La variable $sql contient ta précédente requête et non pas la valeur de l'ID voulue.

et tu n'as pas besoin de faire autant de requêtes:

Code :

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

1
2
 
UPDATE '.$_SESSION['LaTable'].' SET id = (SELECT ID+1 FROM '. $_SESSION['LaTable']. ' ORDER BY ID DESC)'

Et encore mieu:

Code :

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

'INSERT INTO table(ID,....) VALUES(SELECT max(ID)+1 FROM table,...)'

Là tu insert directement une nouvelle ligne avec une ID égale à la plus grande ID plus 1.


De plus ton code est difficilement lisible :/
et essayes d'utiliser MySQLi ou PDO pour tout ce qui concerne MySQL si possible pour des raisons de protections contre les injections et par ce que mysql_*() sera bientôt déprécie ^^"

De plus, essais de protéger par des `` les champs ayant des noms très utilisés en programmation (ID l'est énormément.)

[beegees]

Bonjour,

Merci pour ta réponse.

J'obtiens ce message d'erreur :

- You can't specify target table 'cours' for update in FROM clause

Avec ton code :

Code :

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

1
2
$IncrementationID = 'UPDATE '.$_SESSION['LaTable'].' SET id = (SELECT ID+1 FROM '. $_SESSION['LaTable']. ' ORDER BY ID DESC)'; 
$execution=mysql_query($IncrementationID, $link)or die($query . " - " . mysql_error());

Qu'en penses-tu ?

Encore merci.

beegees

[Grepsd]

Code :

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

max(ID)+1

et essayes:

Code :

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

1
2
 
$IncrementationID = 'UPDATE '.$_SESSION['LaTable'].' SET id = (SELECT max(ID)+1)';

je ne sais plus si cette syntaxe est correct mais il me semble que oui.


[edit]

J'avoue ne pas avoir la solution sous la main, je regarderais ça de façon plus approfondie dans la soirée ou d'ici demain. Si quelqu'un n'a pas répondu d'ici là bien entendu :p

[/edit]

[beegees]

Re,

Merci pour ton aide.

On dirait qu'il prend toujours comme ID 1 car j'obtiens ce message:

- Duplicate entry '' for key 1

j'utilise donc ton code :

Code :

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

1
2
$IncrementationID = 'UPDATE '.$_SESSION['LaTable'].' SET id = (SELECT max(ID)+1)';
$execution=mysql_query($IncrementationID, $link)or die($query . " - " . mysql_error());

Edit : le problème est qu'il ajoute un enregistrement avant le dernier.

Il met ensuite à jour l'id du dernier enregistrement, donc l'ID 3 devient 4 et il ajoute une nouvelle linge qui porte l'ID 1 qui existe déjà.

Qu'en penses-tu ?

Encore merci.

beegees

[Jumano]

Bonjour,
Essaye ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
$sql = "INSERT INTO ".$_SESSION['LaTable']." VALUES ()"; 
$execution=mysql_query($sql, $link)or die($query . " - " . mysql_error());
 
$query_IDmax = "SELECT ID FROM ".$_SESSION['LaTable']." ORDER BY ID DESC LIMIT 1";
$IDmax = mysql_query($query_IDmax, $link) or die(mysql_error());
$row_IDmax = mysql_fetch_assoc($IDmax);
$max = ($row_IDmax['ID']+1);
 
$IncrementationID = "UPDATE ".$_SESSION['LaTable']." SET ID = '".$max."' WHERE ID = '0'";
$execution=mysql_query($IncrementationID, $link)or die($query . " - " . mysql_error());

[Grepsd]

Justement, c'est ce que l'on cherche à éviter pour ne pas surcharger avec 3 requêtes différentes ce que l'on peut faire avec une seule comprenant une sous requête.

J'avais déjà utiliser un système proche mais je ne l'ai plus sous la main.

[Grepsd]

http://www.webmasterworld.com/forum112/345.htm

Après une petite recherche je suis tombé là dessus, et donc:

Code :

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

INSERT INTO table (ID, ...) VALUES(SELECT (SELECT max(ID) FROM table) AS ID,....)

[jeca]

Bonjour,

Avec ce genre de requête, tu vas au devant des problèmes : imagine que 2 sessions différentes accèdent en même temps à la requête, elles trouveront toutes les 2 le même id. Quel fouillis ça va faire dans la table.
Il FAUT que ton id soit auto incrémentée si elle doit être unique.

[beegees]

Bonjour à tous,

Message commun :

Merci à tous pour vos réponses.

Je n'ai pas reçu la confirmation de réponse, c'est pourquoi je réponds si tard.

beegees

Bonjour,
Essaye ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
$sql = "INSERT INTO ".$_SESSION['LaTable']." VALUES ()"; 
$execution=mysql_query($sql, $link)or die($query . " - " . mysql_error());
 
$query_IDmax = "SELECT ID FROM ".$_SESSION['LaTable']." ORDER BY ID DESC LIMIT 1";
$IDmax = mysql_query($query_IDmax, $link) or die(mysql_error());
$row_IDmax = mysql_fetch_assoc($IDmax);
$max = ($row_IDmax['ID']+1);
 
$IncrementationID = "UPDATE ".$_SESSION['LaTable']." SET ID = '".$max."' WHERE ID = '0'";
$execution=mysql_query($IncrementationID, $link)or die($query . " - " . mysql_error());

Bonjour Jumano,

Merci pour ton code.

ça fonctionne avec le premier enregistrement, si par exemple le dernier enregistrement avait l'ID 33, il donne l'ID 34 au nouveau, c'est une bonne avancée.

Par contre, dès que j'ajoute un autre enregistrement, il me dit ceci :

- Duplicate entry '' for key 1

Qu'en penses-tu ?

Merci encore.

beeges

Justement, c'est ce que l'on cherche à éviter pour ne pas surcharger avec 3 requêtes différentes ce que l'on peut faire avec une seule comprenant une sous requête.

J'avais déjà utiliser un système proche mais je ne l'ai plus sous la main.

http://www.webmasterworld.com/forum112/345.htm

Après une petite recherche je suis tombé là dessus, et donc:

Code :

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

INSERT INTO table (ID, ...) VALUES(SELECT (SELECT max(ID) FROM table) AS ID,....)

Bonjour,

Merci pour ta réponse Grepsd.

J'ai essayé de faire comme tu le dis :

Code :

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

$sql = "INSERT INTO ".$_SESSION['LaTable']." VALUES(SELECT (SELECT max(ID) FROM ".$_SESSION['LaTable']. "AS ID));"

Pour l'instant, ça me donne :

Parse error: syntax error, unexpected T_VARIABLE in C:\wamp\www\Scripts\Insert_Into_Generation_Chaine_SQL.php on line 26

La ligne 26 correspond à ceci :

Code :

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

$execution=mysql_query($sql,$link)or die($query . " - " . mysql_error());

merci encore pour tout.

beegees

Bonjour,

Avec ce genre de requête, tu vas au devant des problèmes : imagine que 2 sessions différentes accèdent en même temps à la requête, elles trouveront toutes les 2 le même id. Quel fouillis ça va faire dans la table.
Il FAUT que ton id soit auto incrémentée si elle doit être unique.

Bonjour,

Merci pour ta réponse.

Deux choses :

1) Le même ID --> impossible car je suis le seul à ajouter des enregistrements et je ne peux ajouter qu'un enregistrement qu'à la fois. En plus, il prend le dernier ID d'une certaine table et ensuite d'une autre table si j'ajoute dans une autre table.

2) si j'aurais pu laisser en auto incrément, je l'aurais fait, c'est juste que mon analyse ne me le permet pas.

merci
beegees

[Jumano]

Re,
Tes champs ID ne sont-ils pas de TYPE = UNIQUE ?

[beegees]

Re,
Tes champs ID ne sont-ils pas de TYPE = UNIQUE ?

Re,

Non il ne sont pas de type "Unique".

On sait quand même faire quelque chose ?

Je peux le mettre en unique si tu veux, ça doit pas être obligatoirement une primary key alors ?

Merci pour ton aide.

beegees

[Jumano]

Tu dois avoir une clé primaire ou un champs de type unique dans ta table qui contient déjà une valeur vide, donc quand tu ajoutes une nouvelle valeur vide tu as ce message d'erreur.

[beegees]

Tu dois avoir une clé primaire ou un champs de type unique dans ta table qui contient déjà une valeur vide, donc quand tu ajoutes une nouvelle valeur vide tu as ce message d'erreur.

OK problème résolu, il faut encoder une ligne à la fois et faire une mise à jour, ensuite on peut ajouter une autre ligne.

Merci à toi jumano

beegees