Re: [swift-evolution] [Review] SE-0167: Swift Encoders

[piotr gorzelany via swift-evolution]

For me it boils down to a question if Data is the best type to represent
JSON. Data is really generic, you could serialize an UIImage into data and
pass it to the decoding function.

Yes, JSON is not a very strong type but that is the nature of JSON, it is a
dynamic data structure like a dictionary. But giving it a type gives us
future flexibility to extend it if needed.

Also, raw JSON manipulation is common practice since the server does not
always give you the perfect JSON to deserialize into a model object.

Say you have a model object with some nested objects in it. To initialize
them from JSON you will need to full JSON for all the objects. But the
common practice is that the server will return only the full top level
object and IDs for the nest d objects so you fetch them separately.

W dniu pon., 10.04.2017 o 22:47 Jordan Rose <jordan_r...@apple.com>
napisał(a):

>
> On Apr 10, 2017, at 02:52, David Hart via swift-evolution <
> swift-evolution@swift.org> wrote:
>
>
> On 10 Apr 2017, at 11:36, piotr gorzelany via swift-evolution <
> swift-evolution@swift.org> wrote:
>
> This is a really great proposal. As an iOS developer I work with JSON
> almost every day since most mobile apps communicate with a backend through
> JSON messages. It's good to see that better JSON support is coming to
> Foundation so we don't have to rely on third party libraries.
>
> That being said, there is one thing I don't like which is that the JSON
> encoding function returns Data and the decoding function takes Data.
>
> It would be really great if the Foundation team could introduce a
> dedicated type of JSON.
> There are several advantages of working with a dedicated type.
> - The underlying data is always valid JSON
> - You can extend this type in the future with helper methods for
> manipulating JSON
> - It makes it explicit what you are dealing with
>
> A good analogy is the URL type. You could represent an URL with a String
> or Data type, but by introducing a dedicated type you have the full
> advantages mentioned above. Data on the other hand is like a generic
> container which you cannot easily extend with URL or JSON specific methods.
>
> Having a dedicated JSON type is also one of the reasons third party
> libraries like SwiftyJSON are so popular. It makes it super easy to
> manipulate JSON structures. And sometimes developers like would like to
> manipulate JSON directly.
>
>
> I don’t think it would be a good thing to have this in Foundation.
> Foundation needs to push developers towards adopting best practices, and
> manipulating JSON directly instead of going through Codable’s strong-typing
> is not necessarily recommended. I’m not saying it doesn’t have its uses.
> But it’s usefulness is definitely narrow now once we have Codable. I think
> this is something that should stay in a third-party library.
>
>
> I haven't thought *heavily* about this, but the thoughts that I have say
> that JSON just isn't a good type in Swift. It's not an efficient in-memory
> representation and it's not a convenient structural representation (because
> of the use of either 'Any' or 'JSON' in the recursive position). I'll admit
> that sometimes you're handed some JSON and you want to extract a little bit
> of information out of it without building a typed representation of the
> whole thing, but that still seems like something that doesn't need to be
> the common path.
>
> Jordan
>
>
___
swift-evolution mailing list
swift-evolution@swift.org
https://lists.swift.org/mailman/listinfo/swift-evolution

Re: [swift-evolution] [Proposal] Foundation Swift Encoders

[piotr gorzelany via swift-evolution]

https://github.com/apple/swift-evolution/blob/master/proposals/0167-swift-encoders.md

This is a really great proposal. As an iOS developer I work with JSON
almost every day since most mobile apps communicate with a backend through
JSON messages. It's good to see that better JSON support is coming to
Foundation so we don't have to rely on third party libraries.

That being said, there is one thing I don't like which is that the JSON
encoding function returns Data and the decoding function takes Data.

It would be really great if the Foundation team could introduce a dedicated
type of JSON.
There are several advantages of working with a dedicated type.
- The underlying data is always valid JSON
- You can extend this type in the future with helper methods for
manipulating JSON
- It makes it explicit what you are dealing with

A good analogy is the URL type. You could represent an URL with a String or
Data type, but by introducing a dedicated type you have the full advantages
mentioned above. Data on the other hand is like a generic container which
you cannot easily extend with URL or JSON specific methods.

Having a dedicated JSON type is also one of the reasons third party
libraries like SwiftyJSON are so popular. It makes it super easy to
manipulate JSON structures. And sometimes developers like would like to
manipulate JSON directly.

If the proposal would pass in the current form, I would still use
SwiftyJSON for manipulating JSON which I would rather avoid and have this
all sorted on the Foundation level.

Just my two cents from the perspective of an iOS developer. JSON handling
is really important to us so I hope you make it great!

>* On Apr 3, 2017, at 1:31 PM, Itai Ferber via swift-evolution at swift.org > wrote:
*>* Hi everyone,
*> >* With feedback from swift-evolution and additional internal
review, we've pushed updates to this proposal, and to the Swift
Archival & Serialization proposal.
*>* Changes to here mostly mirror the ones made to Swift Archival &
Serialization, but you can see a specific diff of what's changed here.
Full content below.
*> >* We'll be looking to start the official review process very soon,
so we're interested in any additional feedback.
*> >* Thanks!
*> >* — Itai*
___
swift-evolution mailing list
swift-evolution@swift.org
https://lists.swift.org/mailman/listinfo/swift-evolution

[swift-evolution] [Review] SE-0167: Swift Encoders

[piotr gorzelany via swift-evolution]

This is a really great proposal. As an iOS developer I work with JSON
almost every day since most mobile apps communicate with a backend through
JSON messages. It's good to see that better JSON support is coming to
Foundation so we don't have to rely on third party libraries.

That being said, there is one thing I don't like which is that the JSON
encoding function returns Data and the decoding function takes Data.

It would be really great if the Foundation team could introduce a dedicated
type of JSON.
There are several advantages of working with a dedicated type.
- The underlying data is always valid JSON
- You can extend this type in the future with helper methods for
manipulating JSON
- It makes it explicit what you are dealing with

A good analogy is the URL type. You could represent an URL with a String or
Data type, but by introducing a dedicated type you have the full advantages
mentioned above. Data on the other hand is like a generic container which
you cannot easily extend with URL or JSON specific methods.

Having a dedicated JSON type is also one of the reasons third party
libraries like SwiftyJSON are so popular. It makes it super easy to
manipulate JSON structures. And sometimes developers like would like to
manipulate JSON directly.

If the proposal would pass in the current form, I would still use
SwiftyJSON for manipulating JSON which I would rather avoid and have this
all sorted on the Foundation level.

Just my two cents from the perspective of an iOS developer. JSON handling
is really important to us so I hope you make it great!



Hello Swift community,

The review of SE-0167 "Swift Encoders" begins now and runs through
April 12, 2017. The proposal is available here:
https://github.com/apple/swift-evolution/blob/master/proposals/0167-swift-encoders.md
Note that this proposal is closely related to (and dependent on)
SE-0166: Swift Archival & Serialization
.
Please read and review that proposal as well!

Reviews are an important part of the Swift evolution process. All
reviews should be sent to the swift-evolution mailing list at
https://lists.swift.org/mailman/listinfo/swift-evolution

or, if you would like to keep your feedback private, directly to the
review manager. When replying, please try to keep the proposal link at
the top of the message:

Proposal link:
https://github.com/apple/swift-evolution/blob/master/proposals/0167-swift-encoders.md
Reply text
Other replies
 
What
goes into a review?

The goal of the review process is to improve the proposal under review
through constructive criticism and, eventually, determine the
direction of Swift. When writing your review, here are some questions
you might want to answer in your review:

What is your evaluation of the proposal?
Is the problem being addressed significant enough to warrant a change to Swift?
Does this proposal fit well with the feel and direction of Swift?
If you have used other languages or libraries with a similar feature,
how do you feel that this proposal compares to those?
How much effort did you put into your review? A glance, a quick
reading, or an in-depth study?
More information about the Swift evolution process is available at
https://github.com/apple/swift-evolution/blob/master/process.md

Thank you,

-Doug
___
swift-evolution mailing list
swift-evolution@swift.org
https://lists.swift.org/mailman/listinfo/swift-evolution