preg_replace functie om H1 t/m 6 te replacen
Heb hier een functie om alle H1 t/m 6 te vervangen
Code (php)
1
2
3
2
3
<?
$row['korte_content'] = preg_replace('/().*.(<\/h[1-3]>)/i', '', $row['korte_content']);
?>
$row['korte_content'] = preg_replace('/().*.(<\/h[1-3]>)/i', '', $row['korte_content']);
?>
deze is geplaatst na de while($row = mysql_fetch_assoc($result)) {
Echter is deze functie vreselijk traag (0.6 sec.)
Doe ik dit
Code (php)
1
2
3
2
3
<?
$row['korte_content'] = preg_replace('#(\<(h1|h2|h3)[^\>]*\>.*\</(h1|h2|h3)\>)#isU', "", $row['korte_content']);
?>
$row['korte_content'] = preg_replace('#(\<(h1|h2|h3)[^\>]*\>.*\</(h1|h2|h3)\>)#isU', "", $row['korte_content']);
?>
dan draait het als een speer (0.24 sec.)
wat is er fout aan de eerste preg_replace ?
Thanks
ps.
Hier nog een functie die goed werkt
Code (php)
1
<? $row['korte_content'] = preg_replace('|<h[1-9][^>]*>.*?</h[1-9][^>]*>|i', '', $row['korte_content']); ?>
Wat zijn de voordelen van de onderste twee voorbeelden ?
Die tweede is beter, want je hebt daar maar drie mogelijkheden, verder houd je er ook rekening mee dat er dus een id o.i.d. aan de tag toegevoegd kan zijn. Ondanks dat er in sommige cheatsheets staat dat je < en > moet escapen, hoeft dat dus niet. Verder gebruik je de U modifier, die er voor zorgt dat je ungreedy matcht. Deze matcht ook verkeerd.
De derde is hetzelfde als de tweede, alleen zijn de escapes weggehaald, heb je hem ungreedy gemaakt en mag de sluit h tag ook meer inhoud bevatten.
Edit: Ik vind dit de beste regex:
Code (php)
1
2
3
4
5
6
7
2
3
4
5
6
7
<?php //Kleurtjes
$subject = '<h1 id="test">test</h1> test <h2>test</h2> test test <h3 id="tesst">blaat<span id="test">test</span></h3>';
$result = preg_replace('%<(h[1-6])[^>]*>([^\1]*)</\1>%', '\2', $subject);
var_dump($result);
[/code]
Heb hem niet op snelheid getest. Wat wil je berhaupt eigenlijk doen? Kan je misschien niet beter [php]strip_tags[/php] gebruiken?
$subject = '<h1 id="test">test</h1> test <h2>test</h2> test test <h3 id="tesst">blaat<span id="test">test</span></h3>';
$result = preg_replace('%<(h[1-6])[^>]*>([^\1]*)</\1>%', '\2', $subject);
var_dump($result);
[/code]
Heb hem niet op snelheid getest. Wat wil je berhaupt eigenlijk doen? Kan je misschien niet beter [php]strip_tags[/php] gebruiken?
In het block komt dan de titel te staan met daarna 250 woorden met lees meer
Vaak staat in het artikel bovenaan een H1 met dezelfde tekst als de title (voor goede SEO)
In het verkorte block moet dan dus die H1 (2,3 ect.) eruit en uiteraard daarna strip_tags
Heb nu deze
Code (php)
1
2
3
2
3
<?
$row['korte_content'] = preg_replace('~<(h[1-6])(?=[\s>])[^<>]*>([^<]*(?:<(?!/\1)[^<]*)*)</\1>~' , '', $row['korte_content']);
?>
$row['korte_content'] = preg_replace('~<(h[1-6])(?=[\s>])[^<>]*>([^<]*(?:<(?!/\1)[^<]*)*)</\1>~' , '', $row['korte_content']);
?>
Voor mij is snelheid in dit cms het belangrijkste :)
Ben bezig met een supersnel cms die toch voldoende functies bevat. Snelheid zover 0.30 seconden per pagina
Gewijzigd op 15/05/2010 18:03:21 door Ceasar Feijen
Caesar:
In het verkorte block moet dan dus die H1 (2,3 ect.) eruit en uiteraard daarna strip_tags
En waarom kan dat dan niet direc in één keer? Als je die strip_tags er toch nog overheen gooit...
Alles nu in een functie
Thanks
Je kunt wel leuk die headers eruit gaan filteren, dat doet strip_tags ook. Waarom moet jij het dan ook nog eens apart doen? Test het anders eens, met alleen strip_tags...
Ik wil niet alleen alle tags eruit hebben maar ook de inhoud van een H1
vb. <h1>Dit moet er ook uit</h1>
Code (php)
1
2
3
4
5
6
7
8
2
3
4
5
6
7
8
<?php
$text = '<h1>Test paragraph.</h1><!-- Comment --> <a href="#fragment">Other text</a>';
echo strip_tags($text);
echo "\n";
// Allow <h>
echo strip_tags($text, '<h1><h2><h3>');
?>
$text = '<h1>Test paragraph.</h1><!-- Comment --> <a href="#fragment">Other text</a>';
echo strip_tags($text);
echo "\n";
// Allow <h>
echo strip_tags($text, '<h1><h2><h3>');
?>
Gewijzigd op 15/05/2010 21:54:52 door Ceasar Feijen
Caesar:
Ik denk dat je het niet helemaal begrijpt.
Ik wil niet alleen alle tags eruit hebben maar ook de inhoud van een H1
vb. <h1>Dit moet er ook uit</h1>
Ik wil niet alleen alle tags eruit hebben maar ook de inhoud van een H1
vb. <h1>Dit moet er ook uit</h1>
Ah, jah, dat was niet geheel duidelijk. Ik geloof niet dat die regex van jou hierboven sneller is dan die van mij, bij die van mij moet je alleen effe die \2 in de replace weghalen.