Intercepter un require_onceRésolu

Question

Hello, je bloque rarement au point de poser une question, mais là...je ne parviens pas à mes fins et j'ai épuisé mes idées ;-) La problématique est relativement simple : je travaille sur un bug connu de mon script de documentation. Le principe pour documenter des classes est simple : le fichier dans lequel se trouve la classe est choisi, et je fais un require_once dessus. Jusque là, tout va bien. Mais comme mon script remonte les classes héritées, j'ai dû faire un require_once sur toutes les classes uploadées, car, sans règle de nommage, je n'ai pas de moyen de savoir dans quel fichier se trouve la classe héritée. Ca marche très bien (tant qu'on a pas 200 fichiers uploadés, évidemment, mais ça, c'est un autre problème) quand le fichier de la classe héritée se trouve "au-dessus", ou "avant" le fichier de la classe héritante dans le répertoire. Mais si elle est après, le require_once du fichier de la classe héritante me sort évidemment une erreur, car il ne trouve pas la aclasse héritée (normal, elle n'a pas encore été incluse). J'ai donc fait plusieurs tests : @require_once ne plante plus, mais bloque par contre (évidemment, pas de message d'erreur, mais ça bloque quand même au require défaillant). J'ai alors testé ça : $obj) { if ($obj !== '.' && $obj !== '..') { try { require_once ('classes/'.$obj); $aFlag[$clef] = 1; } catch (Exception $e) { continue; } } } } ?> Le but étanht de flagger les fichiers inclus, et, si on rencontre un problème lors d'une inclusion, on saute et on passe aau suivant. Et on reboucle tant que tous les fichiers n'ont pas été flaggés à 1. Mais rien à faire, le require_once plante, même dans le bloc du try {}. J'ai ensuite utilisé la bufferisation de sortie : $obj) { if ($obj !== '.' && $obj !== '..') { try { ob_start (); require_once ('classes/'.$obj); $aFlag[$clef] = 1; $contents .= ob_get_contents (); ob_end_clean (); } catch (Exception $e) { continue; } } } } echo $contents; ?> Mais pareil, le require_once plante lamentablement. Idem avec un @, je précise. Je me suis alors lancé dans un $contents .= get_file_contents (...) à la place du require_once (), et un eval à la fin (voui je sais...mlais j'étais désespéré) Mais là, ça tourne carrément trop longtemps au niveau du file_get_contents (je ne parle donc même pas du eval...). Et là, je n'ai plus d'idée... Alors si quelqu'un en avait une!! ;-) Merci :-)

J_G · Accepted Answer

Salut malalam.

Probl&#232;me interessant... Je vais essayer de te donner une piste pour te d&#233;coincer (je dis bien "essayer").

Je pense que l'id&#233;e du scandir est bonne... mais pas assez exploit&#233;e.

C'est &#224; dire, au lieu de charger (require_once) tous les fichiers contenant des classes, je te propose de plut&#244;t parser ces fichiers.

Par exemple :
* Tu ouvre le r&#233;pertoire du projet &#224; documenter

* Tu parcours r&#233;curssivement le projet, soit tous les fichiers de type texte =>
preg_match('@^text/@',mime_content_type($file)); 

* enfin tu parses chacun de ces fichiers pour trouver des d&#233;finition de classes 
$content = file_get_contents($file);
if( preg_match_all('@class\s+([\w\d_]+)\s+(extends\s+([\w\d_]+)\s+)?{@im',$content,$res) ) {
    /* ici, tu retrouve les classes et leur h&#233;ritage dans $res */
    $file; // le fichier qui contient des d&#233;finitions de classes
    $res[1]; // les noms des classes d&#233;finies
    $res[3]; // les classe parentes de ces derni&#232;res
}


* Avec ces infos, tu te construis un tableau, un arbre, des objets, ... tout ce que tu veux pour ensuite pouvoir le parcourir et retrouver facilement les fichier &#224; inclure pour tel ou tel h&#233;ritage.

Voil&#224;, je ne pense pas que cette passe de parsing soit trop longue et 'esp&#232;re que &#231;a pourra t'aider...

A+ et bonne chance

malalam · Answer

Hello J_G,

j'avais pens&#233; au parsing en effet, mais je pensais que ce serait trop long et fastidieux...ceci dit, vue que je me retrouve bien coinc&#233; l&#224;...et que ce que tu me montres est plus abouti que ce &#224; quoi j'avais pens&#233;, je vais tenter le coup.
Ca permettrait en plus de r&#233;soudre le probl&#232;me des require inutiles.

Donc, je teste &#231;a de suite :-) Merci J_G!

J_G · Answer

De rien... surtout que je te laisse le plus dur &#224; faire :

La structure du tableau/arbre &#224; parcourir et les fonction de parcours. Ces exercices algorithmiques sont mes pire casses-t&#234;tes.

A+

malalam · Answer

Je suis dessus lol...je r&#233;fl&#234;chis &#224; la meilleure structure pour avoir un arbre facile &#224; remonter. Et effecivement, ce sont les pires casse-t&#234;tes :-( Mais c'est rigolo!
J'ai ajout&#233; implements dans ton pattern au fait, comme choix entre exteneds et implements (pour les interfaces).
Et sinon, ton pattern marche &#224; merveille :-)

malalam · Answer

Bon vl&#224; le r&#233;sultat qui fonctionne &#224; merveille :

<?php
if  (isset ($_POST['documenter']) && $_POST['documenter'] === $oloc -> getMsg ('gui', 'app_document') && !empty ($_POST['objet']) && !empty ($_POST['nom'])) {
 $aDocs = scandir ('classes');
 foreach ($aDocs as $clef => $obj) {
  if ($obj !== '.' && $obj !== '..') {
   $content = file_get_contents ('classes/'.$obj);
   if( preg_match_all([mailto:'@class\s+([\w\d_]+)\s+((extends|implements)\s+([\w\d_]+)\s+)?{@im',$content,$res '@class\s+([\w\d_]+)\s+((extends|implements)\s+([\w\d_]+)\s+)?{@im',$content,$res])) {
    $aHierarchy[$res[1][0]]['parent'] = $res[4][0];
    $aHierarchy[$res[1][0]]['file'] = $obj;
       }
  }
 }
 $className = $_POST['nom'];
 $aIncs[] = $aHierarchy[$className]['file'];
 while (!empty ($aHierarchy[$className]['parent'])) {
  $className = $aHierarchy[$className]['parent'];
  $aIncs[] = $aHierarchy[$className]['file'];
 }
 $aIncs = array_reverse ($aIncs);
 foreach ($aIncs as $files) {
  require_once ('classes/'.$files);
 }
}
?>

Meci &#224; J_G, et &#224; Anthomicro :-)

malalam · Answer

Bon, l'acceptation ne marche pour le moment pas, alors j'accepterai la r&#233;ponse plus tard :-)

J_G · Answer

Bravo...

malalam · Answer

Bah j'ai pas de m&#233;rite lol, tu m'as m&#226;ch&#233; tout le travail :-)
Antho m'a aid&#233; &#224; comprendre pq mon pattern marchait mal (en fait, il marchait tr&#232;s bien, j'avais juste oubli&#233; que les offset bougeaient...).
Quant &#224; la hi&#233;rarchie, &#231;a a finalement &#233;t&#233; tr&#232;s simple, comme tu peux le voir :-)

J_G · Answer

Oui, c'est pour &#231;a que je dis "bravo"... je ne le voyais pas si simple.

Par contre, j'ai l'impression que tu ne prends pas en compte le fait qu'il puisse y avoir plusieur d&#233;finition de classe dans un m&#234;me fichier. Par exemple parent + enfant.

C'est pour &#231;a que je t'ai indiqu&#233; le preg_match_all

Voilou.
A+

malalam · Answer

C'est exact lol. Mais c'est pas un probl&#232;me. Je dois juste modifier la fa&#231;on de construire le tableau hi&#233;rarchique c'est tout. Je vais le faire de ce pas d'ailleurs ;-)

malalam · Answer

Voil&#224;, le code complet :

if  (isset ($_POST['documenter']) && $_POST['documenter'] === $oloc -> getMsg ('gui', 'app_document') && !empty ($_POST['objet']) && !empty ($_POST['nom'])) {
 $aDocs = scandir ('classes');
 foreach ($aDocs as $clef => $obj) {
  if ($obj !== '.' && $obj !== '..') {
   $content = file_get_contents ('classes/'.$obj);
   if( preg_match_all([mailto:'@class\s+([\w\d_]+)\s+((extends|implements)\s+([\w\d_]+)\s+)?{@im',$content,$res '@class\s+([\w\d_]+)\s+((extends|implements)\s+([\w\d_]+)\s+)?{@im',$content,$res])) {
    foreach ($res[1] as $clef => $val) {
     $aHierarchy[$res[1][$clef]]['parent'] = $res[4][$clef];
     $aHierarchy[$res[1][$clef]]['file'] = $obj;
    }
       }
  }
 }
 $className = $_POST['nom'];
 $aIncs[] = $aHierarchy[$className]['file'];
 while (!empty ($aHierarchy[$className]['parent'])) {
  $className = $aHierarchy[$className]['parent'];
  $aIncs[] = $aHierarchy[$className]['file'];
 }
 $aIncs = array_reverse ($aIncs);
 foreach ($aIncs as $files) {
  require_once ('classes/'.$files);
 }


 try {
  $doc = new odocclass ($_POST['nom']);
  $sError = $oloc -> getMsg ('gui', 'app_document_created').$_POST['objet'];
  //header ('Location: docs/'.$_POST['nom'].'/'.$_POST['nom'].'.html');
 } catch (Exception $e) {
  $sError = $oloc -> getMsg ('errors', 'failed_instance');
 }
}

J_G · Answer

Ben super... c'&#233;tait pas grand chose :)

Et niveau timing : correct ?

malalam · Answer

Tr&#232;s, aucun probl&#232;me contrairement &#224; ce que je pensais.
Tu avais raison :-) Je n'ai pas fait de bench pour comparer, mais disons qu'&#224; l'oeuil nu, on ne voit aucune diff&#233;rence avec avant. De plus, avec ton syst&#232;me, les pages vont &#234;tre bcp plus l&#233;g&#232;res m&#234;me avec bcp de classes upload&#233;es, puisqu'on inclut plus que les classes &#224; analyser :-) Donc c'est bien mieux qu'avant ! :-)

Je pourrais remplacer 
     $aHierarchy[$res[1][$clef]]['parent'] = $res[4][$clef];
     $aHierarchy[$res[1][$clef]]['file'] = $obj;

par

     $aHierarchy[$val]['parent'] = $res[4][$clef];
     $aHierarchy[$val]['file'] = $obj;

mais bon, je ne pense pas que &#231;a change grand chose...

J_G · Answer

Ouai non... Ca change rien. 

Le pr&#233;processeur va un peu plus gal&#233;rer dans la premi&#232;re version. Mais une fois le script pr&#233;-mach&#233; (c'est &#224; dire uniquement la premi&#232;re fois qu'il est ex&#233;cut&#233;) c'est quasiment pareil.

Note : Il y'a un risque de bug avec la RegExp...
class salut{/*...*/} ne colle pas et pourtane elle est bien d&#233;clar&#233;e ! (=> il n'y a pas d'espace entre le nom et l'accolade)
Correction propos&#233;e :
[http:// class\s+([\w\d_]+)(\s+(extends|implements)\s+([\w\d_]+))?\s*{]

A+

malalam · Answer

Oui j'y ai pens&#233;...puis je n'y ai plus pens&#233; ;-)
Je vais changer, effectivement. Faut faire gaffe &#224; &#231;a, fa&#231;on d'&#233;crire un code. Moi, je mets des espaces partout, d'autres pas (toi, par exemple, lol : fonction(bla) et fonction (bla) ou fonction( bla ) ou fonction ( bla )...c'est chiant lol).

malalam · Answer

Voil&#224;, c'est fait, et apparemment, &#231;a marche :-)
Merci encore.
Mon application commence s&#233;rieusement &#224; prendre forme ;-)
je suis maintenant juste un peu &#224; court d'id&#233;e pour la faire encore &#233;voluer lol ;-)

J_G · Answer

Elle fait quoi ton application ?

Enfin, je veux dire... Elle documente que les classses ou elle fait des diagrammes d'h&#233;ritage et de la doc sur les fonctions ?
Est-ce qu'elle &#233;num&#232;re aussi les impl&#233;mentations des classes/fonctions ?

(Et si elle fait tout &#231;a... je peux l'avoir ?)

FhX · Answer

J'ai m&#234;me pas eu le temps de me pencher sur le probl&#232;me que c'est d&#233;ja r&#233;solu...

Rahhh, chui encore malade 4 jours et y se passe un tas de trucs !!

J_G · Answer

Faut pas &#234;tre malalade avec Internet ;)

malalam · Answer

Ben heu sers-toi lol :
http://www.phpcs.com/codes/PHP5-CLASSE-DOCUMENTATION-CLASSES-FONCTIONS_35950.aspx

Elle documente automatiquement les classes (remonte la hi&#233;rarchie s'il y en a une, comme tu l'as compris lol), ainsi que les fichiers de fonctions.

Bref, tu uploades tes fichiers de classes, tes fichiers de fonctions.
Puis tu choisis les fichiers que tu veux documenter, et hoip c'est fait : elle cr&#233;e des fichiers html pourt la doc.
Enfin, je te laisse regarder , j'ai mis dans le zip des exemples de documentation cr&#233;&#233;es. C'est tr&#232;s facile &#224; utiliser.

Pour le moment, elle ne fait pas de diagramme mais c'est une super id&#233;e lol!
J'ai m&#234;me une id&#233;e pour ce propos...g&#233;n&#233;rer &#231;a par package (pour l'instant, c'est g&#233;n&#233;r&#233; par classe/fichier de fonctions), cxela permattrait de browser un package plus facilement. Et de cr&#233;er un diagramme du package justement.

Et comme &#231;a, si tu testes, tu pourras me donner des id&#233;es :-) Et des suggestions d'am&#233;liorations. C'est un projet qui me plait et sur lequel j'aime bien passer du temps (j'ai commenc&#233; y a 5-6 jours je crois, pdt mes pauses au boulot lol).

malalam · Answer

Lol, Fhx...bon r&#233;tablissement ;-) Suis s&#251;r que &#231;a va vite aller mieux!

FhX · Answer

Ca fait 3 fois en 2 mois... ca commence &#224; faire beaucoup chez moi je trouve :)
Jtrouve m&#234;me plus le temps de coder, jpasse mon temps &#224; dormir en ce moment.

C'est triste, je me serais bien pench&#233; sur ton projet vite fait, mais d&#232;s que je lis plus de 5 lignes de codes j'ai un mal de crane qui s'installe :D

On verra un peu plus tard si ca va mieux :)

malalam · Answer

Ouais et puis comme &#231;a, j'ai le temps de le faire avancer lol.

T'es victime de la serial-gastro qui s&#233;vit en ce moment...?
Y en a plein ici! J'y &#233;chappe pour le moment...je croise les doigts lol.

J_G · Answer

Ooops... J'ai juste oubli&#233; que je n'utilisai pas encore PHP5...

Parse error ! bien s&#251;r...
Domage pour moi

malalam · Answer

Effectivement, lol.
Et tu ne peux pas mettre &#224; jour cette source pour php4...l'API de r&#233;flexion utilis&#233; est sp&#233;cifique PHP5.

Faut t'y mettre! Une tite install de WAMP, tranquille, et le tour est jou&#233; pour tester un peu PHP5 :-)

J_G · Answer

C'est pas le probl&#232;me de faire une installation...

Mais le serveur sur lequel je d&#233;veloppe est en PHP 4.23 et MySQL 3.23
(Il aime les .23 on dirait)

Pour PHP, c'est pas dramatique... Mais pour MySQL c'est un drame !!!

Enfin bref, mon ordi est un outils de travail. Donc je m'aligne sur les technologies utilis&#233;es.

Voil&#224; pour la petite histoire.

FhX · Answer

Nan... rhume je sais pas trop quoi. Pas bien compris ce qu'on m'a racont&#233; lol !

Bref, un bourrage au m&#233;doc 24h/24... une lutte quotidienne tous les soirs pour aller dormir =) Jdois me coucher il est 5h30/6h du mat' c'est super g&#233;nial :)

Mais bon, voui fait avancer ton truc, jm'y pencherais dans pas longtemps promi !!

malalam · Answer

Bon ben tant pis J_G, tu ne profiteras pas de mon fabuleux code ;-)
Perso en tous cas, je me fais pas chier parce que mon pc est juste un outil de d&#233;v "test", pas de d&#233;v de prod...je fais les d&#233;v de prod directement sur nos serveurs en g&#233;n&#233;ral (ou je copie mon d&#233;v de test lol, &#231;a arrive). Du coup dessus j'ai wamp avec php5 et tout le toutim (mysql 5 et des poussi&#232;res aussi), et une autre install avec php4.3 et mysql 3.23 aussi (directement sous Apache, sans wamp et consors quoi). ET un serveur sql server 2000 lol.
Bon faut faire gaffe &#224; pas tout ouvrir en m&#234;me temps...mais &#231;a tourne ;-)

malalam · Answer

FhX => tous mes voeux de bon r&#233;tablissement...parce que t'as pas l'air loin d'y passer l&#224; lol.

FhX · Answer

Sql serveur 2000 !! Wouahou :D

Ca doit bien donn&#233; ca j'imagine !

malalam · Answer

Vou&#233;...disons que...vou&#233;. Lol.
C'est costaud, et stable, &#231;a c'est s&#251;r. Rapide aussi (je n'ai pas compar&#233; avec mysql mais bon, c'est une bdd rapide).
Apr&#232;s, y a plein de petits plus sympas, comme tjrs avec Microsoft, MAIS...des plus qu'il ne faut pas utiliser si on se dit qu'ujn jour, pt&#234;te, on voudra migrer la base suir une autre plateforme (remarque, c'est pareil avec mysql, mais y en a moins).
Les outils fournis sont tr&#232;s bien : Enterprise Manager est sympa mais bon, je pr&#233;f&#232;re phpmyadmin lol...par contre, l'Analyseur de requ&#234;tes lui est g&#233;nial! Les &#233;quivalents pour mysql (Query Browser) sont loin, tr&#232;s loin derri&#232;re!

Mais bon y a des trucs vraiment chiants : le format de dates est p&#233;nible (surtout en travaillant avec php en fait). Le fait qu'un link ne renvoie pas une ressource, mais un identifiant de ressource (tjrs en php donc), faut le savoir sous peine de se prendre m&#233;chamment la t&#234;te (&#231;a a &#233;t&#233; mon cas avec ma classe d'abstraction de bdd lol).
Y a pas de num-rows, faut passer par une requ&#234;te : SELECT @@IDENTITY. Ca marche bien, mais faut pas perdre le lien pr&#233;c&#233;dent, sinon...c'est la merde!
Mais le pire, je crois, c'est pour &#233;chapper les apostrophes (par exemple) : il faut les doubler! le \ ne marche pas, donc addslashes est totalement inefficace...et &#231;a, c'est quand m&#234;me une grosse daube de la part de microsoft : \ est quand m&#234;me LE caract&#232;re d'&#233;chappement connu et reconnu par la plupart des langages. C'est lourd lol.
Y a aussi la clause TOP, qui est franchement moisie. On regrette le LIMIT propri&#233;taire de mysql finalement. Pour faire une pagination, du coup, c'est un peu gal&#232;re...on peut passer par la d&#233;claration d'une variable, etc...mais bon.
En fait on peut faire &#233;norm&#233;ment de choses juste par les requ&#234;tes. C'est un avantage, ET un d&#233;savantage parce que &#231;a rend l'appli propri&#233;taire. On est chez Microsoft, faut pas l'oublier... ;-)

FhX · Answer

Voui, le serveur SQL parfait n'existe pas :)

Y'a toujours du bon et du mauvais partout.
Donc ca y est, tu te met avec SQL Server... Donc dans pas longtemps tu passes &#224; ASP ?? ;) et apr&#232;s &#224; .Net ?? ;) ;) ;)

malalam · Answer

Ca va pas nan?? lol

Nan nan, c'est juste que c'est le serveur bdd qui a &#233;t&#233; choisi (avant mon arriv&#233;e), par ma bo&#238;te pour le d&#233;veloppement d'un de nos gros projets.
Mais j'ai travaill&#233; d&#233;j&#224; avec sql server, jusqu'en d&#233;cembre : avec sql server donc, et le framework XMLRad (langage serveur : Delphi, + xml, + xsl, + javascript).

Intercepter un require_once

33 réponses

Votre réponse

Discussions similaires