AFFICHER GÉNÉRIQUEMENT PLUSIEURS FICHIERS DANS IFRAME AVEC 2 FICHIERS PHP

Commentaire de coucou747 le 04/06/2005 11:32:00 administrateur CS

c'est du html...

Commentaire de originalcompo le 04/06/2005 12:09:40

euh, pardon, je crois que j'ai compris, je corrige ca...

Commentaire de fg85 le 04/06/2005 12:47:26

Il faudrai que tu fasse un  file_exists (http://fr.php.net/manual/fr/function.file-exists.php) même si ce script n'a pas besoin d'être protégé.
De plus une variable qui n'est pas traité avant d'être inséré est très dangereuse  (ex : $_GET["source"])

----------------------
http://fg.logiciel.free.fr

Commentaire de originalcompo le 04/06/2005 12:59:06

Merci FG85 pour l'histoire de "la variable non traité", je vais me renseigner pour voir de quoi il s'agit... (quand au 1er point, j'avoue que je n'avais pas pour but de faire un programme qui prévoit toutes les erreurs possibles et imaginables, mais le problème de sécurité, ca m'intrigue)

Commentaire de originalcompo le 04/06/2005 15:47:14

But de la source:
En fait, sur mon site je propose des composants Delphi. Certains meritent explications, d'autre pas.
Pour ces derniers, j'ai décidé d'ouvrir une fiche toute simple qui propose le code dans un iframe.

Mais j'avais pas envie de générer 10 fichiers html identiques avec juste le "src" du "iframe" qui change.

J'y suis arrivé, mais apparemment coucou47 trouve que y'a des trucs à corriger, et soit je suis trop inculte en php pour comprendre les raisons, soit ses commentaires sont trop vagues...

Donc mon but est :

Faire une page web qui permet d'afficher un document à choisir parmis une liste prédéfinie de documents (txt, ou ce que tu veux).

Dans l'exemple tel que je l'ai programmé, il y a un bouton par fichier sélectionnable.
On clique, ca charge!

Si tu peut l'exécuter, c'est évident...

Je demandais conseil parceque faire un php qui s'appelle lui même (via le formulaire et $_SERVER["PHP_SELF"]) en appelant un deuxième php dans le frame, je trouvais ca un peu tordu et je me demandais si y'avait pas plus simple, mais vu mon niveau, je suis hyper satisfait par le résultat...
;-)

Commentaire de Anthomicro le 05/06/2005 19:24:04

Ouais déjà, et faut changer d'antipub aussi (lol)

Commentaire de Anthomicro le 06/06/2005 08:53:33

bah t'as le XHTML transitionnal pour les frames, mais bon perso je considère pas ça comme du xhtml, c'est du html bricolé...

Bref ouais c'est pas valide XHTML strict, ça nuit au réféncement, bref je vois pas pourquoi on les utilise, ça ne sert pas à grand chose.

a +

Commentaire de malalam le 06/06/2005 10:46:36 administrateur CS

hello,

quelques commentaires :
- tout d'abord, tant mieux si ce source te convient, c'est toujours TRES agreable, que l'on debute ou pas, de parvenir a ses fins :-)
- en effet, comme les autres, j'evite les iframes. Tu peux faire la meme chose sans iframes. Les div, pourquoi pas :-) De plus si tu tiens a tes barres eventuelles de scroll etc...quelques styles et c'est regle (overflow...).
-  for ($i=0, $n = count($line); $i < $n; $i++) : on evite ca en php. La boucle for () evalue la condition qu'on lui passe a chaque iteration. Ce qui signifie que pour i allant de 0 a 99(par exemple), ton code va effectuer 100 fois le count ($line), ce qui, tu en conviendras, et meme si tu n'es pas un fan de vitesse ;-) est quand meme inutile. Et puis finalement, c'est plus clair d'ecrire ca :
$cptLine = count ($line);
for ($i=0, $n = $cptLine; $i < $n; $i++)
heu et la du coup je me rends compte que je n'ai pas compris l'interet du $n... ? (je ne l'avais pas vu lol).
$cptLine = count ($line);
for ($i = 0; $i < $cptLine; $i++)

Sinon c'est plutot propre pour un debut.

Commentaire de originalcompo le 06/06/2005 13:26:11

OK pour le iframe: je l'ai enlevé
euhh GAME2OVR, je me demande si c'est ton image ou ton nom, mais j'ai presque l'impression d'avoir été agressé par ton message !! mais merci quand même parceque là, du coup, j'ai utilisé un div associé à javascript (chose que je fais déjà couremment dans mon site) ce qui m'a permis de ne plus avoir qu'un seul fichier php. Je ne l'avais pas fait avant parceque jusqu'à l'avant dernière correction, je ne voyais pas comment faire a part utiliser un include...

Ok pour le
  $cptLine = count ($line);
  for ($i=0, $n = $cptLine; $i < $n; $i++)
Pour ma défense, c'était un simple copier/coller de code (que je citais d'ailleurs) que j'avais récupéré... j'ai pas lu à fond je l'avoue ... même si je suis toujours pas "fan" de vitesse ;-)

a+

Et merci encore pour vos critiques, elles ont été constructives pour la plupart...

Commentaire de malalam le 06/06/2005 13:34:42 administrateur CS

Alors la...aucune idee :-)
Et en effet c'est dommage, tout a fait d'accord.

Commentaire de originalcompo le 06/06/2005 16:42:30

ah? et puisque tu as l'air de savoir, ca vient d'où, s'il te plait?

Commentaire de Anthomicro le 06/06/2005 16:51:20

Bah ça vient du langage qu'est comme ça, de toute façon c'est plus intelligent de faire comme ça, si ta valeur maximale change en cours de bouclage, t'es bonbon en delphi apparemment, pas en php.

Commentaire de originalcompo le 06/06/2005 17:10:29

"si ta valeur maximale change en cours de bouclage, t'es bonbon en delphi apparemment, pas en php"

c'est tout a fait vrai, sauf que tu semble voir ca comme un inconvenient, et moi (surement par habitude, encore que j'ai commencé par le C), je vois ca comme de la rigueur de programmation 'c'est un point de vue, ne vous acharnez pas sur moi ;-) ! ).

(il ne me viendrait pas à l'idée d'utiliser un for si je sais que le nombre d'iteration que je veux faire peux varier)

D'autant plus qu'on peut faire la même chose avec un for et un while (en supposant que le while existe en php, mais ca m'etonnerait que non)