PHP: Недопустимое смещение строки

Введение

Я знаю, что на это был дан ответ, но Illegal string offset ERROR – это не единственная проблема, которую я вижу здесь. Я верю, что это лучшие способы представить гибкость, которую вы хотите, а также сохранить элементы без такой complexity и использования $GLOBALS .

Вы можете начать с рассмотрения:

• Являются ли глобальные переменные в PHP плохой практикой? Если да, то почему?
• Глобальные функции PHP
• Почему глобальное состояние – дьявол, и как его избежать

Быстрый обзор

 $input = new Input(); <-- You add to initiate a class $server_name = $input->server("SERVER_NAME"); ^ ^ ^ | | | variable | | Variable | Variable 

Не уверен, что мешает вам просто использовать

  $_SERVER['SERVER_NAME'] = makeSave($_SERVER['SERVER_NAME']); ^ |- I guess this is what you want to introduce 

Успение – вы хотите феоксичность

Предположим, что вам нужна гибкость, а также рекурсия, тогда ваш вызов класса может быть таким же гибким, как:

 print_r($input->SERVER_NAME); | print_r($input['SERVER_NAME']); |----- Would Produce same result print_r($input->SERVER_NAME()); | 

Если это та гибкость, которую вы хотите, я бы __call что вы объедините __get , __call и ArrayAccess целом …

Позволяет себе представить

 $var = array(); $var["name"] = "<b>" . $_SERVER['SERVER_NAME'] . "</b>"; $var["example"]['xss'] = '<IMG SRC=javascript:alert("XSS")>'; $var["example"]['sql'] = "x' AND email IS NULL; --"; $var["example"]['filter'] = "Let's meet 4:30am Ât the \tcafé\n"; $_SERVER['SERVER_NAME'] = $var ; // Just for example 

Теперь вернемся к вашему формату

 $makeSave = new MakeSafe(MakeSafe::SAVE_XSS | MakeSafe::SAVE_FILTER); $input = new Input($_SERVER, $makeSafe); //You can print_r($input->SERVER_NAME); //Or print_r($input['SERVER_NAME']); //Or print_r($input->SERVER_NAME()); 

Все они будут выводить

 Array ( [0] => <b>localhost</b> [1] => Array ( [0] => <IMG SRC=javascript:alert("XSS")> [1] => x' AND email IS NULL; -- [2] => Let's meet 4:30am Ât the 	café
 ) ) 

Смотрите Live DEMO

Изменен класс INPUT

 class INPUT implements \ArrayAccess { private $request = array(); private $makeSafe; public function __construct(array $array, MakeSafe $makeSafe) { $this->request = $array; $this->makeSave = $makeSafe; } function __get($offset) { return $this->offsetGet($offset); } function __call($offset, $value) { return $this->offsetGet($offset); } public function setRequest(array $array) { $this->request = $array; } public function offsetSet($offset, $value) { trigger_error("Error: SUPER GLOBAL data cannot be modified"); } public function offsetExists($offset) { return isset($this->request[$offset]); } public function offsetUnset($offset) { unset($this->request[$offset]); } public function offsetGet($offset) { return isset($this->request[$offset]) ? $this->makeSave->parse($this->request[$offset]) : null; } } с class INPUT implements \ArrayAccess { private $request = array(); private $makeSafe; public function __construct(array $array, MakeSafe $makeSafe) { $this->request = $array; $this->makeSave = $makeSafe; } function __get($offset) { return $this->offsetGet($offset); } function __call($offset, $value) { return $this->offsetGet($offset); } public function setRequest(array $array) { $this->request = $array; } public function offsetSet($offset, $value) { trigger_error("Error: SUPER GLOBAL data cannot be modified"); } public function offsetExists($offset) { return isset($this->request[$offset]); } public function offsetUnset($offset) { unset($this->request[$offset]); } public function offsetGet($offset) { return isset($this->request[$offset]) ? $this->makeSave->parse($this->request[$offset]) : null; } } 

Сделайте способ сохранения классом

 class MakeSafe { const SAVE_XSS = 1; const SAVE_SQL = 2; const SAVE_FILTER_HIGH = 4; const SAVE_FILTER_LOW = 8; const SAVE_FILTER = 16; private $options; function __construct($options) { $this->options = $options; } function escape($value) { if ($value = @mysql_real_escape_string($value)) return $value; $return = ''; for($i = 0; $i < strlen($value); ++ $i) { $char = $value[$i]; $ord = ord($char); if ($char !== "'" && $char !== "\"" && $char !== '\\' && $ord >= 32 && $ord <= 126) $return .= $char; else $return .= '\\x' . dechex($ord); } return $return; } function parse($mixed) { if (is_string($mixed)) { $this->options & self::SAVE_XSS and $mixed = htmlspecialchars($mixed, ENT_QUOTES, 'UTF-8'); $this->options & self::SAVE_SQL and $mixed = $this->escape($mixed); $this->options & self::SAVE_FILTER_HIGH and $mixed = filter_var($mixed, FILTER_SANITIZE_STRING, FILTER_FLAG_ENCODE_HIGH); $this->options & self::SAVE_FILTER_LOW and $mixed = filter_var($mixed, FILTER_SANITIZE_STRING, FILTER_FLAG_ENCODE_LOW); $this->options & self::SAVE_FILTER and $mixed = filter_var($mixed, FILTER_SANITIZE_STRING, FILTER_FLAG_ENCODE_HIGH | FILTER_FLAG_ENCODE_LOW); return $mixed; } if (is_array($mixed)) { $all = array(); foreach ( $mixed as $data ) { $all[] = $this->parse($data); } return $all; } return $mixed; return $this->final; } } 

Вывод

Я сказал, что знаю, что на это был дан ответ, но я надеюсь, что это поможет кому-то еще не писать такой код, как ваш …

PS: Это также зафиксировало ваш PHP Предупреждение: неправильное смещение строки ERROR

Я нашел здесь свой ответ. По-видимому, линия

 return $this->$name[ $this->clean__makesafe_key( $arguments[0] ) ] = $this->clean__makesafe_value( $GLOBALS[ $_name_of_superglobal ][ $arguments[0] ], array(), true ); 

должен выглядеть следующим образом:

 return $this->{$name}[ $this->clean__makesafe_key( $arguments[0] ) ] = $this->clean__makesafe_value( $GLOBALS[ $_name_of_superglobal ][ $arguments[0] ], array(), true ); 

из-за приоритета.