Как исправить ошибку «PHP Warning: array_map() expects parameter 1 to be a valid callback, first array member is not a valid class name or object»?

Question

[Петр Волханов]

Есть вот такой кусок PHP кода:

if(isset($_POST)){
      if(!isset($_POST['ft'])){
      $_POST = @array_map(array($database->connection, 'real_escape_string'), $_POST);
      $_POST = array_map('htmlspecialchars', $_POST);
      }
    }
      
    $rsargs=$_GET['rsargs'];
    $_GET = array_map(array($database->connection, 'real_escape_string'), $_GET);
    $_GET = array_map('htmlspecialchars', $_GET);
    
    $_GET['rsargs']=$rsargs;
    $_COOKIE = array_map(array($database->connection, 'real_escape_string'), $_COOKIE);
    $_COOKIE = array_map('htmlspecialchars', $_COOKIE);

Везде, где в функцию
array_map()

вложена функция
array()

происходит ошибка:

PHP Warning: array_map() expects parameter 1 to be a valid callback, first array member is not a valid class name or object

Как мне убрать эту ошибку? Или достаточно в начале поставить @?

Comments

[Петр Волханов]

Есть такое подозрение, что ошибка именно в подключении к БД ($database->connection). Как думаете?

Answer 1

[Николай Васильчук]

Ну говнокод ведь лютый

Comments

[Петр Волханов]

@Anonym Так посоветуйте, пожалуйста, как переписать так, чтобы говнокода не было...

[Николай Васильчук]

1. Применять array_map с нативной функцией к $_POST (и к $_GET, и к $_COOKIE) - глупо. Во-первых, $_POST может быть многомерным массивом, а во-вторых, там могут быть бинарные данные.
2. Передавать в array_map первым параметром array($database->connection, 'real_escape_string') - что это? Откуда в объекте $database->connection возьмется метод real_escape_string?
3. Переопределять суперглобальные системные массивы - плохой тон. Другой разработчик будет в шоке - данные не передавал, а в $_POST что-то лежит.

Answer 2

[toxa82]

$database->connection должно быть объектом у которого есть метод real_escape_string. Вполне может быть что у вас не подключается к БД и соответственно $database->connection пустой.

Answer 3

[Андрей]

Думаю, этот код не нужен. Данные должны эскейпиться по месту использования (вставки в данном случае), а не по месту получения (глобальные массивы). Хороший пример, как это делать в кустарных условиях (за рамками фреймворков в смысле) - dklab.ru/lib/DbSimple

Ну и да - очевидно, что $database->connection - не объект в вашем случае.

Answer 4

[Алексей Жуков]

одно из двух. либо $database->connection не является объектом и соответственно в нём нет метода real_escape_string либо является но в нём нет метода real_escape_string

сделайте var_dump для $database->connection и посмотрите что это. если всё таки объект то проверьте есть ли метод, например так: if (method_exists($obj, 'method') echo 'Метод есть'