Programmation PHP : trouver le bug ! @ Prise2Tete

EfCeBa · #1

Comme vous le savez Prise2Tete a été désactivé par son hébergeur, OVH pour ne pas le citer à cause d'une surcharge serveur causée par un script : celui du flux RSS. J'ai passé ma soirée d'hier dessus à chercher la faille qui conduisait à ce problème, mais comme je n'ai rien trouvé, je m'en remets à vous si vous pouviez faire en sorte que je puisse réactiver le flux RSS sans risquer un blocage définitif...

J'explique le fonctionnement de la page PHP :

Une connexion BDD,
Une requete SQL pour récupérer les derniers sujets du forum.
Une boucle qui génère le flux au format XML

Voici le code :
http://www.prise2tete.fr/upload/EfCeBa-rss.phps

Vous avez le droit de faire toutes les modifications et remarques que vous voulez, je voudrais savoir dans quels cas il peut y avoir un bug.
Merci d'avance.

Taupine · #2

C'est la question subsidiaire de l'énigme 44

Limenthia · #3

Je pense pas que l'érreur vienne de la.

scarta · #4

J'aimerai bien t'aider, mais à mon avis c'est justement dans le morceau tronqué qu'il faut chercher (ie. la query SQL, ou à la rigueur ses résultats).
Sinon, à quoi sert le 3eme parametre de cut_str ?
Enfin, cette dernière est codée de manière beaucoup trop compliquée à mon goût: split( wordrap( ... ) ) devrait le faire d'un coup
Edit: ou même un split tout seul, puis un while en enlevant le dernier élèment jusquà être inférieur à length. Quoi qu'il en soit, reconstruire ton Array constant à chaque fois entraine l'allocation d'un tableau en mémoire, c'est assez gourmand en plus d'être inutile: instancie le une fois pour toute.

EfCeBa · #5

La requete ne pose pas de problème car c'est la même qu'il y a sur la page d'accueil car elle n'est pas incriminée.

Le troisième paramètre est superflu, le l'utilisais pour mettre un lien "lire la suite" comme sur la page d'accueil et donc j'avais besoin de l'id. Je ne m'étais pas apercu qu'il était resté dans ce script.

EfCeBa · #6

Je viens de regarder la requête, elle s'exécute en 0.02 secondes, elle est très simple, il n'y a pas de group by.

titane.86 · #7

scarta a écrit:
J'aimerai bien t'aider, mais à mon avis c'est justement dans le morceau tronqué qu'il faut chercher (ie. la query SQL, ou à la rigueur ses résultats).
Sinon, à quoi sert le 3eme parametre de cut_str ?
Enfin, cette dernière est codée de manière beaucoup trop compliquée à mon goût: split( wordrap( ... ) ) devrait le faire d'un coup
Edit: ou même un split tout seul, puis un while en enlevant le dernier élèment jusquà être inférieur à length. Quoi qu'il en soit, reconstruire ton Array constant à chaque fois entraine l'allocation d'un tableau en mémoire, c'est assez gourmand en plus d'être inutile: instancie le une fois pour toute.

Suis-je la seule à n'avoir rien compris

EfCeBa · #8

Code:

function couperalafindunmot($phrase, $longueurmaxi) {
 if (strlen($phrase) <= $longueurmaxi) return $phrase;
 $phrase = substr($phrase,0,$longueurmaxi);
 $coupe = strrpos($phrase," ");
 if ($coupe === false) return $phrase;
 else return substr($phrase,0,$coupe);
}

scarta · #9

Exact c'est strrpos qu'il faut utiliser, je cherchais un truc du genre lastindexof dans mon glossaire php, et je ne l'avais pas trouvé du coup (il faudrait normaliser les noms de ces genres de fonctions) mais c'est clairement ce qu'il y a de mieux.

D'autant plus que suivant les valeurs de $length (ok là il fait 200 mais bon) ton algo peut planter.

Je pensais aussi à ce qu'il pourrait se passer si ton dernier mot faisait plus de 15 caractères: ok tu va me dire un mot de plus de 15 lettres ça traîne pas les rues, n'empêche que dans le cadre d'une énigme cryptée, c'est envisageable. Ça ne fait pas planter l'algo, ça le fait juste tourner pour rien et un substring = une nouvelle instance de chaîne. Ensuite, fais le compte: un chaîne de n chars (2000 par exemple), puis 17 chaînes de 200 chars, ça fait un bon paquet alloué et je ne pense pas le plus grand bien du "garbage collector" de php. Autrement dit, à éviter.

Enfin, une dernière piste à explorer: essaye de faire toute ta cuisine sur les chaînes (formatages, remplacements, troncature, ...) directement au niveau du select (même MySql propose des fonctions pour ça, on est loin du SQL analytique d'Oracle mais bon c'est suffisant).

En esperant avoir été utile.

scarta · #10

Autre chose: j'ai retrouvé ce très bon article sur le garbage collector php:
http://hudzilla.org/phpwiki/index.php?t … collection

Et ça donne en plus l'idée de balancer quelques traces memory_get_usage() : si tu as une surcharge, au moins tu sauras où précisement.

EfCeBa · #11

On a toujours besoin d'un autre regard pour avancer, clairement plus le site grossi plus mes codes deviennent obscurs, et il ne peut faire plus de bien qu'une bonne relecture.
J vais récupérer le fichier exact qui faisait tout planter et je montrerai le nouveau et je remettrai ca en ligne.

En attendant, je vais faire quelques test avec des caractères aléatoires pour voir si tout fonctionne...

EfCeBa · #12

Je viens d'éditer le premier post, c'est exactement le fichier qui posait problème, c'est le fichier tel qu'il était sur le serveur lors du second blocage.

Il y a un bug évident sur la fonction cut_str qui occasionne une boucle infinie si le message ne contient aucun espace. J'ai testé sur un serveur, une boucle comme celle ci occasionne simplement une erreur 500 sur la page mais pas de surcharge.

kduke · #13

Tu pourrais éviter la boucle while (éventuellement infinie) de cut_str en utilisant la fonction strrpos qui te retourne la position d'un caractère dans une chaine en partant de la fin (r=reverse).
les deux dernières lignes deviennent une seule:
return substr($s, 0, strrpos($s, " "));

Une petite optimisation devrait même être possible en évitant la variable intermédiaire $s (et donc les éventuelles fuites mémoire):
return substr($str, 0, strrpos(substr($str, 0, $length+1), " "));

..mais ça devient beaucoup moins lisible d'un coup :-)

En farfouillant dans la doc PHP, je vois qu'un paramètre supplémentaire offset à strrpos a été ajouté en PHP 5 qui permet de dire où commencer la recherche. Dans ce cas, le substr devient inutile.

kduke · #14

pfff.. j'avais lu de travers ton code de hier soir, et même la réponse de scarta d'hier soir...désolé pour le bruit inutile
il faut que je me réveille, ça va bientôt être l'heure d'aller manger

EfCeBa · #15

OK merci kduke pour tes précisions quand meme.

Je viens de trouver un problème avec la fonction

Code:

preg_replace("/\[([^[])*]/", "", $txt);

Celle ci entraine des erreurs de type : Action annulée sous IE et Connexion interrompue (La connexion avec le serveur a été réinitialisée pendant le chargement de la page.) sous Firefox.

Je vais essayer de trouver une autre expression

EfCeBa · #16

Alors il semblerait que le problème vienne des parenthèses

Code:

preg_replace("/\[[^[]*]/", "", $txt);

Là il n'y a plus aucun souci, si quelqu'un peut m'expliquer...

EfCeBa · #17

Bon, si personne ne voit de problème, je mets cette nouvelle version en ligne, et si il y a encore des surcharges, je ne sais plus quoi faire...

RSS v2 :
http://www.prise2tete.fr/upload/EfCeBa-rss-v2.phps

Arcounette · #18

Je suppose qu'il est un peu tard pour répondre mais c'est comme une énigme pour moi donc : tes parenthèses placées comme dans l'avant-dernier post ferment "précocément" ton instruction donc ça ne marche pas, tu dois les indiquer comme "caractères" et les "échapper" pour que ton langage les reconnaissent comme telles et non comme élément du code.

chevi123 · #19

je me demande comment vous pouvez comprendre ce genre de trucs franchement moi je vois rien de NORMAL

Forum dédié aux énigmes et à toutes formes de jeux de logique.	Déconnexion
Tu n'es pas identifié sur Prise2tete : s'identifier. Accueil Forum	[+]

#1 - 29-07-2008 20:52:02

Programmatino PHP : trouver le bug !

#0 Pub

#2 - 29-07-2008 20:54:49