preg replace

kan du ikke bruge 'capturing groups' à la

'/(^| ):\-?\)( )?/im' => '$1<div style="xxx"></div>$2'

???

jeg er ikke helt med?

hmm - det kan jeg egentlig godt forståe...
Er du vant til at bruge regular expressions?
Hvordan ser dit kald fra php ud? Når der er ting i pattern'et som er i ()paranteser kan de genbruges i replacement-strengen vha $1, $2 osv. Pas på med $-tegnet, hvis det er en "" streng skal den backslashes.

jeg er ikke vildt god til reg exp..

$SMILEY = array(
    '/(^| ):\-?\)/im' => '\\1<div style="height:13px; width:13px; background:url(gfx/smiley.smile.png)"></div>',
    '/(^| );\-?\)/im' => '\\1<div style="height:13px; width:13px; background:url(gfx/smiley.wink.png)"></div>',
    '/(^| ):\-?d/im' => '\\1<div style="height:13px; width:13px; background:url(gfx/smiley.laugh.png)"></div>',
    '/(^| ):\-?p/im' => '\\1<div style="height:13px; width:13px; background:url(gfx/smiley.tongue.png)"></div>',
    '/(^| ):\-?\//im' => '\\1<div style="height:13px; width:13px; background:url(gfx/smiley.think.png)"></div>',
    '/(^| ):\-?\|/im' => '\\1<div style="height:13px; width:13px; background:url(gfx/smiley.disappointed.png)"></div>',
    '/(^| ):\-?\(/im' => '\\1<div style="height:13px; width:13px; background:url(gfx/smiley.sad.png)"></div>',
    '/(^| ):\-?o/im' => '\\1<div style="height:13px; width:22px; background:url(gfx/smiley.sleep.png)"></div>',
    '/(^| ):\-?x/im' => '\\1<div style="height:13px; width:13px; background:url(gfx/smiley.dead.png)"></div>'
);

preg_replace(array_keys($SMILEY), array_values($SMILEY), $str);

aah, jeg kan se at \1 også er en 'backreference' - jeg tror du skal lave dem sådan her:
'/(^| ):\-?\)([\s]?)/im' => '$1<div style="height:13px; width:13px; background:url(gfx/smiley.smile.png)"></div>$2'

tilføjet ([\s]?) som gerne skulle tage et mellemrum efter smileyen, og putte det ind i $2...
Meeen, jeg er heller ikke super garvet, men det plejer at kunne lade sig gøre hvis man bare hekser lidt med det..

hvad mener du med backreference?

hvad er forskellen på \\1 og $1??

desuden er det ikke nok kun at lave et mellemrum... det SKAL være &nbsp; :)

"\\1" og "\$1" betyder det samme - nemlig en backreference. De refererer til ()-grupper i det regulære udtryk, i dette tilfælde (^| ) og ([\s]?), som begge måske er et mellemrum, måske ingenting.

>> desuden er det ikke nok kun at lave et mellemrum... det SKAL være &nbsp; :)
Du har ret, du skal nok bruge preg_replace_callback i stedet for preg_replace.

noget i retning af
function smiley_callback ($m) {
  $rval = "";
  if ($m[1]!='') $rval .= '&nbsp';
  $style=array (
    ':-)' => '<div style="height:13px; width:13px; background:url(gfx/smiley.smile.png)"></div>',
    ';-)' => '<div style="height:13px; width:13px; background:url(gfx/smiley.wink.png)"></div>'
  );
  $rval .= $style[$m[2]];
  if ($m[3]!='') $rval .= '&nbsp';
  return $rval;
}

echo preg_replace_callback ('/(^| )([;:]\-?[\\\|\/\)\(spdSPD])(\s?)/im', 'smiley_callback', $str);

----------hvis du kan se hvad jeg mener

nemmere:

function smiley_callback ($m) {
  $rval = "";
  if ($m[1]!='') $rval .= '&nbsp';
  $style=array (
    ':-)' => 'smile',
    ':)' => 'smile',
    ';-)' => 'wink'
    /*...der skal fyldes ud...*/
  );
  $rval .= '<div style="height:13px; width:13px; background:url(gfx/smiley.'.$style[$m].'.png)"></div>';
  if ($m[3]!='') $rval .= '&nbsp';
  return $rval;
}

echo preg_replace_callback ('/(^| )([;:]\-?[\\\|\/\)\(spdSPD])(\s?)/im', 'smiley_callback', $str);

aah fuck

$style[$m[2]]

jeg ser lige på det :)

vender tilbage om et par dage, for jeg har lidt travlt

jeg må altså lige be dig om at skære det ud i pap :) jeg fatter ikke rigtig noget af det kode du har skrevet...

hvad gør \s ??

i den preg du har lavet SKAL der være - med i min smiley før den bliver lavet

jeg er kommet frem til følgende, men det virker stadig ikke som det skal :(

når der står /im er det så stadig nødvendigt at have både store og små bogstaver med?

function smiley_callback($m)
{
    $smiley = array(
        '/:\-?\)/im' => '<div style="height:13px; width:13px; background:url(gfx/smiley.smile.png)"></div>',
        '/;\-?\)/im' => '<div style="height:13px; width:13px; background:url(gfx/smiley.wink.png)"></div>',
        '/:\-?d/im' => '<div style="height:13px; width:13px; background:url(gfx/smiley.laugh.png)"></div>',
        '/:\-?p/im' => '<div style="height:13px; width:13px; background:url(gfx/smiley.tongue.png)"></div>',
        '/:\-?\//im' => '<div style="height:13px; width:13px; background:url(gfx/smiley.think.png)"></div>',
        '/:\-?\|/im' => '<div style="height:13px; width:13px; background:url(gfx/smiley.disappointed.png)"></div>',
        '/:\-?\(/im' => '<div style="height:13px; width:13px; background:url(gfx/smiley.sad.png)"></div>',
        '/:\-?o/im' => '<div style="height:13px; width:22px; background:url(gfx/smiley.sleep.png)"></div>',
        '/:\-?x/im' => '<div style="height:13px; width:13px; background:url(gfx/smiley.dead.png)"></div>'
    );
   
    if($m[1] != '') $rval .= '&nbsp;';
   
    $rval .= preg_replace(array_keys($smiley), array_values($smiley), $m[2]);
   
    if($m[3] != '') $rval .= '&nbsp;';
   
    return $rval;
}

1.>> hvad gør \s ??
- det er et mellemrum

2.>> i den preg du har lavet SKAL der være - med i min smiley før den bliver lavet
- Det *burde* ikke være nødvendigt, "\-?" er "nul eller ingen -"

3.>> når der står /im er det så stadig nødvendigt at have både store og små bogstaver med?
- nææ, det har du ret i, i'et står for 'case Insensitive'. Så det skulle faktisk være '/(^| )([;:]\-?[\\\|\/\)\(spdx])(\s?)/im', eller '/(^| )([;:]\-?[\\\|\/\)\(dpox])(\s?)/im', alt efter hvilke du vil understøtte.

4. Ideen i preg_replace_callback er at du kan være mere selektiv med hensyn til hvad der bliver erstattet med - funktionen (eks. smiley_callback) modtager som argument et array, hvor [0] er hele matchet (eks. smiley'en), og [1], [2] etc er de 'underinddelinger' du har lavet i dit regex vha () paranteser.  Der er m.a.o. ingen grund til at kalde preg_replace INDEN I smiley_callback, da du her har resultatet som $m[2]. Min ide er at lave et array, hvor hver enkelt smiley - som streng - er 'key' til den tilhørende billedfil.. altså
  $style=array (
    ':-)' => 'smile',
    ':)' => 'smile',
    ';-)' => 'wink'
    /*...der skal fyldes ud...*/
  );
som du ser er der to der peger på 'smile', én med og én uden bindestreg.

C.U. -J:^)

i dit forslag SKAL der være - med før det virker :(

Nej. Nu har jeg testet lidt... Problemet er mellemrummene, og formuleringen (^| ) - hvis jeg skriver (\s?) i stedet for både først og sidst virker det.

<?php
function smiley_callback ($m) {
  $rval = "";
  if ($m[1]!='') $rval .= '&nbsp';
  $style=array (
    ':-)' => 'smile',
    ':)' => 'smile',
    ';-)' => 'wink',
    ';)' => 'wink'
    /*...der skal fyldes ud...*/
  );
  $rval .= '['.$style[$m[2]].']';
  if ($m[3]!='') $rval .= '&nbsp';
  return $rval;
}

$str=";) hej meddig;-)  :-) :)";
echo preg_replace_callback ('/(\s?)([;:]\-?[\\\|\/\)\(spdox])(\s?)/im', 'smiley_callback', $str);
?>

ligger på http://www.krimskrams.dk/test/preg.php

-J.^)

Det virker som om den også får dine &nbsp; 'er med nu-- men der er stadig potentielle problemer - fx vil "ftp://" blive til en mellemfornøjet... Evt ku' man bruge \b, som står for "word boundary", men det kan jeg ikke rigtig få til at virke.

Joe Regex'ene kan være tricky at have med at gøre...

det ser ud til at det nogenlunde virker nu, men det er ikke alle mine smilies som virker :(

function smiley_callback($var)
{
    $trans_arr = array(
        '/:\-?\)/i' => 'height:13px; width:13px; background:url(gfx/smiley.smile.png)',
        '/;\-?\)/i' => 'height:13px; width:13px; background:url(gfx/smiley.wink.png)',
        '/:\-?d/i' => 'height:13px; width:13px; background:url(gfx/smiley.laugh.png)',
        '/:\-?p/i' => 'height:13px; width:13px; background:url(gfx/smiley.tongue.png)',
        '/:\-?\//i' => 'height:13px; width:13px; background:url(gfx/smiley.think.png)',
        '/:\-?\|/i' => 'height:13px; width:13px; background:url(gfx/smiley.disappointed.png)',
        '/:\-?\(/i' => 'height:13px; width:13px; background:url(gfx/smiley.sad.png)',
        '/:\-?o/i' => 'height:13px; width:22px; background:url(gfx/smiley.sleep.png)',
        '/:\-?x/i' => 'height:13px; width:13px; background:url(gfx/smiley.dead.png)'
    );
   
    if($var[1] != '') $str .= '&nbsp;';
   
    $str .= '<div style="'.preg_replace(array_keys($trans_arr), array_values($trans_arr), $var[2]).'"></div>';
   
    if($var[3] != '') $str .= '&nbsp;';
   
    return $str;
}

$str = ":-) hehe :)\nhej ;) ftp:// hej:) :)hey";

echo preg_replace_callback('/(^|\s)([:;]\-?[\)\(\\\|pdox])(\s|$)/im', 'smiley_callback', $str);

hov... jeg havde lavet en lille fejl, men der er stadig nogle af mine smilies som ikke virker :(

http://clark.avansis.dk/test/smilies/

joe, det går da fremad... men satdigvæk: det er spild af krudt at kalde preg_replace inde fra din callback! Du må kunne finde på noget smartere... Jeg kan godt se at det virker lidt tåbeligt at have den der array med to ting der peger på det samme, men det er i hvert fald bedre end den anden måde... Ikke mindst mht kode-effektivitet. Regex koster processortid.

;^J

ok...

men hvorfor er det ikke alle mine smilies der virker? det ser da ellers rigtigt ud?

arh fuck... det virker jo ikke fordi mellemrummene bliver lavet om til &nbsp;

hvad kan jeg gøre? :(

jeps problemet er mellemrummene... SKAL der være mellemrum mellem smilies og ord? Hvis der skal, vil f.eks. "vi ses:)" ikke blive erstattet, hvilket jo er synd, hvis der ikke skal, får du problemet med "http://gedebuk.net" eller lign. Det kunne man dog hacke sig ud af, vha noget emre preg stads.. prøver lige...

altså... selve funktionen virker fint, lige på nær at man ikke kan lave flere smilies lige efter hinanden

nej "hej med dig:)", "http://noget" og lign. skal ikke erstattes :)

Yep, den kører.. En 'hurtig' betingelse om at der ikke må komme en / efter smilien... Nu er reg-ex'et: '/(\s?)([;:]\-?[\\\|\/\)\(spdox])(\s?)(?!\/)/im'

Stadig http://www.krimskrams.dk/test/preg.php, men nu er test strengen ";) hej meddig;-)  :-) :) : ):) proev via ftp://google.com"

På den måde behøver du heller ikke matche '&nbsp;'; Hvis det står der, bliver det bare stående, men hvis der er et mellemrum, kommer der i stedet '&nbsp;'.

Lægger lige showsource ind i min...

jeg ville egentlig gerne have at "hej med dig:)" ikke blev erstattet - altså så der SKAL være mellemrum mellem smilies og tekst

http://clark.avansis.dk/test/smilies/index2.php

\s tager jo alle whitespaces som et :( se i mit eksempel jeg har taget fra dig hvor jeg har lavet et break i stringen... der kommer et mellemrum foran smileyen i linje to

ja, det kan godt være det er at gå i detaljer, men vil gerne have så det virker 100% :)

jeg kan godt se det med break-- det ser åndssvagt ud at en linie starter med et mellemrum.
Så er problemet at mellemrummene bliver 'spist' så de ikke er der til næste match. Hvad med det her:
Matche mellemrummet før smilien, men ikke bagefter: så du skal bruge ( |^) foran og (?= |&nbsp;|\n|<) (lookahead, se http://dk2.php.net/manual/da/reference.pcre.pattern.syntax.php) bagefter... Samme url igen ;^J

jeg ser lige på det i morgen... er ved at være lidt for træt :)

hvorfor er der ?= med i den sidste gruppe?

'/( |&nbsp;|^)([;:]\-?[\\\|\/\)\(spdox])(?= |&nbsp;|\n|<)/im'

(?= |&nbsp;|\n|<) betyder at der SKAL komme mellemrum, newline eller < (html-tag) (- altså " |&nbsp;|\n|<") efter smilien for at matchen bruges. Mellemrummet kommer ikke med i matchen som gruppe, så linien med $m[3] skal faktisk slettes fra php-koden, se http://www.krimskrams.dk/test/preg.php igen-igen...

her er din kode smidt ind... men nu laver den jo ikke &nbsp; efter smilies

http://clark.avansis.dk/test/smilies/index2.php

hvorfor kan den ikke finde ud at at matche &nbsp; ??

- for i princippet burde den her jo virke http://clark.avansis.dk/test/smilies/

jeg har droppet gruppe 1 i stedet for, og nu ser det ud til at virke... :)

http://clark.avansis.dk/test/smilies/

joe, --- men nu laver den vel så ikke &nbsp; FØR smilies !?! Plus at hej:) bliver matchet nu!

Hos mig ser det nu ikke ud til at der er problemer med at matche &nbsp; !?!
'/( |&nbsp;|^)([;:]\-?[\\\|\/\)\(spdox])( |&nbsp;|$|(?=<))/im' virker tilsyneladende også fint.

ja, jeg ved godat at den tager hej:) med - men kunne ikke lige finde andre løsninger

kan du ikke lige prøve at lave din version med det sidste du skrev? :)

det er det allerede -- ændret lidt i teststrengen også, men det kan du jo se i showsource !

Go' nat, og happy hacking... ;^J

den laver indryk og tager ikke den sidste :o med

http://clark.avansis.dk/test/smilies/index2.php

jeg har kopieret din kode ind med mine smilies

mangler (?<=>) i ( |&nbsp;|^), check mit test eks. igen. Laver indryk forstår jeg ik hvad du mener, jeg kan ikke se billederne på din test... Ka' du ikke indsætte <html> og <head> ? Jeg bruger ikke Explorer... hjemme i hvert fald.

Men nu SKAL jeg altså i seng!

ååh, check http://www.google.com/search?q=php+smiley+preg&hl=da&lr=&ie=UTF-8&start=10&sa=N ... Vi er blevet indekseret!

næh, pokkers kun hvis man har dansk indstilling http://www.google.com/search?hl=da&ie=ISO-8859-1&q=php+smiley+preg&btnG=S%F8g&lr=

zzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz

altså
preg_replace_callback ('/( |&nbsp;|^|(?<=>))([;:]\-?[\\\|\/\)\(spdox])( |&nbsp;|$|(?=<))/im', 'smiley_callback', $str)
nu... Er den ved at være der?

jeg vender lige tilbage om et par dage :)

knudx > lav et svar :)

lukker