Optimisation structure et requetes

[joboy84]

Bonjour

Je suis en train de faire un script d'envoi de newsletter.

Je souhaite envoyer les newsletter suivant les centres d'intérêts des membres.

Voici la structure de ma base actuelle pour les tests :

Code :

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
--
-- Structure de la table `CentresInterets`
--
 
CREATE TABLE IF NOT EXISTS `CentresInterets` (
  `id` int(4) NOT NULL auto_increment,
  `type` varchar(20) collate latin1_general_ci NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci AUTO_INCREMENT=7 ;
 
-- --------------------------------------------------------
 
--
-- Structure de la table `membres`
--
 
CREATE TABLE IF NOT EXISTS `membres` (
  `id` int(8) NOT NULL auto_increment,
  `email` varchar(40) collate latin1_general_ci NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci AUTO_INCREMENT=5 ;
 
-- --------------------------------------------------------
 
--
-- Structure de la table `MembresInterets`
--
 
CREATE TABLE IF NOT EXISTS `MembresInterets` (
  `idmembre` int(8) NOT NULL,
  `idinteret` int(4) NOT NULL,
  KEY `idmembre` (`idmembre`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci;
 
-- --------------------------------------------------------
 
--
-- Structure de la table `TempEnvoiMail`
--
 
CREATE TABLE IF NOT EXISTS `TempEnvoiMail` (
  `email` varchar(40) collate latin1_general_ci NOT NULL,
  UNIQUE KEY `Mail` (`email`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci;

Voici le code qui permet de traiter cette demande (j'ai pas ajouté la fonction mail ici pour ne pas alourdir le code)

Code :

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
 
<?php
include('connectionBase.php');
 
if (isset($_POST['mon_choix'])) 
{   
    $n = 0;
 
    // $i vaut 0, on compte le nombre de fois que mon_choix apparait, tant que $i est inferieur...
	for($i = 0, $c = count($_POST['mon_choix']); $i < $c; $i++) 
	{
		//On selectionne l id des membres ayant choisi le centre d interet coché
	   $test = mysql_query('SELECT idmembre FROM MembresInterets WHERE idinteret = "'.$_POST['mon_choix'][$i].'"') or die('Erreur'.mysql_error());   
 
		while($do = mysql_fetch_assoc($test))
		{
		//On récupère l'email des membres ayant selectionné le centre d'intérêt
		$mail = mysql_query('SELECT email FROM membres WHERE id = "'.$do['idmembre'].'"');
		$recmail = mysql_fetch_assoc($mail);
 
		//On insère les email dans une base temporaire afin d'etre sûr de ne pas envoyer les mails en doublon et
		// pour reprendre l'envoi si plantage ou restrictions il y a (nbre envoi mail limité par heure..)
		$BaseTemp = mysql_query('INSERT IGNORE INTO TempEnvoiMail (email) VALUES ("'.$recmail['email'].'")') or die ('Erreur : '.mysql_error());
 
		$n++;
		} 
    }
	echo'Il y a '.$n.' personnes qui correspondent aux crit&egrave;res';
}
 
#############################
# On affiche le formulaire  #
#############################
 
//On récupère les centres d'intérêts dans la base de données
$centre = mysql_query('SELECT id, type FROM CentresInterets') or die('erreur:'.mysql_error());
 
echo'<form method="POST" action="pagetest.php">';
     while($data = mysql_fetch_assoc($centre))
	 {
    echo'<input type="checkbox" name="mon_choix[]" value="'.$data['id'].'"/>'.$data['type'].'<br>';		 
	 }
 
 echo'<input type="submit" value="OK">
</form>';
?>

Tout fonctionne, les mails sont récupérés et insérés dans la table.

Je souhaite savoir s'il est possible d'optimiser mon code et si mon approche est la bonne (structure base..)

Merci

[sabotage]

Code :

for($i = 0, $c = count($_POST['mon_choix']); $i < $c; $i++)

Utilise une boucle foreach() déjà.

Sinon tu peux faire une seule requête :

Code :

1
2
3
$sql = 'SELECT email FROM MembresInterets 
JOIN membres ON membres.id=MembresInterets.idmembre 
WHERE idinteret = '.intval($id)