Function

[Sirus]

Hi, I have tried unsuccessfully to create a function that will strip the forward slashes ("/") and periods (".") from a string. Could someone please help me out with the correct way to do it?

Thanks,
-Jeff

[seanf]

Here is a simple function:

<?php

    function seanstrip($string) {

        $string = str_replace(".", "", $string);
        $string = str_replace("/", "", $string);

        return $string;

    }


    $strip = seanstrip("This is a full stop. and this is a forward slash /");

    echo $strip;


?>

Sean

[Sirus]

Hey Sean, thanks for your help

-Jeff

[Paul S]

Here's a shorter version:

<?php
    function paulstrip($string) {
      return preg_replace("/\.|\//", '', $string);
    }
?>

You need to escape the dot character and the backslash(. = any character, \. = just a dot).

[seanf]

I am going to have to pull rank (as a mentor)

1) No you don't need to escape the dot

2) There will be no paulstrip functions around here, they will be seanstrip or nothing!

Sean

[Paul S]

Originally posted by seanf
I am going to have to pull rank (as a mentor)
1) No you don't need to escape the dot

mm, yes you have. Check the examples here

2) There will be no paulstrip functions around here, they will be seanstrip or nothing!

NOOO!!!!!

[seanf]

Originally posted by Paul S
mm, yes you have. Check the examples here

Please try this:

<?php

    $string = "This wont be replaced >. but this will >\.";

    $string = str_replace("\.", "", $string);

    echo $string;

?>

This is because str_replace is matching \. not .

Sean

[Paul S]

Wops!!, I was thinking all the time in regular expressions

[edit]Function above fixed

[DR_LaRRY_PEpPeR]

i find it's faster (literally) to do it like Paul's code - looking through the string once is faster than looking through it twice w/ str_replace(). actually, if you want to get down to the details (or something...), this would be the best:

$txt = preg_replace('#[./]+#', '', $txt); 

it's better to use a character class than alternation w/ a pipe.