From a8dcaddec5ed190aa0644edd3bc4800aad74204c Mon Sep 17 00:00:00 2001 From: jorgecc Date: Mon, 7 Nov 2022 19:47:01 -0300 Subject: [PATCH] 2.3 --- README.md | 20 ++- composer.json | 3 + examples/example_decode.php | 2 +- examples/example_decode_unquoted.php | 2 +- src/Services_JSON.php | 182 +++++++++++++++------------ tests/FirstTest.php | 67 ++++++++++ 6 files changed, 192 insertions(+), 84 deletions(-) create mode 100644 tests/FirstTest.php diff --git a/README.md b/README.md index 360640c..276fc83 100644 --- a/README.md +++ b/README.md @@ -53,12 +53,12 @@ See the folder examples for further examples ### Decode -Decode transform (de-codified) a JSON string into a stdclass or an associative array +Decode a JSON string into a stdclass or an associative array ```php $json='{"hello":{"a":2,"b":3},"world":[1,2,3,"aaa"]}'; var_dump(Services_JSON::decode($json)); // as stdclass -var_dump(Services_JSON::decode($json,Services_JSON::SERVICES_JSON_AS_ARRAY)); // as array +var_dump(Services_JSON::decode($json,Services_JSON::GET_ARRAY)); // as array ``` It also works with unquoted keys @@ -66,9 +66,18 @@ It also works with unquoted keys $json='{hello:{a:2,b:3},world:[1,2,3,"aaa","bbbb"]}'; // the keys are unquoted. var_dump(Services_JSON::decode($json)); // as stdclass -var_dump(Services_JSON::decode($json,Services_JSON::SERVICES_JSON_AS_ARRAY)); // as array +var_dump(Services_JSON::decode($json,Services_JSON::GET_ARRAY)); // as array ``` +It also works (with the flag Services_JSON::DECODE_FIX_ROOT) where the string misses [] and {} at the start of the code + +```php +Services_JSON::decode('1,2,3',Services_JSON::GET_ARRAY | Services_JSON::DECODE_FIX_ROOT); // returns [1,2,3] +Services_JSON::decode('"k1":"v1", k2:2',Services_JSON::GET_ARRAY | Services_JSON::DECODE_FIX_ROOT) // returns [ 'k1' => 'v1','k2'=>2] +``` + + + ### Encode Encode transform a value (array, object, primitive value, etc.) into a json expression (a string) @@ -83,7 +92,10 @@ var_dump(Services_JSON::encode($obj)); // encode an object ## Changelog -* 2.0 +* 2.3 + * Fixed a typo with a comment. + * added phpunit. The entire code is tested but special codification. +* 2.2 * Now the library is static, so you can call the methods without creating an instance. * If you want to work with the non-static library, then install 1.1 * 1.1 diff --git a/composer.json b/composer.json index 3aa9888..da5b87c 100644 --- a/composer.json +++ b/composer.json @@ -29,5 +29,8 @@ }, "suggest": { "ext-mbstring": "*" + }, + "require-dev": { + "phpunit/phpunit": "^9.5" } } diff --git a/examples/example_decode.php b/examples/example_decode.php index cfb71b3..32913b3 100644 --- a/examples/example_decode.php +++ b/examples/example_decode.php @@ -10,5 +10,5 @@ var_dump(Services_JSON::decode($json)); echo ""; echo "Decode as array:
"; echo "
";
-var_dump(Services_JSON::decode($json, Services_JSON::SERVICES_JSON_AS_ARRAY));
+var_dump(Services_JSON::decode($json, Services_JSON::GET_ARRAY));
 echo "
"; diff --git a/examples/example_decode_unquoted.php b/examples/example_decode_unquoted.php index d9ea133..92513b2 100644 --- a/examples/example_decode_unquoted.php +++ b/examples/example_decode_unquoted.php @@ -16,5 +16,5 @@ var_dump(Services_JSON::decode($json)); echo ""; echo "Decode as array:
"; echo "
";
-var_dump(Services_JSON::decode($json,Services_JSON::SERVICES_JSON_AS_ARRAY));
+var_dump(Services_JSON::decode($json,Services_JSON::GET_ARRAY));
 echo "
"; diff --git a/src/Services_JSON.php b/src/Services_JSON.php index a10a718..d192561 100644 --- a/src/Services_JSON.php +++ b/src/Services_JSON.php @@ -1,5 +1,5 @@ - * // create a new instance of Services_JSON - * $json = new Services_JSON(); * * // convert a complexe value to JSON notation, and send it to the browser * $value = array('foo', 'bar', array(1, 2, 'baz'), array(3, array(4))); - * $output = $json->encode($value); + * $output = Services_JSON::encode($value); * * print($output); * // prints: ["foo","bar",[1,2,"baz"],[3,[4]]] * * // accept incoming POST data, assumed to be in JSON notation * $input = file_get_contents('php://input', 1000000); - * $value = $json->decode($input); + * $value = Services_JSON::decode($input); * */ class Services_JSON { - public const SERVICES_JSON_SLICE = 1; - public const SERVICES_JSON_IN_STR = 2; - public const SERVICES_JSON_IN_ARR = 3; - public const SERVICES_JSON_IN_OBJ = 4; - public const SERVICES_JSON_IN_CMT = 5; + protected const SLICE = 1; + protected const IN_STR = 2; + protected const IN_ARR = 3; + protected const IN_OBJ = 4; + protected const IN_CMT = 5; + /** @var int the value returned will be an array instead of a stdclass */ + public const GET_ARRAY = 16; + /** @var int supresses the errors and replaces by a null */ + public const SUPPRESS_ERRORS = 32; + /** @var int supports to-json */ + public const USE_TO_JSON = 64; + /** @var int If the fix value is broken, then it adds [] or {} automatically */ + public const DECODE_FIX_ROOT = 128; - public const SERVICES_JSON_AS_ARRAY = 16; - public const SERVICES_JSON_SUPPRESS_ERRORS = 32; - public const SERVICES_JSON_USE_TO_JSON = 64; - - protected static $use=0; + protected static $use = 0; // private - cache the mbstring lookup results.. protected static $_mb_strlen = false; protected static $_mb_substr = false; @@ -106,7 +108,7 @@ class Services_JSON protected static function init(?int $use = null): void { - if($use!==null) { + if ($use !== null) { self::$use = $use; } self::$_mb_strlen = function_exists('mb_strlen'); @@ -134,16 +136,16 @@ class Services_JSON switch (true) { case ((0x7F & $bytes) == $bytes): // this case should never be reached, because we are in ASCII range - // see: http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see: https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 return chr(0x7F & $bytes); case (0x07FF & $bytes) == $bytes: // return a 2-byte UTF-8 character - // see: http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see: https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 return chr(0xC0 | (($bytes >> 6) & 0x1F)) . chr(0x80 | ($bytes & 0x3F)); case (0xFFFF & $bytes) == $bytes: // return a 3-byte UTF-8 character - // see: http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see: https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 return chr(0xE0 | (($bytes >> 12) & 0x0F)) . chr(0x80 | (($bytes >> 6) & 0x3F)) . chr(0x80 | ($bytes & 0x3F)); @@ -170,17 +172,17 @@ class Services_JSON switch (self::strlen8($utf8)) { case 1: // this case should never be reached, because we are in ASCII range - // see: http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see: https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 return $utf8; case 2: // return a UTF-16 character from a 2-byte UTF-8 char - // see: http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see: https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 return chr(0x07 & (ord($utf8[0]) >> 2)) . chr((0xC0 & (ord($utf8[0]) << 6)) | (0x3F & ord($utf8[1]))); case 3: // return a UTF-16 character from a 3-byte UTF-8 char - // see: http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see: https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 return chr((0xF0 & (ord($utf8[0]) << 4)) | (0x0F & (ord($utf8[1]) >> 2))) . chr((0xC0 & (ord($utf8[1]) << 6)) @@ -201,7 +203,7 @@ class Services_JSON * @return mixed JSON string representation of input var or an error if a problem occurs * @access public */ - public static function encode($var,$use=0) + public static function encode($var, $use = 0) { //header('Content-type: application/json'); self::init($use); @@ -290,7 +292,7 @@ class Services_JSON break; case (($ord_var_c & 0xE0) == 0xC0): // characters U-00000080 - U-000007FF, mask 110XXXXX - // see http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 if ($c + 1 >= $strlen_var) { ++$c; $ascii .= '?'; @@ -308,7 +310,7 @@ class Services_JSON break; } // characters U-00000800 - U-0000FFFF, mask 1110XXXX - // see http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 $char = pack('C*', $ord_var_c, @ord($var[$c + 1]), @ord($var[$c + 2])); @@ -323,7 +325,7 @@ class Services_JSON break; } // characters U-00010000 - U-001FFFFF, mask 11110XXX - // see http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 $char = pack('C*', $ord_var_c, ord($var[$c + 1]), ord($var[$c + 2]), @@ -334,7 +336,7 @@ class Services_JSON break; case (($ord_var_c & 0xFC) == 0xF8): // characters U-00200000 - U-03FFFFFF, mask 111110XX - // see http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 if ($c + 4 >= $strlen_var) { $c += 4; $ascii .= '?'; @@ -356,7 +358,7 @@ class Services_JSON break; } // characters U-04000000 - U-7FFFFFFF, mask 1111110X - // see http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 $char = pack('C*', $ord_var_c, ord($var[$c + 1]), ord($var[$c + 2]), @@ -410,12 +412,12 @@ class Services_JSON return '[' . implode(',', $elements) . ']'; case 'object': // support toJSON methods. - if ((self::$use & self::SERVICES_JSON_USE_TO_JSON) && method_exists($var, 'toJSON')) { + if ((self::$use & self::USE_TO_JSON) && method_exists($var, 'toJSON')) { // this may end up allowing unlimited recursion, so we check the return value to make sure it's // not got the same method. $recode = $var->toJSON(); if (method_exists($recode, 'toJSON')) { - if ((self::$use & self::SERVICES_JSON_SUPPRESS_ERRORS)) { + if ((self::$use & self::SUPPRESS_ERRORS)) { return 'null'; } self::Services_JSON_Error(get_class($var) . @@ -434,7 +436,7 @@ class Services_JSON } return '{' . implode(',', $properties) . '}'; default: - if ((self::$use & self::SERVICES_JSON_SUPPRESS_ERRORS)) { + if ((self::$use & self::SUPPRESS_ERRORS)) { return 'null'; } self::Services_JSON_Error(gettype($var) . " can not be encoded as JSON string"); @@ -486,29 +488,50 @@ class Services_JSON * * @param string $str JSON-formatted string * @param int $use object behavior flags; combine with boolean-OR possible values:
- * - self::SERVICES_JSON_LOOSE_TYPE: loose typing.
- * "{...}" syntax creates associative arrays
- * instead of objects in decode().
- * - self::SERVICES_JSON_SUPPRESS_ERRORS: error suppression.
- * Values which can't be encoded (e.g. resources)
- * appear as NULL instead of throwing errors.
- * By default, a deeply-nested resource will
- * bubble up with an error, so all return values
- * from encode() should be checked with isError()
- * - self::SERVICES_JSON_USE_TO_JSON: call toJSON when serializing objects
- * It serializes the return value from the toJSON call rather
- * than the object it'self, toJSON can return associative arrays,
- * strings or numbers, if you return an object, make sure it does
- * not have a toJSON method, otherwise an error will occur.
+ * - self::SERVICES_JSON_AS_ARRAY: syntax creates associative arrays
+ * instead of objects in decode().
+ * - self::SERVICES_JSON_SUPPRESS_ERRORS: error suppression.
+ * Values which can't be encoded (e.g. resources)
+ * appear as NULL instead of throwing errors.
+ * By default, a deeply-nested resource will
+ * bubble up with an error, so all return values
+ * from encode() should be checked with isError()
+ * - self::SERVICES_JSON_USE_TO_JSON: call toJSON when serializing objects
+ * It serializes the return value from the toJSON call rather
+ * than the object it'self, toJSON can return associative arrays,
+ * strings or numbers, if you return an object, make sure it does
+ * not have a toJSON method, otherwise an error will occur.
+ * -self::SERVICES_JSON_DECODE_FIX_ROOT: Fix the code if the root parenthesis are + * missing
Example: "1,2,3" works as "[1,2,3]" and "a:1,b:2" works as "{a:1,b:2}" + * * @return mixed number, boolean, string, array, or object
* corresponding to given JSON input string.
* See argument 1 to Services_JSON() above for object-output behavior.
* Note that decode() always returns strings
* in ASCII or UTF-8 format!
- * @access public + * @access public + * @noinspection MultiAssignmentUsageInspection */ public static function decode($str, int $use = 0) { + if ($use & self::DECODE_FIX_ROOT) { + $str = trim($str); + $firstChar = $str[0]; + $lastChar = $str[-1]; + if ($firstChar !== '{' && $firstChar !== '[' && $lastChar !== '}' && $lastChar !== ']') { + // fixing a malformed json-u, if the json-u doesn't start with { [ and ends with ] } then it wraps it + // note:: it will fail for a simple value su as "hello", so it will return ["hello"] + $p0 = strpos($str, ':'); // content:2,content2:3 => {content:2,content2:3} + $p0 = $p0 === false ? PHP_INT_MAX : $p0; + $p1 = strpos($str, ','); // 2,3 => [2,3] + $p1 = $p1 === false ? PHP_INT_MAX : $p1; + if ($p0 < $p1) { + $str = '{' . $str . '}'; + } else { + $str = '[' . $str . ']'; + } + } + } self::init($use); return self::decode2($str); } @@ -519,6 +542,7 @@ class Services_JSON * @param $asAssocArray * @return array|bool|float|int|mixed|stdClass|string|null * + * @noinspection PhpUndefinedVariableInspection */ protected static function decode2($str, $asAssocArray = false) { @@ -593,31 +617,31 @@ class Services_JSON break; case ($ord_chrs_c & 0xE0) == 0xC0: // characters U-00000080 - U-000007FF, mask 110XXXXX - //see http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + //see https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 $utf8 .= self::substr8($chrs, $c, 2); ++$c; break; case ($ord_chrs_c & 0xF0) == 0xE0: // characters U-00000800 - U-0000FFFF, mask 1110XXXX - // see http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 $utf8 .= self::substr8($chrs, $c, 3); $c += 2; break; case ($ord_chrs_c & 0xF8) == 0xF0: // characters U-00010000 - U-001FFFFF, mask 11110XXX - // see http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 $utf8 .= self::substr8($chrs, $c, 4); $c += 3; break; case ($ord_chrs_c & 0xFC) == 0xF8: // characters U-00200000 - U-03FFFFFF, mask 111110XX - // see http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 $utf8 .= self::substr8($chrs, $c, 5); $c += 4; break; case ($ord_chrs_c & 0xFE) == 0xFC: // characters U-04000000 - U-7FFFFFFF, mask 1111110X - // see http://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 + // see https://www.cl.cam.ac.uk/~mgk25/unicode.html#utf-8 $utf8 .= self::substr8($chrs, $c, 6); $c += 5; break; @@ -628,23 +652,23 @@ class Services_JSON if (preg_match('/^\[.*]$/s', $str) || preg_match('/^\{.*}$/s', $str)) { // array, or object notation if ($str[0] === '[') { - $stk = [self::SERVICES_JSON_IN_ARR]; + $stk = [self::IN_ARR]; $arr = []; } else { - $stk = [self::SERVICES_JSON_IN_OBJ]; - if (self::$use & self::SERVICES_JSON_AS_ARRAY) { + $stk = [self::IN_OBJ]; + if (self::$use & self::GET_ARRAY) { $obj = []; } else { $obj = new stdClass(); } } - $stk[] = ['what' => self::SERVICES_JSON_SLICE, + $stk[] = ['what' => self::SLICE, 'where' => 0, 'delim' => false]; $chrs = self::substr8($str, 1, -1); $chrs = self::reduce_string($chrs); if ($chrs == '') { - if (reset($stk) == self::SERVICES_JSON_IN_ARR) { + if (reset($stk) == self::IN_ARR) { return $arr; } return $obj; @@ -654,16 +678,16 @@ class Services_JSON for ($c = 0; $c <= $strlen_chrs; ++$c) { $top = end($stk); $substr_chrs_c_2 = self::substr8($chrs, $c, 2); - if (($c == $strlen_chrs) || (($chrs[$c] === ',') && ($top['what'] == self::SERVICES_JSON_SLICE))) { + if (($c == $strlen_chrs) || (($chrs[$c] === ',') && ($top['what'] == self::SLICE))) { // found a comma that is not inside a string, array, etc., // OR we've reached the end of the character list $slice = self::substr8($chrs, $top['where'], ($c - $top['where'])); - $stk[] = ['what' => self::SERVICES_JSON_SLICE, 'where' => ($c + 1), 'delim' => false]; + $stk[] = ['what' => self::SLICE, 'where' => ($c + 1), 'delim' => false]; //print("Found split at {$c}: ".$this->substr8($chrs, $top['where'], (1 + $c - $top['where']))."\n"); - if (reset($stk) == self::SERVICES_JSON_IN_ARR) { + if (reset($stk) == self::IN_ARR) { // we are in an array, so just push an element onto the stack $arr[] = self::decode2($slice); - } elseif (reset($stk) == self::SERVICES_JSON_IN_OBJ) { + } elseif (reset($stk) == self::IN_OBJ) { // we are in an object, so figure // out the property name and set an // element in an associative array, @@ -674,59 +698,61 @@ class Services_JSON // "name":value pair $key = self::decode2($parts[1]); $val = self::decode2(trim(substr($slice, strlen($parts[0])), ", \t\n\r\0\x0B")); - if (self::$use & self::SERVICES_JSON_AS_ARRAY) { + if (self::$use & self::GET_ARRAY) { $obj[$key] = $val; } else { $obj->$key = $val; } - } /** @noinspection NotOptimalRegularExpressionsInspection */ elseif (preg_match('/^\s*(\w+)\s*:/Ui', $slice, $parts)) { + } /** @noinspection NotOptimalRegularExpressionsInspection */ + elseif (preg_match('/^\s*(\w+)\s*:/Ui', $slice, $parts)) { // name:value pair, where name is unquoted $key = $parts[1]; $val = self::decode2(trim(substr($slice, strlen($parts[0])), ", \t\n\r\0\x0B")); - if (self::$use & self::SERVICES_JSON_AS_ARRAY) { + if (self::$use & self::GET_ARRAY) { $obj[$key] = $val; } else { $obj->$key = $val; } } } - } elseif ((($chrs[$c] === '"') || ($chrs[$c] === "'")) && ($top['what'] != self::SERVICES_JSON_IN_STR)) { + } elseif ((($chrs[$c] === '"') || ($chrs[$c] === "'")) && ($top['what'] != self::IN_STR)) { // found a quote, and we are not inside a string - $stk[] = ['what' => self::SERVICES_JSON_IN_STR, 'where' => $c, 'delim' => $chrs[$c]]; + $stk[] = ['what' => self::IN_STR, 'where' => $c, 'delim' => $chrs[$c]]; //print("Found start of string at {$c}\n"); } elseif (($chrs[$c] == $top['delim']) && - ($top['what'] == self::SERVICES_JSON_IN_STR) && - ((self::strlen8(self::substr8($chrs, 0, $c)) - self::strlen8(rtrim(self::substr8($chrs, 0, $c), '\\'))) % 2 != 1)) { + ($top['what'] == self::IN_STR) && + ((self::strlen8(self::substr8($chrs, 0, $c)) - + self::strlen8(rtrim(self::substr8($chrs, 0, $c), '\\'))) % 2 != 1)) { // found a quote, we're in a string, and it's not escaped // we know that it's not escaped becase there is _not_ an // odd number of backslashes at the end of the string so far array_pop($stk); //print("Found end of string at {$c}: ".$this->substr8($chrs, $top['where'], (1 + 1 + $c - $top['where']))."\n"); } elseif (($chrs[$c] === '[') && - in_array($top['what'], [self::SERVICES_JSON_SLICE, self::SERVICES_JSON_IN_ARR, self::SERVICES_JSON_IN_OBJ])) { + in_array($top['what'], [self::SLICE, self::IN_ARR, self::IN_OBJ])) { // found a left-bracket, and we are in an array, object, or slice - $stk[] = ['what' => self::SERVICES_JSON_IN_ARR, 'where' => $c, 'delim' => false]; + $stk[] = ['what' => self::IN_ARR, 'where' => $c, 'delim' => false]; //print("Found start of array at {$c}\n"); - } elseif (($chrs[$c] === ']') && ($top['what'] == self::SERVICES_JSON_IN_ARR)) { + } elseif (($chrs[$c] === ']') && ($top['what'] == self::IN_ARR)) { // found a right-bracket, and we're in an array array_pop($stk); //print("Found end of array at {$c}: ".$this->substr8($chrs, $top['where'], (1 + $c - $top['where']))."\n"); } elseif (($chrs[$c] === '{') && - in_array($top['what'], [self::SERVICES_JSON_SLICE, self::SERVICES_JSON_IN_ARR, self::SERVICES_JSON_IN_OBJ])) { + in_array($top['what'], [self::SLICE, self::IN_ARR, self::IN_OBJ])) { // found a left-brace, and we are in an array, object, or slice - $stk[] = ['what' => self::SERVICES_JSON_IN_OBJ, 'where' => $c, 'delim' => false]; + $stk[] = ['what' => self::IN_OBJ, 'where' => $c, 'delim' => false]; //print("Found start of object at {$c}\n"); - } elseif (($chrs[$c] === '}') && ($top['what'] == self::SERVICES_JSON_IN_OBJ)) { + } elseif (($chrs[$c] === '}') && ($top['what'] == self::IN_OBJ)) { // found a right-brace, and we're in an object array_pop($stk); //print("Found end of object at {$c}: ".$this->substr8($chrs, $top['where'], (1 + $c - $top['where']))."\n"); } elseif (($substr_chrs_c_2 === '/*') && - in_array($top['what'], [self::SERVICES_JSON_SLICE, self::SERVICES_JSON_IN_ARR, self::SERVICES_JSON_IN_OBJ])) { + in_array($top['what'], [self::SLICE, self::IN_ARR, self::IN_OBJ])) { // found a comment start, and we are in an array, object, or slice - $stk[] = ['what' => self::SERVICES_JSON_IN_CMT, 'where' => $c, 'delim' => false]; + $stk[] = ['what' => self::IN_CMT, 'where' => $c, 'delim' => false]; $c++; //print("Found start of comment at {$c}\n"); - } elseif (($substr_chrs_c_2 === '*/') && ($top['what'] == self::SERVICES_JSON_IN_CMT)) { + } elseif (($substr_chrs_c_2 === '*/') && ($top['what'] == self::IN_CMT)) { // found a comment end, and we're in one now array_pop($stk); $c++; @@ -736,10 +762,10 @@ class Services_JSON //print("Found end of comment at {$c}: ".$this->substr8($chrs, $top['where'], (1 + $c - $top['where']))."\n"); } } - if (reset($stk) == self::SERVICES_JSON_IN_ARR) { + if (reset($stk) == self::IN_ARR) { return $arr; } - if (reset($stk) == self::SERVICES_JSON_IN_OBJ) { + if (reset($stk) == self::IN_OBJ) { if ($asAssocArray) { return self::arrayCastRecursive($obj); } diff --git a/tests/FirstTest.php b/tests/FirstTest.php new file mode 100644 index 0000000..2527030 --- /dev/null +++ b/tests/FirstTest.php @@ -0,0 +1,67 @@ +assertEquals( + (object)[ 'k1' => "v1",'k2'=>'v2'] + ,Services_JSON::decode('{"k1":"v1","k2":"v2"}') + ); + $this->assertEquals( + (object)[ 'k1' => "v1\n\t\f\r",'k2'=>'v2'] + ,Services_JSON::decode('{"k1":"v1\n\t\f\r","k2":"v2"}') + ); + $this->assertEquals( + [ 'k1' => 'v1','k2'=>2] + ,Services_JSON::decode('{"k1":"v1","k2":2}',Services_JSON::GET_ARRAY) + ); + $this->assertEquals( + [ 'k1' => 'v1','k2'=>2] + ,Services_JSON::decode('{"k1":"v1","k2":2 /* comment */}',Services_JSON::GET_ARRAY) + ); + $this->assertEquals( + [ 'k1' => 'v1','k2'=>2] + ,Services_JSON::decode('{k1:"v1",k2:2}',Services_JSON::GET_ARRAY) + ); + $this->assertEquals( + [ 'k1' => 'v1','k2'=>2] + ,Services_JSON::decode('"k1":"v1", k2:2', + Services_JSON::GET_ARRAY | Services_JSON::DECODE_FIX_ROOT) + ); + $this->assertEquals( + [ 1,2,3] + ,Services_JSON::decode('1,2,3', + Services_JSON::GET_ARRAY | Services_JSON::DECODE_FIX_ROOT) + ); + $this->assertEquals( + [1] + ,Services_JSON::decode('1', + Services_JSON::GET_ARRAY | Services_JSON::DECODE_FIX_ROOT) + ); + $this->assertEquals( + ['a'=>1] + ,Services_JSON::decode('a:1', + Services_JSON::GET_ARRAY | Services_JSON::DECODE_FIX_ROOT) + ); + } + public function testEncode(): void + { + $this->assertEquals( + '{"k1":"v1","k2":"v2"}' + ,Services_JSON::encode([ 'k1' => 'v1','k2'=>'v2']) + ); + $this->assertEquals( + '{"k1":"v1","k2":[1,2,3]}' + ,Services_JSON::encode((object)[ 'k1' => 'v1','k2'=>[1,2,3]]) + ); + $this->assertEquals( + '{"k\"1":"v1","k2":[1,2,3],"k3":{"k4":{"0":1,"1":2,"k5":[1,2]}}}' + ,Services_JSON::encode((object)[ 'k"1' => 'v1','k2'=>[1,2,3],'k3'=>['k4'=>[1,2,'k5'=>[1,2]]]]) + ); + } + +}