LaurentKOogar
Messages postés369Date d'inscriptionsamedi 4 septembre 2004StatutMembreDernière intervention20 octobre 2013 11 févr. 2008 à 11:20
ouaip il y a get_meta_tags et c'est pas nouveau. bon en admettant qu'on bosse sur un fichier deja ouvert... on peut s'amuser un peu ;) mais la j'ai rien compris a ton histoire de preg_replace...
alors que preg_match('#<meta name="description" content="(.*?)">#i', $a, $b); devrait tres bien faire l'affaire.
++
KOogar
cs_Windows300
Messages postés7Date d'inscriptionmercredi 6 février 2008StatutMembreDernière intervention10 février 2008 10 févr. 2008 à 16:04
je sais c'est vraiment triste !!!
webdeb
Messages postés488Date d'inscriptionsamedi 5 avril 2003StatutMembreDernière intervention31 mars 20094 10 févr. 2008 à 15:56
Bah pas bien !!! Sachant qu'il existe la fonction get_meta_tags() qui te permet de récupérer les balises meta avec une seule ligne de code.
cs_Windows300
Messages postés7Date d'inscriptionmercredi 6 février 2008StatutMembreDernière intervention10 février 2008 10 févr. 2008 à 15:46
Merci COUCOU747, tu ne verra mon code modifier
à partir de la semaine prochaine prochaine !
car je n'ai pas le temps ;)
WEBDEB, tu veux dire quoi par ton message ?
bien ou pas bien ?
webdeb
Messages postés488Date d'inscriptionsamedi 5 avril 2003StatutMembreDernière intervention31 mars 20094 10 févr. 2008 à 13:53
Ahahah excellente la source ^^ Pourquoi faire si compliqué quand c'est si simple.
coucou747
Messages postés12303Date d'inscriptionmardi 10 février 2004StatutMembreDernière intervention30 juillet 201244 10 févr. 2008 à 12:14
le commentaire de malalam l'etait aussi... si tu le relis, il te dit comment faire la meme fonctionalite, mais :
-de facon plus lisible
-de facon plus rapide
-en consomant moins de memoire
-de facon plus facilement modifiable
on se fait souvent ecraser windows300, si tu vois un mec qui vient avec un pseudo redhat9, si il code un peu en php, crois moi, ton code il le prendra, il laissera un commentaire genre :
- c'est mal code
soit tu mets du html, soit tu mets du xhtml, mais pas les deux
if(preg_match("#\[(.+)\]#i", "$array[$numero]"))
pourquoi "$array[$numero]" et pas $array[$numero] ?
t'as fait plusieurs fois cette erreur, je ne la cite qu'une fois...
sinon, ton code n'est pas indente, et file_get_contents c'est bien plus simple pour recuperer le contenu d'un fichier
Bref, tu vois, si qqn te dit que ce que tu as fait c'est mauvais, si il argumente comme ca... alors ca te permet de pouvoi ameliorer...
tu remarqueras que ni malalam ni moi n'avons note (parce-que j'espere voir ce code evoluer, je noterais plus tard, quand t'auras mis a jours en corrigeant ces erreurs)
cs_Windows300
Messages postés7Date d'inscriptionmercredi 6 février 2008StatutMembreDernière intervention10 février 2008 10 févr. 2008 à 11:58
Merci de ton commentaire COUCOU
enfin un commentaire inscructif, avec des idée d'améliroration, !!
encore merci !!
coucou747
Messages postés12303Date d'inscriptionmardi 10 février 2004StatutMembreDernière intervention30 juillet 201244 10 févr. 2008 à 11:43
moi je trouve l'idee originale, je proposerais cependant d'en faire plus une class genre :
class InfoPage{ ...
qui servirait par exemple lors de l'ajout de liens : quand tu ajoutes un lien de ton site vers un site exterieur, tu veux verifier si ce site exterieur :
-est toujours accessible
-correspond bien a la description que tu en avais (si t'aurais pu faire une faute de frappe en le tapant)
-si le site contient un lien vers chez toi (ce qui est utile pour les partenaires)
-les mots clefs du site (histoire d'etre sur que les deux sites sont en rapport)
ca peut permettre de mettre une description automatique des liens sous forme d'infobulles ou de notes de fin de pages aussi...
cs_Windows300
Messages postés7Date d'inscriptionmercredi 6 février 2008StatutMembreDernière intervention10 février 2008 10 févr. 2008 à 10:11
ou la la, !!!
- pour plus de faciliter
- ok, je le ferais !
- c'est pour le preg_replace(), pour ne pas avoir 36 solutions
- je pense pas, car l'array n'existe pas !
- j'ai jamais apris cette fonction
- pour plus de faciliter, et pour que j'apprène à les utilser
Merci de ton commentaire, je suis désolé si mon code
existait déja, mais je pensait que sa pouvait servir à certains ;)
malalam
Messages postés10839Date d'inscriptionlundi 24 février 2003StatutMembreDernière intervention 2 mars 201025 10 févr. 2008 à 10:05
Hello,
- pourquoi faire de la double assignation ? ($lien = $_POST["lien"])
- utiliser des apostrophes au lieu de guillemets est un gain de temps et de rapidité en général
- ton stripslashes et ton str_replace servent à quoi au juste ?
- pourquoi faire deux expression régulières, l'une pour tester si tu as des meta, l'autre pour les extraire ? Une suffirait amplement
- pourquoi preg_replace() alors que tu devrais utiliser preg_match_all() ?
- pourquoi re-expression régulière après...?
Même si tu voulais faire ça de cette manière, ton code est largement trop complexifié. Une seule expression régulière suffirait à la tâche.
11 févr. 2008 à 11:20
alors que preg_match('#<meta name="description" content="(.*?)">#i', $a, $b); devrait tres bien faire l'affaire.
++
KOogar
10 févr. 2008 à 16:04
10 févr. 2008 à 15:56
10 févr. 2008 à 15:46
à partir de la semaine prochaine prochaine !
car je n'ai pas le temps ;)
WEBDEB, tu veux dire quoi par ton message ?
bien ou pas bien ?
10 févr. 2008 à 13:53
10 févr. 2008 à 12:14
-de facon plus lisible
-de facon plus rapide
-en consomant moins de memoire
-de facon plus facilement modifiable
on se fait souvent ecraser windows300, si tu vois un mec qui vient avec un pseudo redhat9, si il code un peu en php, crois moi, ton code il le prendra, il laissera un commentaire genre :
- c'est mal code
soit tu mets du html, soit tu mets du xhtml, mais pas les deux
if(preg_match("#\[(.+)\]#i", "$array[$numero]"))
pourquoi "$array[$numero]" et pas $array[$numero] ?
t'as fait plusieurs fois cette erreur, je ne la cite qu'une fois...
sinon, ton code n'est pas indente, et file_get_contents c'est bien plus simple pour recuperer le contenu d'un fichier
Bref, tu vois, si qqn te dit que ce que tu as fait c'est mauvais, si il argumente comme ca... alors ca te permet de pouvoi ameliorer...
tu remarqueras que ni malalam ni moi n'avons note (parce-que j'espere voir ce code evoluer, je noterais plus tard, quand t'auras mis a jours en corrigeant ces erreurs)
10 févr. 2008 à 11:58
enfin un commentaire inscructif, avec des idée d'améliroration, !!
encore merci !!
10 févr. 2008 à 11:43
class InfoPage{ ...
qui servirait par exemple lors de l'ajout de liens : quand tu ajoutes un lien de ton site vers un site exterieur, tu veux verifier si ce site exterieur :
-est toujours accessible
-correspond bien a la description que tu en avais (si t'aurais pu faire une faute de frappe en le tapant)
-si le site contient un lien vers chez toi (ce qui est utile pour les partenaires)
-les mots clefs du site (histoire d'etre sur que les deux sites sont en rapport)
ca peut permettre de mettre une description automatique des liens sous forme d'infobulles ou de notes de fin de pages aussi...
10 févr. 2008 à 10:11
- pour plus de faciliter
- ok, je le ferais !
- c'est pour le preg_replace(), pour ne pas avoir 36 solutions
- je pense pas, car l'array n'existe pas !
- j'ai jamais apris cette fonction
- pour plus de faciliter, et pour que j'apprène à les utilser
Merci de ton commentaire, je suis désolé si mon code
existait déja, mais je pensait que sa pouvait servir à certains ;)
10 févr. 2008 à 10:05
- pourquoi faire de la double assignation ? ($lien = $_POST["lien"])
- utiliser des apostrophes au lieu de guillemets est un gain de temps et de rapidité en général
- ton stripslashes et ton str_replace servent à quoi au juste ?
- pourquoi faire deux expression régulières, l'une pour tester si tu as des meta, l'autre pour les extraire ? Une suffirait amplement
- pourquoi preg_replace() alors que tu devrais utiliser preg_match_all() ?
- pourquoi re-expression régulière après...?
Même si tu voulais faire ça de cette manière, ton code est largement trop complexifié. Une seule expression régulière suffirait à la tâche.
Ensuite -et enfin :
http://www.php.net/manual/fr/function.get-meta-tags.php