In article <[EMAIL PROTECTED]>, [EMAIL PROTECTED] (Jtjohnston) wrote:
> Suggestions for improvement anyone? > > > $mystring = ereg_replace (".html", "", $mystring); > > > $mystring = ereg_replace (".htm", "", $mystring); > > > $mystring = ereg_replace (" ", "", $mystring); > > > > > > $mystring = ereg_replace ("}", "", $mystring); > > > $mystring = ereg_replace ("{", "", $mystring); > > > $mystring = ereg_replace ("\|", "", $mystring); > > > $mystring = ereg_replace ("\]", "", $mystring); > > > $mystring = ereg_replace ("\[", "", $mystring); > > > $mystring = ereg_replace ("\\\\", "", $mystring); <snip much, more of same> Since you ask... You can save yourself a lot of trouble and a ton of wasted function calls by checking out a POSIX (ereg_*) or PCRE (preg_*) syntax tutorial. When you want to match space or punctuation characters, character classes are a Good Thing(TM). That whole loooooong block of code could be rendered with a single good, concise regex. With the bonus of a lot less escaping. -- CC -- PHP General Mailing List (http://www.php.net/) To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] To contact the list administrators, e-mail: [EMAIL PROTECTED]