Discussion :

[gastoncs]

Bonjour j'ai ma requête sql suivante :

Code :

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

INSERT INTO feraille (datefer, postefer, nbrfer) VALUES ('$dateDuJour', '$poste', '$nbrfer') ON DUPLICATE KEY UPDATE nbrfer='$nbrfer'

et ma table feraille suivante :
******************
*ID****NOM****NBR*
******************
*1***PIERRE****4***
*2***PAUL*****5***
*3***JAQUE****2***

id est ma clef primaire, et lorsque je fais une modification, si par exemple je modifie le nbr de PIERRE et bien il va crée une nouvelle entrée dans la table avec pour nom PIERRE et un nouvel id !
je ne vois pas ou est le probléme !

[sabotage]

Il faut que ton champ "nom" soit UNIQUE pour que le "ON DUPLICATE KEY" agisse.

[gastoncs]

donc je dois mettre la clef primaire sur nom et pas id alors ?
dans le code de la requete que j'ai copié, j'ai "postefer" et "datefer" et id qui est sur clef primaire, dans ce cas la aussi je devrai definir les deux colonne là en clef primaire ???

[RunCodePhp]

Salut

id est ma clef primaire, et lorsque je fais une modification, si par exemple je modifie le nbr de PIERRE et bien il va crée une nouvelle entrée dans la table avec pour nom PIERRE et un nouvel id !
je ne vois pas ou est le probléme !

Le code que tu donne INSERT INTO est fait pour créer une nouvelle ligne, un nouvel enregistrement.

Pour faire une modification, c'est

Code sql :

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

UPDATE une_table SET champ = 'une_valeur' WHERE ... etc ...

C'est peut être pour ça que tu remarque que ça crée une ligne, non ?

[gastoncs]

mais je pensez que le "insert into" et le "on "duplicat key" compiler permetté lorsque la valeur n'existe pas de la crée sinon si il le met à jour !

[sabotage]

Tu dois définir un index UNIQUE sur les champs qui doivent l'être.

[RunCodePhp]

mais je pensez que le "insert into" et le "on "duplicat key" compiler permetté lorsque la valeur n'existe pas de la crée sinon si il le met à jour !

C'est la façon dont tu procède qui me chiffonne, de la logique.

Au départ, il y a bien une action, non ?
Il doit bien avoir une manip de l'utilisateur, c'est à dire, soit une création, soit une modification, il ne devrait pas avoir de hasard ici, non ?

Si c'est une modification par exemple, et théoriquement, avant même que l'utilisateur valide la modif, il doit bien avoir une donnée (un ID en général) qui va déterminer quelle ligne à modifier.

Si c'est un nouvelle enregistrement, c'est encore plus évident, car par définition, un nouvel enregistrement veut dire que la ligne n'existe pas.
Je dirais même qu'en général ici, c'est l'inverse qui se fait, soit vérifier (une requête avant) si part hasard on ne serait pas entrain de créer 2 fois la même donnée.


Théoriquement, les choses sont connues.

mais je pensez que le "insert into" et le "on "duplicat key" compiler permetté lorsque la valeur n'existe pas de la crée sinon si il le met à jour !

Théoriquement oui, mais il me semble que c'est sur la clé primaire, au cas ou celle ci existerait déjà.
Mais c'est quoi qui te pose problème ?
Qu'est-ce que tu veux faire et surtout quoi éviter ?
Pour ma part, si tu as un ID comme clé primaire et en auto_increment, tu ne devrait pas avoir de problème, il y aurait rien à rajouter.
Ne peux tu pas t'appuyer sur ce que tu as pour soit faire un UPDATE ou un INSERT INTO

[gastoncs]

d'accord donc je résume car j'ai du mal à comprendre Désolé, sur ma page j'ai ma requete SQL :

Code :

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

("INSERT INTO feraille (datefer, postefer, nbrfer) VALUES ('$datefer', '$postefer', '$nbrfer') ON DUPLICATE KEY UPDATE nbrfer='$nbrfer'");

j'ai $datefer qui vaut la date du jour
et $postefer qui vaut le poste en cours (soit après midi entre 13h et 21h)

voici ma table feraille avec id comme clef primaire :


ce que je souhaite faire c'est que lorsque l'utilisateur rentre un chiffre dans "$nbrfer" celui ci crée une entré dans ma table avec pour "datefer" la date du jour et "postefer" le poste en cours, mais lorsqu'il y a deja une entré avec la date du jour et le poste en cours, alors on modifie juste la valeur "$nbrfer" que l'utilisateur va rentré.
Merci pour votre aide

[RunCodePhp]

mais lorsqu'il y a deja une entré avec la date du jour et le poste en cours

Et bien la voilà ta condition
Il te faudrait donc faire une requête SQL avant, et c'est son résultat qui va déterminer si c'est une insertion ou une modification qu'il faudra faire après.
Théoriquement, il n'y pas lieu de "casser" la conception de cette table ou rajouter je ne sais quoi.

Ta requête (un SELECT) doit rechercher : pour le poste en court si la date correspond à la date en court. (ça devrait être 0 ou 1).

[gastoncs]

Super bonne idée, j'ai fait comme tu ma dit, mais j'ai un dernier petit soucis, il veu bien me faire la mise à jour, mais pas l'insertion ;

voici mon code :

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
17
18
19
20
21
if (isset($_POST['datefer']) AND isset($_POST['postefer']) AND isset($_POST['nbrfer'])){
 
	// passe les valeurs $POST en variables
		$datefer = $_POST['datefer'];
		$postefer = $_POST['postefer'];
		$nbrfer = $_POST['nbrfer'];
 
if (isset($_POST['action']) && $_POST['action'] == "rajouterfer"){
		//insert into dans commentaires
		if ($retour = mysql_query("SELECT * FROM feraille WHERE postefer='$poste' AND datefer='$dateDuJour'")){
			$modif_feraille = "UPDATE feraille SET nbrfer= '$nbrfer'"; 
		$modifier = mysql_query($modif_feraille, $dbprotect) or die(mysql_error());
		header("Location:rapport.php?addfer=ok"); // message comme quoi l'enregistrement à été fait
		}
		else { $add_ferailles = sprintf("INSERT INTO feraille (datefer, postefer, nbrfer) VALUES ('$datefer', '$postefer', '$nbrfer')");
  		mysql_select_db($database_dbprotect, $dbprotect);
  		$result = mysql_query($add_ferailles, $dbprotect) or die(mysql_error());
	header("Location:rapport.php?addfer=ok"); // message comme quoi le commentaire à été rajouter
		}
}
}

[RunCodePhp]

Code :

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

if ($retour = mysql_query(...) ...)

Non, c'est pas bon car la fonction mysql_query() retourne une ressource, et non pas le résultat, que la requête retourne 1 ligne ou 0 ligne.
Fais un echo $retour tu comprendras mieux.
Si la requête ne plante pas (erreur de syntaxe par exemple), faire un if ($retour) retournera toujours VRAI.

Il te faut vérifier le résultat, et dans ton cas un 0 ou 1 suffirait.
Du coup, et théoriquement un SELECT COUNT() ... suffirait.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
$res_total = mysql_query("SELECT COUNT(*) AS total FROM feraille WHERE postefer='$poste' AND datefer='$dateDuJour'") or die('Erreur : '.mysql_error());
// La requête retournera le nombre de ligne.
$total = mysql_fetch_array($res_total);
 
if ((int)$total['total'] > 0) {
    // L'enregistrement existe : INSERT INTO
}
else {
    // L'enregistrement n'existe pas : UPDATE
}