SELECT COUNT au lieu d'un while ?

Version imprimable

Bonjour :)

Je suis en train de coder un petit forum pour un ami en php.
Il m'a demandé un petit compteur des topics sur lesquels il y a eu des réponses depuis la dernière visite d'un membre. Pour se faire, je me suis inspité d'un tuto trouvé sur le net qui montrait une façon de procéder. Après quelques tests, j'ai un code fonctionnel.
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 <?php $query=$bdd->prepare(' SELECT forum_topic.topic_last_post, Mb.membre_pseudo AS post_createur, tv_id, tv_post_id, tv_poste, Ma.membre_pseudo FROM forum_topic LEFT JOIN general_membres Mb ON Mb.membre_id = forum_topic.topic_createur LEFT JOIN forum_post ON forum_topic.topic_last_post = forum_post.post_id LEFT JOIN general_membres Ma ON Ma.membre_id = forum_post.post_createur LEFT JOIN forum_topic_view ON forum_topic.topic_id = forum_topic_view.tv_topic_id AND forum_topic_view.tv_id = :id '); $query->bindValue(':id',$_SESSION['id'],PDO::PARAM_INT); $query->execute(); $nbrposts = 0; while ($data = $query->fetch()) { if ($data['tv_id'] == $_SESSION['id']) { if ($data['tv_poste'] == '0') { if ($data['tv_post_id'] != $data['topic_last_post']) { $nbrposts++; } } else { if ($data['tv_post_id'] != $data['topic_last_post']) { $nbrposts++; } } } else { $nbrposts++; } } $query->CloseCursor(); echo $nbrposts; ?>
Ma question est la suivante, le while n'est-il pas plus lourd qu'un SELECT COUNT ? Sachant que ce compteur sera rappelé via AJAX toutes les X secondes. Et dans le cas où ce serait préférable de n'utiliser qu'un SELECT COUNT, quelqu'un pourrait-il m'aiguiller, parce que je ne suis pas le meilleur en syntaxe SQL. Je me doute qu'il faut utiliser des WHERE mais à chaque tentative, j'échoue :/

Merci bien :D

14/03/2014, 19h05
sabotage

Montre nous ta meilleure tentative, on te dira ce que tu as mal fait.

Je n'ai plus mon bout de code (m'étant rabattu sur le if par dépit) mais ça devait donner quelque chose comme ça :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
SELECT COUNT(*) forum_topic.topic_last_post, Mb.membre_pseudo AS post_createur, tv_id, tv_post_id, tv_poste, Ma.membre_pseudo FROM forum_topic
LEFT JOIN general_membres Mb ON Mb.membre_id = forum_topic.topic_createur
LEFT JOIN forum_post ON forum_topic.topic_last_post = forum_post.post_id
LEFT JOIN general_membres Ma ON Ma.membre_id = forum_post.post_createur
LEFT JOIN forum_topic_view ON forum_topic.topic_id = forum_topic_view.tv_topic_id AND forum_topic_view.tv_id = :id
WHERE tv_id <> :id 
OR (tv_id = :id 
  AND (tv_poste = 0 
    AND (tv_post_id = topic_last_post)
  ) 
  OR tv_poste <> 0 
    AND (tv_post_id = topic_last_post)
)

Saignez pas trop des yeux, je suis débutant et absolument pas dans des études de ce type (je suis en médecine), j'apprends sur le fil par moi même donc soyez un peu indulgents ^^

15/03/2014, 22h55
sabotage

Je ne comprends ton cas $data['tv_id'] != $_SESSION['id'] puisque dans ta requête il y a une égalité entre les deux valeurs.

Tu parles du dernier else je présume (dans mon while) ? celui qui ajoute 1 à $nbrposts lorsque la condition

Code:

if ($data['tv_id'] == $_SESSION['id'])

n'est pas validée c'est ça ?
J'ajoute des commentaires pour que tu comprennes le code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
while ($data = $query->fetch()) {
  if ($data['tv_id'] == $_SESSION['id']) { // Si le membre $id a lu le topic...
    if ($data['tv_poste'] == '0') {  // Et que le membre n'a pas posté dans le topic...
      if ($data['tv_post_id'] != $data['topic_last_post']) {  // Mais qu'il y a un nouveau message posté par un autre
        $nbrposts++;  // Alors on ajoute 1 au compteur
      }
    }
    else {  // Et si le membre $id a posté dans le topic
      if ($data['tv_post_id'] != $data['topic_last_post']) {  // Et qu'il y a un nouveau message
        $nbrposts++;  // Alors, de même, on ajoute +1 au compteur
      }
    }
  }
  else {  //Enfin, s'il n'a même pas daigné lire le topic
    $nbrposts++;  // Aors, rebelotte, on ajoute +1 au conteur.
  }
}

Dans le SQL, ca reste flou, mais j'ai essayé de refaire le même schéma.

Déjà tu as deuxs scénarios identiques : que le membre ait posté ou non, tu fais la même chose.
Tu peux donc réduire d'une condition
Code:

1 2 3 4 5 6 7 while ($data = $query->fetch()) { if ($data['tv_id'] == $_SESSION['id'] && $data['tv_post_id'] != $data['topic_last_post'] ) { // Si le membre $id a lu le topic et qu'il y a un nouveau message posté par un autre $nbrposts++; } else { $nbrposts++; }
j'ai bon ?

Attention pour le passage à la requête préparée : tu ne peux pas utiliser plusieurs fois ":id" il faut donner un nom différent à chaque fois.

16/03/2014, 14h09
Alem180

Pour raccourcir la requête, j'y ai pensé aussi, mais à l'issue des tests, ce qui semble quand même logique et compactable (sans développer les deux scénarios donc), le code me sort une valeur erronée. Concrètement, il compte juste le nombre de topics..
Ici : while long : http://imgur.com/FDPD4xR
Ici : le while compacté : http://imgur.com/3oxwx0Z

Edit : le compteur qui retourne $nbrposts est fur fond bleu en haut à droite ;)

Bizarre, comme ça je ne vois pas.
Pour debuguer :

Code:

1
2
3
4
5
while ($data = $query->fetch()) {
    echo '************ session ' . $_SESSION['id'] . '<br/>';
    print_r ($data);
  .......
}

16/03/2014, 21h32
Alem180

Pour le debug voilà ce que ça donne :)

Citation:

************ session 1
Array ( [topic_last_post] => 4 [0] => 4 [post_createur] => Albartros98 [1] => Albartros98 [tv_id] => 1 [2] => 1 [tv_post_id] => 4 [3] => 4 [tv_poste] => 1 [4] => 1 [membre_pseudo] => Albartros98 [5] => Albartros98 )
************ session 1
Array ( [topic_last_post] => 18 [0] => 18 [post_createur] => test [1] => test [tv_id] => 1 [2] => 1 [tv_post_id] => 18 [3] => 18 [tv_poste] => 1 [4] => 1 [membre_pseudo] => Albartros98 [5] => Albartros98 )
************ session 1
Array ( [topic_last_post] => 7 [0] => 7 [post_createur] => Albartros98 [1] => Albartros98 [tv_id] => 1 [2] => 1 [tv_post_id] => 7 [3] => 7 [tv_poste] => 1 [4] => 1 [membre_pseudo] => test [5] => test )
************ session 1
Array ( [topic_last_post] => 8 [0] => 8 [post_createur] => Albartros98 [1] => Albartros98 [tv_id] => 1 [2] => 1 [tv_post_id] => 6 [3] => 6 [tv_poste] => 1 [4] => 1 [membre_pseudo] => test [5] => test )
************ session 1
Array ( [topic_last_post] => 23 [0] => 23 [post_createur] => test [1] => test [tv_id] => 1 [2] => 1 [tv_post_id] => 23 [3] => 23 [tv_poste] => 1 [4] => 1 [membre_pseudo] => test [5] => test )
************ session 1
Array ( [topic_last_post] => 24 [0] => 24 [post_createur] => test [1] => test [tv_id] => 1 [2] => 1 [tv_post_id] => 20 [3] => 20 [tv_poste] => 1 [4] => 1 [membre_pseudo] => test [5] => test )
************ session 1
Array ( [topic_last_post] => 12 [0] => 12 [post_createur] => test [1] => test [tv_id] => [2] => [tv_post_id] => [3] => [tv_poste] => [4] => [membre_pseudo] => test [5] => test )
************ session 1
Array ( [topic_last_post] => 14 [0] => 14 [post_createur] => test [1] => test [tv_id] => [2] => [tv_post_id] => [3] => [tv_poste] => [4] => [membre_pseudo] => test [5] => test )
************ session 1
Array ( [topic_last_post] => 15 [0] => 15 [post_createur] => test [1] => test [tv_id] => 1 [2] => 1 [tv_post_id] => 15 [3] => 15 [tv_poste] => 0 [4] => 0 [membre_pseudo] => test [5] => test )
************ session 1
Array ( [topic_last_post] => 21 [0] => 21 [post_createur] => test [1] => test [tv_id] => 1 [2] => 1 [tv_post_id] => 16 [3] => 16 [tv_poste] => 0 [4] => 0 [membre_pseudo] => test [5] => test )
************ session 1
Array ( [topic_last_post] => 22 [0] => 22 [post_createur] => test [1] => test [tv_id] => 1 [2] => 1 [tv_post_id] => 22 [3] => 22 [tv_poste] => 0 [4] => 0 [membre_pseudo] => test [5] => test )
16/03/2014, 21h43
sabotage

Pour la ligne 1,2, 3, 5, 9 et 11 $data['tv_post_id'] == $data['topic_last_post'], donc on ne les compte pas.
Il n'y a donc pas 11 mais bien 5.

Je sais bien mais le script me redonne bel et bien 11...
Chose étrange, pour tester, j'ai juste fait ceci :
Code:

1 2 3 4 5 while ($data = $query->fetch()) { if ($data['tv_post_id'] != $data['topic_last_post']) { // Qui équivaut au "Mais qu'il y a un nouveau message posté par un autre" mais sans vérifier si $id a lu le topic. $nbrposts++; } }
Et bien j'ai bien 5 (logique) en soi. Dois-je considérer que c'est ok dans ce cas ou est-ce que mon php a décidé d’interpréter les fichiers comme il le souhaitais ? :mouarf:
Dans ce cas, ma requète peut se résumer ainsi :
Code:

1 2 3 SELECT forum_topic.topic_last_post, tv_post_id FROM forum_topic LEFT JOIN forum_post ON forum_topic.topic_last_post = forum_post.post_id LEFT JOIN forum_topic_view ON forum_topic.topic_id = forum_topic_view.tv_topic_id AND forum_topic_view.tv_id = :id
Et tant que j'y suis à supposer que cette version fonctionne (j'ai essayé sur 3 comptes, ça marche bien..)
voilà une tentative échouée de requete SQL en COUNT, j'ai besois de cours de SQL je pense bien :roll:
Code:

1 2 3 SELECT COUNT(*) FROM forum_topic WHERE forum_topic.topic_last_post <> tv_post_id LEFT JOIN forum_post ON forum_topic.topic_last_post = forum_post.post_id LEFT JOIN forum_topic_view ON forum_topic.topic_id = forum_topic_view.tv_topic_id AND forum_topic_view.tv_id = :id

Il faudrait que tu dois plus clair, tu me disais que mon code ne fonctionnait pas, maintenant tu me dis que si.

Bref

Code:

1
2
3
4
SELECT COUNT(*) FROM forum_topic
LEFT JOIN forum_post ON forum_topic.topic_last_post = forum_post.post_id
LEFT JOIN forum_topic_view ON forum_topic.topic_id = forum_topic_view.tv_topic_id
WHERE forum_topic_view.tv_id = :id AND tv_post_id <> topic_last_post

16/03/2014, 22h40
Alem180

Je n'ai pas été clair dans ma dernière réponse, ton code ne fonctionnait bel et bien pas (ça se dit ?). J'ai enlevé la condition de vérification avec l'id du membre pour que ça marche.
Désolé pour la confusion, c'est ma faute :oops:

Je teste la requete de ce pas.

Bonjour, Après une phase de dépression, j'ai trouvé une solution à mon problème :)

Cependant j'ai dû apporter une modification à ton code pour qu'il marche, le voilà
Code:

1 2 3 4 SELECT COUNT(*) FROM forum_topic LEFT JOIN forum_post ON forum_topic.topic_last_post = forum_post.post_id LEFT JOIN forum_topic_view ON forum_topic.topic_id = forum_topic_view.tv_topic_id AND forum_topic_view.tv_id = :id WHERE forum_topic.topic_last_post <> tv_post_id OR tv_post_id IS NULL
J'attend une confirmation que c'est fonctionnel et je met résolu :)

Merci mille fois ;)