Include pour insérer du code dans un IF

Version imprimable

31/05/2012, 17h02
cyberlp

Include pour insérer du code dans un IF
Bonjour,

J'ai une petite question liée à l'include en php.

J'ai un code du style :
Code:

1 2 3 4 if ( ($foo==$bla) || ($foo1==$blo) || ($foo2==$blu) ) { echo $var; }
J'ai un petit fichier code.txt qui contient des lignes :
Code:

1 2 3 4 ($foo3==$bly) || ($foo4==$blee) ||
etc., et ce généré par un autre code.

Je n'arrive pas à insérer mon fichier code.txt pour pouvoir allonger la liste des OR de la condition IF.

Si je fais par exemple :
Code:

1 2 3 4 5 if ( include('code.txt'); ($foo==$bla) || ($foo1==$blo) || ($foo2==$blu) ) { echo $var; }
cela génère un message d'erreur.

Comment procéder ?

Merci par avance.

essaye ça :

Code:

1
2
include('code.txt');
if ( eval(file_get_contents('code.txt')) || $truc || $muche )

Il me fait une erreur :

Code:

Parse error: syntax error, unexpected '('

au niveau de la ligne de l'eval.

Pour info, voilà à quoi ressemble le code :

Code:

1
2
3
4
5
6
7
8
9
10
11
include('code.txt');
 
if (
 
eval(file_get_contents('code.txt'))
 
(($EnumArray_categorie[$j]=="Wikipedia") && ereg("wikipedia",extraitRacine($resultats[$i])))
||
(($EnumArray_categorie[$j]=="Wikipedia") && ereg("wikiquote",extraitRacine($resultats[$i])))
)
{ ... }

et code.txt :

Code:

1
2
(($EnumArray_categorie[$j]=="E-commerce") && ereg("fnac",extraitRacine($resultats[$i])))||
(($EnumArray_categorie[$j]=="") && ereg("ebooks-gratuit",extraitRacine($resultats[$i])))||

31/05/2012, 18h15
Benjamin Delespierre

Tu pars du mauvais pied. La directive include ne sert par à incorporer des fragments de script dans d'autres scripts mais à inclure des traitements ou de l'affichage. De ce fait, tout ce que contiens ton script include doit impérativement être du PHP valide qui doit pouvoir être interprété séparément de tout autre script.

Si tu veux vraiment pouvoir séparer tous les cases de ta structure de contrôle dans différents fichiers (ce que je trouve impropre pour ma part), il faut que tu construise l'intégralité de ta structure dans une chaîne de caractères et que tu l'évalues. Mais franchement je te le déconseille vu le problème de sécurité que ça soulève. Il y a forcément un autre moyen, qu'est ce que tu essaie de faire au juste ?
31/05/2012, 18h21
cyberlp

En gros voilà l'idée :

1) Je tape un mot clé dans un formulaire.
2) Le script récupère et affiche les 10 premiers résultats Google sur ce mot clé
3) Pour chacun des 10 liens, il affiche à côté un petit menu déroulant :
- si le lien Google est déjà qualifié, il affiche la catégorie correspondante (par ex: "Blog"), récupérée d'après une BDD MySql.
- si le lien n'est pas qualifié, alors le menu déroulant n'affiche rien par défaut, mais propose plusieurs choix.

L'idée à partir de là est qu'à chaque fois que je qualifie un nouveau lien, il "apprend", c'est-à-dire qu'il va générer un fichier ('code.txt') qui permettra à l'avenir, à chaque fois qu'un lien de ce type reviendra, de faire apparaître dans le menu déroulant la catégorie correspondante.
31/05/2012, 18h22
Benjamin Delespierre

Donc tu cherches à faire un cache en somme ?

Si c'est juste pour se "souvenir" de la qualification d'un domaine, tu peux t'en sortir avec des enregistrements en base simples.

Si tu veux enregistrer des comportements pour que ton algorithme s'enrichisse. Tu devrais regarder du coté des patron de comportement comme Strategy ou Chain of Responsibility et mettre en cache les objets de strategy ou de la chaine.

Quoi qu'il en soit, la méthode de construction statique de l'algorithme est une mauvaise idée.
31/05/2012, 18h27
cyberlp

Oui c'est une manière de voir les choses.

Enfin dans ma vision, le bout de code avec plein de conditions "OR" était incrémenté au fur et à mesure par le contenu du fichier 'code.txt'.
31/05/2012, 18h36
Benjamin Delespierre

On ne construit pas un programme lors du runtime.

Oublions la programmation deux secondes et pensons le programme comme une voiture, quand tu veux la modifier, tu vas au garage et tu fais tes modifications à l'arrêt non ? ça ne te viendrait pas à l'esprit d'aller fixer un pot chromé à l'arrière pendant que t'es à 130 sur l'autoroute si ?
Maintenant, tu peux voir ta voiture autrement; comme celle de Taxi qui peut changer l'espacement des pneus ou le volant sans s'arrêter mais c'est parce que la voiture est équipée pour. C'est une illustration (un peu bête) du pattern strategy.
31/05/2012, 18h39
cyberlp

Je comprends ta métaphore :)

Mais je ne voyais pas cela comme ça, puisque :

Le script s'effectue.

Pendant son processus il incrémente le fichier code.txt.

Le script est fini.

Et à la prochaine utilisation du script, le script sera modifié par le nouveau contenu de code.txt.

Donc ce n'est pas vraiment pendant le runtime, si ?
31/05/2012, 18h48
Benjamin Delespierre

Si parce que rien ne te garantis qu'un autre script ne va pas lire ce fichier txt pendant que l'autre l'écrit. En réalité ce que tu fais c'est construire des parties de ton programme à la demande, donc en gros faire les changements pendant que tu roules, si pour une raison X ou Y ça se passe mal, tout le reste du service plante et on retrouve la voiture encastrée dans un poteau :mouarf:

Enfin, moi je te dis comment je m'y serais pris, à toi de voir ce qui te va le mieux, c'est ton projet après tout :ccool:
31/05/2012, 18h51
cyberlp

Ok considérons que la sécurité n'est pas un problème :)

Le script ne tourne de toute façon qu'en local sur ma machine.

Y a-t-il un moyen de faire en gros ce que je voulais faire, l'include d'une partie de script dans ma condition IF ?

Citation:

Envoyé par Benjamin Delespierre

Quoi qu'il en soit, la méthode de construction statique de l'algorithme est une mauvaise idée.

you mean "dynamique", right ?

Sinon,
Code:

1 2 3 4 5 6 7 8 9 10 11 include('code.txt'); if ( eval(file_get_contents('code.txt')) (($EnumArray_categorie[$j]=="Wikipedia") && ereg("wikipedia",extraitRacine($resultats[$i]))) || (($EnumArray_categorie[$j]=="Wikipedia") && ereg("wikiquote",extraitRacine($resultats[$i]))) ) { ... }
il manquerait pas un "||" ligne 6 ?

(au fait c'est pas parce que j'essaye de t'aider que je cautionne tes façons de -vouloir- faire, car effectivement ça m'a pas l'air très propre, imagine qu'un gus arrive a écraser/éditer/modifier ton fichier code.txt par un moyen quelconque, il execute ce qu'il veut après... )

01/06/2012, 00h34
cyberlp

Citation:

Envoyé par lizardman

il manquerait pas un "||" ligne 6 ?

Non car le contenu de 'code.txt' se termine par un ||
01/06/2012, 10h35
Benjamin Delespierre

Citation:

Ok considérons que la sécurité n'est pas un problème

No comment :mrgreen:

Citation:

you mean "dynamique", right ?

Non justement, une construction dynamique serait d'évaluer des comportements définis au runtime. Là il s'agit d'écrire physiquement des scripts PHP donc on construit bien quelque chose de statique ;)

@Benjamin Delespierre : je vois ce que tu veux dire , on créer "dynamiquement" du code statique en gros.

@cyberIp :

Code:

1
2
3
4
5
6
7
8
9
10
include('code.txt');
$contenu=file_get_contents('code.txt');
$contenu .= ((($EnumArray_categorie[$j]=="Wikipedia") && ereg("wikipedia",extraitRacine($resultats[$i])))
||
(($EnumArray_categorie[$j]=="Wikipedia") && ereg("wikiquote",extraitRacine($resultats[$i]))) ) ? " true" : " false"
 
if (
eval($contenu)
 
{ ... }

et ça, ça marche ?

01/06/2012, 11h38
cyberlp

il me fait une syntax erreur :

Code:

Unexpected $end

au niveau de la ligne de l'eval.
01/06/2012, 11h42
lizardman

roo il manque une parenthese ... tu peux le voir ça quand même ! ;)
01/06/2012, 11h44
cyberlp

Sur le IF ? Oui je l'avais ajoutée :)
Mais l'erreur est toujours là.
01/06/2012, 11h45
lizardman

fais un

Code:

echo $contenu

pour voir comment c'est foutu stp.

Bon... Déjà, dans l'idée, rajouter un "include" dans un if, c'est trop crade pour que je te laisse faire.
Le fait d'écrire dans un fichier et (potentiellement), en même temps de le lire, ça peut provoquer des conflits et, dans le pire des cas, crasher ton fichier (qui se retrouvera tout vide). Maintenant, si c'est un script exécuté seulement en local avec une seule machine, le conflit devient vraiment peu probable.

La manière propre, c'est de faire une base de donnée (d'autant que visiblement, tu en as déjà une), d'enregistrer tes infos et de te faire une méthode qui va aller checker si ton nouveau lien est déjà connu ou pas.
Un peu plus simple et qui te conviendras probablement, c'est d'enregistrer tes liens dans un fichier et de te coder une méthode qui va lire ce fichier et retourner "true" ou "false", suivant l'existence d'un nouveau lien (et en cas de false, soyons fou, on écrit la nouvelle entrée dans le-dit fichier).

C'est parti :
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 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 <?php /** * Cette méthode prend une nouvelle entrée en paramètre, vérifie si elle est présente * dans le fichier 'code.txt' et la rajoute le cas échéant. * @param string : $nouvelleEntree : la nouvelle entrée à tester. * @return boolean : true si la nouvelle entrée existait, false sinon. */ function traiteNouvelleEntree($nouvelleEntree) { // première étape, on lit le fichier. $contenu = file_get_contents('code.txt'); // Ensuite, on mets chaque ligne du contenu dans un tableau. $listeLiens = explode("\n", $contenu); //print_r($listeLiens); // Maintenant, on vérifie si la nouvelle entrée existe dans le fichier ou pas. if (!in_array($nouvelleEntree, $listeLiens)) { // Si on a pas trouvé l'entrée, on la rajoute. // ps : quand on joue avec des fichiers, c'est bien de rajouter des protections pour savoir s'il a bien été // ouvert, mais là, j'ai la flemme ^_^. $fp = fopen('code.txt', 'a+'); fwrite($fp, $nouvelleEntree . "\n"); fclose($fp); return false; } else { return true; } } // "Début du code", on teste si le formulaire à envoyé quelque chose, et si c'est le cas, on envoie à la fonction. $action = @$_REQUEST['action']; $nouvelleEntree = @$_REQUEST['nouvelleEntree']; if (!empty($action)) { echo ("traitement de l'action ... <br />\n"); $result = traiteNouvelleentree($nouvelleEntree); // Et on affiche le résultat. if ($result == false) { echo ("L'entrée a été ajoutée !"); } else { echo ("Entrée existante !"); } } ?> <?php /* Petit formulaire qui est présent pour "simuler" ton entrée google. */?> <form name="monFormulaire" method="post"> Nouvelle entrée : <input type="text" name="nouvelleEntree" /><br /> <br /> <input type="hidden" name="action" value="ajouterEntree" /> <input type="submit" value="Valider" /> </form>
Voilà, avec ça, tu devrais t'en sortir ;-)

04/06/2012, 11h05
Benjamin Delespierre

L'idée d'utiliser des @ est trop crade pour que je laisse faire: utilise les filtres à la place :ccool:
06/06/2012, 11h15
Rakken

Hihi ^^

Je connais assez mal les filtres, mais de mémoire, ça sert surtout à valider des champs en entrée non ? Du genre t'as un user qui rentre un email et tu peux le valider / retraiter pour qu'il soit valide.
Du coup, je ne vois pas tellement en quoi ça peut aider dans le cas présent (mais je ne demande qu'à être surpris :ccool:).
06/06/2012, 11h17
Rakken

Ah, au temps pour moi, "@" je pensais que tu parlais des tableaux que j'utilise ! Pas de la vile arnaque pour pas avoir les warnings ^^.
Tu as parfaitement raison ;-))
06/06/2012, 11h30
Benjamin Delespierre

Citation:

Envoyé par Rakken

Je connais assez mal les filtres, mais de mémoire, ça sert surtout à valider des champs en entrée non ? Du genre t'as un user qui rentre un email et tu peux le valider / retraiter pour qu'il soit valide.
Du coup, je ne vois pas tellement en quoi ça peut aider dans le cas présent (mais je ne demande qu'à être surpris :ccool:).

Non, tu peux valider tout et n'importe quoi avec les filtres, en entrée comme en sortie. Voir filter_var_array par exemple. C'est un must-know en termes de validation / nettoyage de données.

L'utilisation du @ est très fortement déconseillée; ça empêche de voir les erreurs donc nuit au débug et à la maintenance et ça dégrade les performances du script. C'est comme mettre un autocollant sur le voyant moteur quand il s'allume, c'est pas une chose à faire.
06/06/2012, 11h38
Rakken

Oui oui, comme dit, je me suis planté sur le sens de "@" dans ton post, (j'avais oublié que j'en avais mis dans le code).
En vrai, j'en mets juste quand j'ai pas envie de me prendre la tête, pour faire rapide (de la même manière, j'ai pas protégé l'écriture dans un fichier, ce qui est le mal, juste, pour les @, j'ai pas précisé en commentaire que c'est aussi le mal). En fait, l'idée c'est que c'est la fonction qui est importante, le formulaire il était juste là pour montrer une utilisation.

Ceci étant, il va effectivement falloir que je me penche un peu plus en détail là dessus. J'ai tendance à protéger mes données moi-même avec une classe que je me suis fait il y a plusieurs années, ça pourrait être bien de la recoder en utilisant cette méthode ;-).
06/06/2012, 11h49
Benjamin Delespierre
Pour clore la parenthèse sur la sécurisation des données en entrée et en sortie, voici les classes que j'utilise (elles exploitent les filtres):
- axRequest
- axResponse