Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:100923
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.83.54 as permitted sender)
To: Sara Golemon <pollita@php.net>, PHP internals <internals@lists.php.net>
References: <CAESVnVpUqpLpKW5X2k7v-nMUxCUhBPQ6P0tEBP9_zNdo63309w@mail.gmail.com>
Message-ID: <b0942929-b7a4-7be8-674f-982fdf40a6b7@gmail.com>
Date: Mon, 23 Oct 2017 23:32:44 -0700
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0)
 Gecko/20100101 Thunderbird/52.4.0
MIME-Version: 1.0
In-Reply-To: <CAESVnVpUqpLpKW5X2k7v-nMUxCUhBPQ6P0tEBP9_zNdo63309w@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] json(Raw)Serializable?
From: smalyshev@gmail.com (Stanislav Malyshev)

Hi!

> It's not terribly unreasonable IMO, but before I just writeup the RFC
> as described (jsonRawSerialize taking preceedence over jsonSerialize),
> I thought I'd ask for opinions on the specifics.
> 
> In psuedo-code:
> 
> if (is_object($obj)) {
>   if ($obj implements JsonRawSerializable) {
>     // use $obj->jsonRawSerialize() as is.
>   } elseif ($obj implements JsonSerializble) {
>     // use json_encode($obj->jsonSerialize())
>   } else {
>     // Serialize the object's public properties as a key/value map
>   }
> }

I'm not sure I feel very comfortable with having specialized serialize
interfaces for every format, yet more with having more than one of them.

There's also validation problem - if we don't ensure this is valid JSON,
then whole serialization setup is broken, and who knows which
consequences this will bring.

Also, I'm not sure what is the case for fixed JSON serialization - cases
where the data is completely static and yet you still need to serialize
it is pretty rare IMHO. Yes, we save some cycles on serializing such
things, but how often that happens, really? If it's static, why not just
set it on loading? Maybe I miss some reasonable use case, but so far
sounds pretty exotic to me.


-- 
Stas Malyshev
smalyshev@gmail.com