PHP: Функция в preg_replace говорит, что функция уже была объявлена ​​

Question

Функция:

function doSomething($url){ 
    $url = "<a href=\"{$url}\" target=\"blank\" title=\"{$url}\">{$url}</a>"; 
    return $url; 
} 

Замена

$content = preg_replace("#(http:\/\/+[^\s]+)#ie","doSomething('$1')", $content); 

Проблема:

Fatal error: Cannot redeclare doSomething() (previously declared in http://example.com/test.php:69) in http://example.com/test.php on line 69

Примечание: Текущая функция не представляет мою действительную функцию, я знаю, что для этой ситуации мне не нужны никакие функции, но в моем реальном коде мне нужно. Но это также лучший пример.

Answer 1

Ну, это потому, что функция уже была определена в предыдущем вызове функции (это опасность при объявлении функции внутри другой функции). Есть несколько вариантов.

условно объявить функцию

if (!function_exists('doSomething')) { 
    function doSomething($url)... 
} 

объявляя анонимной функции:

PHP 5.3+:

$callback = function($url) { 
    //... 
} 

PHP 5.2+:

$callback = create_function('$url', '//...'); 

Использование класса:

class foo { 
    public function doReplace($string) { 
     $callback = array($this, 'doSomething'); 
     // Do your matching here 
    } 
    public function doSomething($url) { 
     //... 
    } 
} 

Кроме того, я бы предложил не использовать модификатор e для регулярных выражений (это просто не нужно, и это в основном просто eval, который, как правило, рассматривается как зло). Вы должны вместо этого просто использовать preg_replace_callback:

Предполагая $callback действительный обратный вызов:

$callback = function($match) { 
    $url = $match[1]; 
    //... Do stuff here 
} 
$string = preg_replace_callback($regex, $callback, $string); 

Answer 2

Ну ... сообщение об ошибке говорит о проблеме. У вас есть объявление для функции doSomething() несколько раз.

Проверьте строку 69 из test.php и попробуйте прокомментировать объявление.

Answer 3

Как говорится, йоЗотеЬЫпд уже определен. Использовать doSomething2 :)