fetch_array revoi trop de donnéeRésolu

Question

Salut,

J'ai un petit soucis dans ce petit script 
Il regarde les personnes qui se sont connect&#233; sur le forum. Et r&#233;agit en fonction de leur temps d'inactivit&#233; (envoi d'un mail, ou autre)
Ici, pour l'exemple, j'ai juste demand&#233; l'affichage &#224; l'arrache des pseudo des 'inactifs' depuis une semaine.
Et la ca bug. Aller soyons fou, disons que il n'y a qu'une personne qui correspond aux criteres. Donc normalement, l'echo me revoi juste une le pseudo.
Or la, il me renvoi le pseudo un paquet de fois. En faite, il le renvoi autant de fois qu'il y a d'entr&#233;e dans la table phpbb_user_group avec le groupe 6 (reformulation au cas ou, il renvoi autant de fois le pseudo qu'il y a de personne dans le groupe d'utilisateur que j'ai demand&#233;)
Si le script doit envoyer un mail, ca va vite spammer quelque boite mail tout ca 

A mon avi c'est juste la requete sql qui merdouille, mais au cas ou, je donne tout le code.

<?php
$db=mysql_connect($dbhost,$dblogi,$dbpass)or die(mysql_error());
mysql_select_db($dbbase,$db)or die(mysql_error());
$sql="SELECT *
FROM phpbb_users U, phpbb_user_group UG, select_team STWHERE U.user_id ST.user_id UG.user_id
AND UG.group_id = 6
ORDER BY `user_lastvisit` DESC";
$res=mysql_query($sql, $db) or die (mysql_error());
while ($donnees = mysql_fetch_array($res) )
{
     //La fait des variable inutile, mais je trouve ca plus simple &#224; manipuler ;)
        $avi = $donnees['avi'];  // On colle l'indice de passage dans une variable $avi
        $timeActuel = time(); //Donne le timestamp actuel
        $semaine = $timeActuel - (7 * 24 * 60 * 60); // Donne le timestamp d'il y a une semaine
        $timestamp = $donnees['user_lastvisit']; //Colle la derniere visite dans une variable sous forme du timestamp
        $semaines = $timeActuel - (2 * (7 * 24 * 60 * 60)); //Donne le timestamps d'il y a deux semaines
        $pseudo = $donnees['phpbb_users'];
        if ($timestamp > $semaines)
        {
                if ($avi != 1 AND $avi != 0)
                {
                        echo $pseudo; //on affiche tous les pseudo qui correspondent.
                }
        }
}
?>

malalam · Accepted Answer

Je ne sais pas comment t'expliquer.
Voici une cr&#233;ation de requ&#234;te sur 4 tables (prise de mon projet actuel) :  function getList($comtId null, $entId null, $contId = null, $artId = null)
        {
        $whereClauseArr = array ();
        if ($comtId !== null) {
            $whereClauseArr[] = ' AND (sk.comt_id = '.$comtId.')';
        }
        if ($entId !== null) {
                    $whereClauseArr[] = ' AND (sk.ent_id = '.$entId.')';
        }
        if ($contId !== null) {
                    $whereClauseArr[] = ' AND (sk.cont_id = '.$contId.')';
        }
        if ($artId !== null) {
                    $whereClauseArr[] = ' AND (sk.art_id = '.$artId.')';
        }
                $sQuery = 'SELECT
                            sk.sk_id, rg.rg_libelle, zone.zone_libelle, etg.etg_libelle
                        FROM
                            stocks sk, sk_rangees rg, sk_zones zone, sk_etages etg
                        WHERE
                            (sk.rg_id = rg.rg_id) AND
                            (sk.zone_id = zone.zone_id) AND
                            (sk.etg_id = etg.etg_id)
                        ';

        foreach ($whereClauseArr as $qry) {
            $sQuery .= $qry;
        }
        $sQuery .= ' ORDER BY
                             sk.rg_id, sk.zone_id, sk.etg_id';
                $aTmp = array();
                $this->oDB->query($sQuery);

                while($a = $this->oDB->fetch_assoc() ) {
                        $aTmp[$a['sk_id']] = $a['rg_libelle'].$a['zone_libelle'].$a['etg_libelle'];
                }
                return $aTmp;
        }

Je ne suis pas s&#251;r, en fait, que tes 3 tables soient bien structur&#233;es.

malalam · Answer

Hello,

ta requ&#234;te foire effectivement. Pas assez de crit&#232;res dans le WHERE.
Montre la structure de tes 3 tables.

bidulesfr · Answer

Pour la table phpbb_users et phpbb_user_groupe, c'est les tables d'origines
 du forum phpbb (donc pas des petite table dsl  )
Aller on commence par la 1ere, le gros molosse, les deux suivantes sont de taille raisonable.
phpbb_users
  `user_id` mediumint(8) NOT NULL auto_increment,
  `user_active` tinyint(1) default '1',
  `username` varchar(25) NOT NULL default '',
  `user_password` varchar(32) NOT NULL default '',
  `user_session_time` int(11) NOT NULL default '0',
  `user_session_page` smallint(5) NOT NULL default '0',
  `user_lastvisit` int(11) NOT NULL default '0',
  `user_regdate` int(11) NOT NULL default '0',
  `user_level` tinyint(4) default '0',
  `user_posts` mediumint(8) unsigned NOT NULL default '0',
  `user_timezone` float NOT NULL default '0',
  `user_style` tinyint(4) default NULL,
  `user_lang` varchar(255) default NULL,
  `user_dateformat` varchar(14) NOT NULL default 'd M Y H:i',
  `user_new_privmsg` smallint(5) unsigned NOT NULL default '0',
  `user_unread_privmsg` smallint(5) unsigned NOT NULL default '0',
  `user_last_privmsg` int(11) NOT NULL default '0',
  `user_emailtime` int(11) default NULL,
  `user_viewemail` tinyint(1) default NULL,
  `user_attachsig` tinyint(1) default NULL,
  `user_allowhtml` tinyint(1) default '1',
  `user_allowbbcode` tinyint(1) default '1',
  `user_allowsmile` tinyint(1) default '1',
  `user_allowavatar` tinyint(1) NOT NULL default '1',
  `user_allow_pm` tinyint(1) NOT NULL default '1',
  `user_allow_viewonline` tinyint(1) NOT NULL default '1',
  `user_notify` tinyint(1) NOT NULL default '1',
  `user_notify_pm` tinyint(1) NOT NULL default '1',
  `user_popup_pm` tinyint(1) NOT NULL default '0',
  `user_rank` int(11) default '0',
  `user_avatar` varchar(100) default NULL,
  `user_avatar_type` tinyint(4) NOT NULL default '0',
  `user_email` varchar(255) default NULL,
  `user_icq` varchar(15) default NULL,
  `user_website` varchar(100) default NULL,
  `user_from` varchar(100) default NULL,
  `user_sig` text,
  `user_sig_bbcode_uid` varchar(10) default NULL,
  `user_aim` varchar(255) default NULL,
  `user_yim` varchar(255) default NULL,
  `user_msnm` varchar(255) default NULL,
  `user_occ` varchar(100) default NULL,
  `user_interests` varchar(255) default NULL,
  `user_actkey` varchar(32) default NULL,
  `user_newpasswd` varchar(32) default NULL,
  `user_inactive_emls` tinyint(1) NOT NULL default '0',
  `user_inactive_last_eml` int(11) NOT NULL default '0',
  `user_tt` tinyint(1) NOT NULL default '0',
  `user_ttavatar` tinyint(1) NOT NULL default '0',
  `user_ttrang` tinyint(1) NOT NULL default '0',

phpbb_user_group
  `group_id` mediumint(8) NOT NULL default '0',
  `user_id` mediumint(8) NOT NULL default '0',
  `user_pending` tinyint(1) default NULL,

select_team
  `user_id` int(11) NOT NULL default '0',
  `avi` int(11) NOT NULL default '0'

malalam · Answer

Oula...

tu veux mon avis ? Essaye d&#233;j&#224; de la faire sous phpmyadmin, ta requ&#234;te.

bidulesfr · Answer

J'ai fais une petite erreur dans ce que j'ai dis, ce script la ne me retourne aucun enregistrement quelque soit les conditions. (C'&#233;tais avant que je bidouille que ca m'en revoy&#233; de trop. Quoi boulet ?)

Je retrouve plus la requette qui me retrouver tous les pseudo. Celle la m'en revoi en faite aucun. Malgres que normalement j'ai bien un champ qui correspond.

Sinon quand je fais mes requete sql sur phpmyadmin, ca me redonne le m&#234;me r&#233;sultat. Soit aucun enregistrement (MySQL n'a retourn&#233; aucun enregistrement). Soit plusieurs fois le meme champ (pour l'ancienne requete, je regarde pour la refaire &#233;galement)

bidulesfr · Answer

Salut,

H&#233;&#233;&#233; bin, moi qui trouve que mes requete commence &#224; devenir compliqu&#233; pour moi, quand je vois la tienne .... je me dis que mon cerveau commence &#224; manquer de m&#233;moire rame 

En faite je suis repartie de ton premier post ou tu parlais d'un manque d'info dans le WHERE, et en inspirant de ce que tu as fait dans ta requete  $sQuery, je suis arriv&#233; la dessus. Et visiblement ca marche.

$sql="SELECT  * 
FROM phpbb_users U, phpbb_user_group UG, select_team ST
WHERE (
U.user_id = ST.user_id
)
AND (
UG.user_id = ST.user_id
)
AND (
U.user_id = UG.user_id
)
AND UG.group_id =6
ORDER  BY  `user_lastvisit`  DESC"

J'ai fais que un essai, mais bon, on va dire que c'est bon hein :P
J'accepte ton dernier post, meme si la r&#233;ponce est en faite m&#233;lang&#233; un peu dans chaque post  et que je donne la solution final &#224; la fin (je vais quand meme pas accepter mon propre post hein, ca ferait tache.

Merci beaucoup du coup de main. Tu ma enlev&#233; une grosse &#233;pine du pied.

A++

malalam · Answer

Si si, j'accepte ta r&#233;ponse avec :-) C'est toi qui donne la r&#233;ponse
finale, donc ceux qui auront le m&#234;me genre de probl&#232;me, et regarderont
ce post, pourront avoir ainsi la m&#233;thode que tu as utilis&#233; :-)

On a le droit de valider ses propres post :-) Je le fais pour toi du coup.

Fetch_array revoi trop de donnée

7 réponses

Votre réponse

Discussions similaires