fjerne tegn i slutning af streng ved hjælp af preg_replace()

Det vil være væsentligt bedre at undgå at bruge regular expressions til så små formål.

En løsning kunne være

Du kan prøve:

$str = preg_replace('/ M$/', '', $str);

#2
Jeg troede faktisk du ville være en af de første til at fraråde brugen af reg exp til så små ting? :)

Tager jeg fejl eller er den gængse opfattelse ikke at reg exp er 'tungt' og hvis opgaverne er små nok, så er det bedre at undgå det?

repox jo det er også lidt mere kompleks den bruges, men jeg har forenklet det for at gøre det lettere for Jer at hjælpe mig :o)

erik din løsning fungerer :o) smid svar tak

"Bedre" .... ud fra hvilke kriterier?

#2
Kriteriet var at reg exp er (påstået) tungere end PHP's generelle streng-funktioner. Det er jeg blevet tudet ørerne fulde af flere gange - ikke kun her på eksperten.dk men også på andre fora.

Hov... #6 var egentlig til #5 - og jeg vil gerne tilføje et link til SO som en reference for min undren: http://stackoverflow.com/questions/634676/which-is-more-efficient-php-string-functions-or-regex-in-php

Jeg ved godt at der ikke er universelle sandheder, men hvis jeg har mulighed for at lære lidt mere, så er jeg åben...

Jeg tror heller ikke på universelle sandheder ;)

Men lad mig lige anføre et par ting

1)
Vi ved ikke om spørgeren skal udføre en replace hver anden time eller 5000 i sekundet.

2)
Vi ved ikke om din løsning konkret er hurtigere, og i givet fald hvor meget.

3)
Din løsning giver ikke samme resultat som min, så det er som at sammenligne æbler med meteorsten.

4)
Et regulært udtryk er dokumentation i sig selv, og din løsning er hverken bevist korrekt, eller testet korrekt.

... men ja, der kan selvfølgelig være forskellige løsninger på samme problem.

#8
Point taken :)

;)

5)
Og ja, det vil være forholdsvis nemt at ændre din kode til at fjerne "mellemrum og stort M i slutningen af strengen". Og ja, det kan godt være det er hurtigere.

jeg skal replace mange gange på alle sidevisninger. Er den da meget langsom den metode? Så vil jeg da gerne vide det og evt have forslag til en anden løsning som kan det samme :o)

her er det komplette jeg bruger:
$title_product_name_remove_patterns[] = '/(\d+),(\d+) kg./i';
$title_product_name_remove_patterns[] = '/(\d+),(\d+) kg/i';
$title_product_name_remove_patterns[] = '/(\d+) kg./i';
$title_product_name_remove_patterns[] = '/(\d+) kg/i';
$title_product_name_remove_patterns[] = '/(\d+)kg./i';
$title_product_name_remove_patterns[] = '/(\d+)kg/i';
$title_product_name_remove_patterns[] = '/(\d+) ml./i';
$title_product_name_remove_patterns[] = '/(\d+) ml/i';
$title_product_name_remove_patterns[] = '/(\d+)ml./i';
$title_product_name_remove_patterns[] = '/(\d+)ml/i';
$title_product_name_remove_patterns[] = '/(\d+) stk./i';
$title_product_name_remove_patterns[] = '/(\d+) stk/i';
$title_product_name_remove_patterns[] = '/(\d+) gr./i';
$title_product_name_remove_patterns[] = '/(\d+) gr/i';
$title_product_name_remove_patterns[] = '/(\d+)gr./i';
$title_product_name_remove_patterns[] = '/(\d+)gr/i';
$title_product_name_remove_patterns[] = '/(\d+) g./i';
$title_product_name_remove_patterns[] = '/(\d+) g/i';
$title_product_name_remove_patterns[] = '/(\d+)g./i';
$title_product_name_remove_patterns[] = '/(\d+)g/i';
$title_product_name_remove_patterns[] = '/(\d+)x(\d+) cm./i';
$title_product_name_remove_patterns[] = '/(\d+)x(\d+) cm/i';
$title_product_name_remove_patterns[] = '/(\d+) cm./i';
$title_product_name_remove_patterns[] = '/(\d+) cm/i';
$title_product_name_remove_patterns[] = '/(\d+)cm./i';
$title_product_name_remove_patterns[] = '/(\d+)cm/i';
$title_product_name_remove_patterns[] = '/(\d+) stk./i';
$title_product_name_remove_patterns[] = '/(\d+) stk/i';
$title_product_name_remove_patterns[] = '/(\d+)stk./i';
$title_product_name_remove_patterns[] = '/(\d+)stk/i';
$title_product_name_remove_patterns[] = '/ XS$/';
$title_product_name_remove_patterns[] = '/ S$/';
$title_product_name_remove_patterns[] = '/ M$/';
$title_product_name_remove_patterns[] = '/ L$/';
$title_product_name_remove_patterns[] = '/ XL$/';
$title_product_name_remove_patterns[] = '/ XXL$/';
$title_product_name_remove_patterns[] = '/ smagsprøve/i';
$title_product_name_remove_patterns[] = '/(\d+) x /i';
$title_product_name_remove_patterns[] = '/(\d+)-(\d+) kilo/i';
$title_product_name_remove_patterns[] = '/med (\d+) pipetter/i';
$title_product_name_remove_patterns[] = '/over (\d+) kilo/i';
$title_product_name_remove_patterns[] = '/(\d+)-(\d+) kg/i';
$title_product_name_remove_patterns[] = '/  /i';
$title_product_name_remove_patterns[] = '/  /i';

$stripped_products_name = trim(preg_replace($title_product_name_remove_patterns, '', trim("Produktnavn Grøn 40x12 cm")));

Det virker voldsomt at gøre det ved hver visning. Hvad med at have et ekstra felt i databasen, så du både har det fulde navn "Produktnavn Grøn 40x12 cm" og kælenavnet "Grøn" - eller hvad det nu bliver til?

Du kan evt. lave den store replace ved oprettelsen af produktet.

nej det skal helst foregå når siden loader. og det foregår altså rigtig mange gange på nogen sider. men kan det laves samme nummer med en anden funktion, som er hurtigere??

"helst" ?  Du vil helst når siden loader. Dvs du udelukker ikke en anden løsning. Det lyder jo godt.

Men du kan jo prøve at se om det er for langsomt. Hvis ikke, så er der ingen grund til at ændre noget.

jo men det er en funktion til en webshop. der selv kigger om der er nogen "beslægtede" produkter og så grupperer produkterne.
det er pænere på produktlisterne på den måde + jeg sætter automatisk canonical tag på så den beregner hvilket produktid der er primær, og på den måde undgår duplicate content.
det er jo rart at den selv finder ud af det, så det skal helst være en funktion som kan det samme hvis det skal laves om.

Men jeg må da tilstå at siden er blevet lidt langsommere efter jeg indførte det.

Men jeg er såmænd meget åben for ændringer, men ser bare lidt skeptisk på at lave noget onsave. Så hellere lave en funktion der er hurtigere.

Du kan samle nogle af de regulære udtryk, så der ikke er så mange, men ellers kan jeg ikke se en beregning, der kan være hurtigere.

En samling kan fx være at ændre:

  $title_product_name_remove_patterns[] = '/(\d+) kg./i';
  $title_product_name_remove_patterns[] = '/(\d+) kg/i';
  $title_product_name_remove_patterns[] = '/(\d+)kg./i';
  $title_product_name_remove_patterns[] = '/(\d+)kg/i';
  $title_product_name_remove_patterns[] = '/(\d+) ml./i';
  $title_product_name_remove_patterns[] = '/(\d+) ml/i';
  $title_product_name_remove_patterns[] = '/(\d+)ml./i';
  $title_product_name_remove_patterns[] = '/(\d+)ml/i';

til (vistnok)

  $title_product_name_remove_patterns[] = '/(\d+) ?(ml|kg)\.?/i';

Men ellers kan jeg ikke se noget problem i et forudberegnet felt i databasen.

ja det var smart - jeg er ikke så god til det

angående: "Men ellers kan jeg ikke se noget problem i et forudberegnet felt i databasen."

jo hvis nu jeg pludselig ændrer $title_product_name_remove_patterns så er det som står i felterne jo ikke nutidigt men forældet og forkert. medmindre jeg laver noget smart der tager hånd om det.

okay nu har jeg lavet dette:
$title_product_name_remove_patterns[] = '/(\d+) x /i';
$title_product_name_remove_patterns[] = '/ (\d+),(\d+) (cm|kg)\.?/i';
$title_product_name_remove_patterns[] = '/ (\d+)x(\d+) cm\.?/i';
$title_product_name_remove_patterns[] = '/ (\d+)-(\d+) (kilo|kg)\.?/i';
$title_product_name_remove_patterns[] = '/ (over|med) (\d+) (kilo|kg|pipetter)\.?/i';
$title_product_name_remove_patterns[] = '/ smagsprøve/i';
$title_product_name_remove_patterns[] = '/ ?(\d+) ?(ml|kg|stk|cm|gr|g)\.?/i';
$title_product_name_remove_patterns[] = '/ (XXXL|XXL|XL|XS|S|M|L)$/';
$title_product_name_remove_patterns[] = '/  /i';


Det ser ud til at alt virker som det skal og det fylder da i hvert fald mindre. Måske kan det forenkles lidt mere??

Hvis du "pludselig ændrer $title_product_name_remove_patterns", så kører du en opdatering af samtlige felter i databasen. Det er ligeyldigt om det tager 10 ms eller 10 sekunder.

Hvis du synes, kan du nu lave det hele som eet udtryk:

  $title_product_name_remove_patterns[]="(første udtryk)|(andet udtryk)";  // osv....

Men det bliver ikke mere læseligt af det.

Jeg formoder du skal bruge resultatet på serveren til generering af noget HTML, for ellers kunne man lægge beregningen ud i browseren i Javascript, hvor "der er tid nok".

hej igen erikj

ja nu er det forsimplet så meget jeg kan, og jeg er tilfreds med det. kan godt se at der er mulighed for at forbedre det yderligere, men iflg speed test så kører hjemmesiden ok hurtigt.

du hjalp med mit spørgsmål, skønt, så smid gerne et svar

og tak for hjælpen :o)

Desværre, men jeg samler slet ikke på point, tak.

OK tak :o)