Discussion :

[piero53]

Bonjour,

Suite à mon précédent sujet, j'ai un petit problème.

Voilà la situation :

Je dois récupérer des parties spécifiques de mes mails(généralement les mails ont la même structure).

Je procède donc par une lecture ligne à ligne du mail puis je récupère ce qui m'interesse :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
$result = explode( "\r\n", $contenu);

	$i = 0;
	$max = count($result);

	//Initialisation
	$desc = "";
	
	while($i < $max) {
		if($result[$i++] == "DESCRIPTION:"){
			while ($result[$i] != "SOLUTION:"){
				$desc = $desc . $result[$i++];
			}
		}
	}

Ce code marche correctement et je parviens bien à récupérer ce que je souhaite dans la variable $desc.

Le problème c'est que dans ce $desc, toutes les ligns sont attachées... je n'ai plus de sauts de lignes.

Ca doit surement se jouer sur la ligne en gras du code, avec la concaténation de desc.

Merci

[jabbounet]

pourquoi ne pas utiliser grep?

http://php.net/manual/fr/function.preg-grep.php

[piero53]

Euh je ne connais pas cette fonction pour tout te dire

Elle serait plus pratique dans mon cas?

Si on admet que mes mails ont généralement la même structure :

INTRO
...
DESCRIPTION
...
SOLUTION
...
CONCLUSION

Ceci est contenu dans une variable $contenu.

Si je veux récupérer ce qui se trouve entre DESCRIPTION et SOLUTION je fais :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
preg_grep("INTRO.CONCLUSION",$contenu);

[Fench]

J'ai suivi un peu la suite de ton post précédent

Pourquoi tu veux les sauts de ligne ?

Sinon dans la concaténation, tu ajoutes les sauts de lignes !

[piero53]

Je souhaite les sauts de lignes car j'affiche après ma variable.

Mais oui c'est vrai j'ai juste à ajouter un saut de lignes içi :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$desc = $desc . $result[$i++] . '<br/>';

Je pense finalement me contenter de cette solution, ça passe...

Mais pour le grep, serait-ce plus efficace comme méthode dans ma situation ?

Sinon autre question :

Imaginons que la structure de mes mails reste la même, soit :

INTRO
...
DESCRIPTION
...
SOLUTION
...
CONCLUSION

Mais que cette fois-ci, dans UN seul mail il y ait plusieurs fois cette structure. Il faudrai dans ce cas que je récupère ce que je souhaite autant de fois qu'il y a la structure précédente. Genre la structure du dessus, elle serai présente 8 fois dans mon mails il me faudrait 8 variables pour chaque partie.

Je ne suis pas sûr d'être clair, en tout cas je vois pas trop comment procéder, car on suppose qu'on ne connais PAS le nombre de fois où la structure est présente.

[Fench]

Ca à l'air d'être une excellente idée de Jabbounet

Pour preg_grep, tu peux essayer un truc du genre:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$result = preg_grep("/^DESCRIPTION(\d)*SOLUTION$/", $contenu);

normalement $result[0] contiendrait les textes entre DESCRIPTION et SOLUTION, $result[1] un deuxième de ce type etc, etc ...

[piero53]

normalement $result[0] contiendrait les textes entre DESCRIPTION et SOLUTION, $result[1] un deuxième de ce type etc, etc ...

si c'est le cas ça serait alors beaucoup plus simple que de parser tout ligne par ligne

en tout cas j'ai pas trop compris comment marchais le patern pour preg_grep, et le :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$result = preg_grep("/^DESCRIPTION(\d)*SOLUTION$/", $contenu);

ne fonctionne pas, mais je pense que ça doit pas en être bien loin

[jabbounet]

ou alors

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$between=substr($contents, strpos($contents, "DESCRIPTION"), strpos($contents, "SOLUTION")

non testé je n'ai jamais fait de php.

[Fench]

Le problème c que je débute en regexp (j'ai un tuto de 10 pages dont j'ai lu que 2 ),

Sinon ya qd même une erreur dans mon code:
C'est plutôt:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$result = preg_grep("/^DESCRIPTION(.)*SOLUTION$/", $contenu);

mais il faudrait demander à un bon (genre spacefrog, s.n.a.f.u) pour c'est truc là. (Pose la question sur le forum Php/langage/Regex)

[jabbounet]

Le problème c que je débute en regexp (j'ai un tuto de 10 pages dont j'ai lu que 2 ),

Sinon ya qd même une erreur dans mon code:
C'est plutôt:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$result = preg_grep("/^DESCRIPTION(.)*SOLUTION$/", $contenu);

mais il faudrait demander à un bon (genre spacefrog, s.n.a.f.u) pour c'est truc là. (Pose la question sur le forum Php/langage/Regex)

dans l'expression régulière je ne mettrais pas le ^ ni le $, potentiellement je mettrais le commutateur m pour un traitement multi-lignes

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$result = preg_grep("/DESCRIPTION(.)*SOLUTION/m", $contenu);

[piero53]

Pour l'instant j'ai toujours ce warning :

Warning: preg_grep() expects parameter 2 to be array, string ...

Sinon pour moi les regex je nage aussi

[BornBanane]

Preg_grep utilise un tableau et $contenu est un string, il faudrait plutôt utiliser preg_match. Par contre j'utiliserais l'option s pour indiquer que . peut être une nouvelle ligne
Et donc :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

preg_match("/DESCRIPTION(.*)SOLUTION/s", $contenu,$result);

Dans $result[1] tu as ton texte.
Un petit trim sur $result[1] pour enlever les espaces en trop avant et après le text.

Et voila

[jabbounet]

Pour l'instant j'ai toujours ce warning :



Sinon pour moi les regex je nage aussi

essaye de passer result à la place de contenu (le resultat de explode)

[piero53]

Preg_grep utilise un tableau et $contenu est un string, il faudrait plutôt utiliser preg_match. Par contre j'utiliserais l'option s pour indiquer que . peut être une nouvelle ligne
Et donc :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

preg_match("/DESCRIPTION(.*)SOLUTION/s", $contenu,$result);

Dans $result[1] tu as ton texte.
Un petit trim sur $result[1] pour enlever les espaces en trop avant et après le text.

Et voila

Ton preg_match fonctionne bien

Il faut que j'enlève le mot DESCRIPTION et SOLUTION au début et à la fin aussi.

Mais cette solution est donc plus intéressante que parser ligne à ligner et comparer?

[piero53]

quand j'affiche en faisant

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

echo $result['0'];

j'ia bien la partie description, mais si j'affiche mon tableau entier :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
	while ($i<count($result)) {
		echo $result[$i];
		$i++;
	}

J'ai toute la description mais cette dernière se répète ...

En gros j'aimerai comprenne ce que contient le tableau, comment il se structure.

[Fench]

Bonjour,

Je continue un peu

A mon avis c $result['1'] qu'il faut prendre car c le résultat de première parenthése capturante (et donc sans DESCRIPTION et SOLUTION).


De même, de mon côté j'aurais testé:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
preg_match_all("/^DESCRIPTION(.*)SOLUTION$/", $contenu, $result);

en cherchant dans les $result de 1 aussi

A tester

[piero53]

Ouaip un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

echo $result[1]

m'enlève bien le mot DESCRIPTION et SOLUTION.

Par contre un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

echo $result['1'];

sur ton preg_match_all me renvoit array seulement.

[Fench]

Oui mais non, c que j'ai écrit trop vite, $result['1'] c pas bon

sur ton preg_match_all me renvoit array seulement

Après le preg_match_all fait un print_r($result); et tu vera bien ce qu'il faudra tester .... (voir aussi print_r($result[1]))

A+

[piero53]

humm... mais quel est l'intérêt d'un preg_match_all si le preg_match normal fonctionne ?

Sinon avec le preg_match j'ai tout ce que je souhaite sans le sauts de lignes, comment je peux les mettre ?

[Fench]

Non, le preg_match_all répéte la séquence de recherche (je crois que c l'équivalent de /s de François) c'est à dire dans le cas de plusieurs blocs (DESCRIPTION .... SOLUTION).

Par contre dans les deux cas, tu devrais avoir les sauts de lignes puisque le masque du preg_match est . donc tous les caractères !!!!

Essayes sinon en ajoutant \r\n dans le masque