Optimisation de code

[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 :

--
-- 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)

<?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è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

[Keripy]

Salut

Il y a des tas de possibilités pour faire un code quelque soit le langage. Je dirais que si il fonctionne et que tu est capable de le débugger (quant il sera en ligne, avec le temps tu en trouvera surement ), c'est qu'il y a pas grand chose de plus à y faire.