Discussion :

[PooSoft]

Bonjour, maintenant que je pense ne plus avoir d'erreur d'ordre SQL, ça doit être du coté PHP.
Je veux insérer dans une table les résultats d'une requête :
J'ai fais ceci :

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
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
		$requete = "SELECT date_fin, Table1.num_contrat, Table2.nom_du_contact, Table1.num_client, email_du_contact
					FROM Table1, Table2 
					WHERE Table1.num_contrat = Table2.num_contrat
					AND  TO_DAYS(`date_fin`) - TO_DAYS(NOW()) <=60;";
 		$resultat = mysql_query($requete) or die(mysql_error());
		if (mysql_num_rows($resultat) == 0) {
			   echo "Aucune ligne trouvée, rien à afficher.";
			   exit;		
		}
		else {
			$num_rows = mysql_num_rows($resultat);
			echo "$num_rows ligne(s) trouvée(s)\n";
		}
		while ($row = mysql_fetch_assoc($resultat)) {
			    $contrat = $row["num_contrat"];
				$nom = $row["nom_du_contact"];
				$client = $row["num_client"];
				$date = $row["date_fin"];
				$mailaddress = $row["email_du_contact"];
		}
$sql1 = "SELECT TO_DAYS(`date_fin`) - TO_DAYS(NOW())
		FROM Table1";
		echo $sql1;
$res1 = mysql_query($sql1) or die(mysql_error());
switch ($res1) {
	case 60 :
		$req2 = "INSERT INTO emailenvoyes (`num_contrat`, `num_client`, `dateenvoi`,`email`, `email_envoye60`) 
 					 VALUES ($contrat, $client, NOW(), '$mailaddress', 1)";
		echo $req2;
		mysql_query($req2) or die(mysql_error());
		break;
	case 30 :
		$req2 = "INSERT INTO emailenvoyes (`num_contrat`, `num_client`, `dateenvoi`,`email`, `email_envoye30`) 
 					 VALUES ($contrat, $client, NOW(), '$mailaddress', 1)";
		echo $req2,"<br />";
		mysql_query($req2) or die(mysql_error());
		break;
	case 15 :
		$req2 = "INSERT INTO emailenvoyes (`num_contrat`, `num_client`, `dateenvoi`,`email`, `email_envoye15`) 
 					 VALUES ($contrat, $client, NOW(), '$mailaddress', 1)";
		echo $req2,"<br />";
		mysql_query($req2) or die(mysql_error());
		break;
	case 7 :
		$req2 = "INSERT INTO emailenvoyes (`num_contrat`, `num_client`, `dateenvoi`,`email`, `email_envoye7`) 
 					 VALUES ($contrat, $client, NOW(), '$mailaddress', 1)";
		echo $req2,"<br />";
		mysql_query($req2) or die(mysql_error());
		break;
}

On dirait qu'il n'entre jamais dans mon switch, pourtant j'ai des données dans ma base.
Si quelqu'un pouvait m'éclairer

[Benjamin Delespierre]

Déjà, le résultat renvoyé par mysql_query est une ressource et non un entier. Tu dois impérativement aller chercher les résultats avec mysql_fetch_*.

Ensuite, tu n'as pas besoin de faire ce traitement au niveau de PHP, tu peux le faire en une seule requête en faisant

Code :

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

INSERT INTO table1 (...) SELECT ... FROM table2

Enfin, mysq_query(...) or die est une très mauvaise habitude que tu dois oublier tout de suite !

[PooSoft]

Enfin, mysq_query(...) or die est une très mauvaise habitude que tu dois oublier tout de suite !

Pourquoi donc ?

[Benjamin Delespierre]

Est-ce que le script doit se terminer brutalement si une seule requête échoue ?
Est-ce que l'utilisateur à quelque chose à faire des messages d'erreur de MySQL ?
Est-ce que les informations fournies dans l'erreur MySQL pourrait servir à un attaquant pour comprendre comment fonctionne ton application ?
N'est-il pas préférable de sortir proprement une page d'erreur avec le bon code HTTP (500 par exemple) et un message utilisateur suivi d'un lien ?
Est-ce que l'administrateur du site ne préférerait pas avoir une entrée de log avec des messages d'erreur clairs pour pouvoir corriger rapidement les erreurs ?

Mets toi deux minutes dans les bottes de l'utilisateur de ton service et tu comprendra pourquoi il vaut mieux gêrer correctement les erreurs. C'est pas une application C que tu écris, tu dois considérer que même en production, y'aura potentiellement des bugs et que donc, tu dois au maximum protéger l'utilisateur et le service.

[PooSoft]

D'accord merci pour ces détails, mais concernant votre première réponse, je dois laisser un switch ? je comprends pas vraiment

[Celira]

Tu peux laisser ton switch, mais tu dois extraire l'information dont tu as besoin de la ressource, de la même façon que tu l'as fait juste avant avec $row = mysql_fetch_assoc($resultat.

Pour nuancer le propos de Benjamin, je dirais que la structure mysql_query($req2) or die(mysql_error()); est bien utile en développement mais ne doit pas surtout pas rester dans le code final, de la même façon que tu ne vas pas laisser les echo $req2; qui parsèment ton code