Meme transcription: Panel 1. Two images of JSON, one is the empty object, one is an object in which the key name maps to the value null. Caption: “Corporate needs you to find the difference between this picture and this picture”

Panel 2. The Java backend dev answers, “They’re the same picture.”

  • expr
    8·
    6 months ago

    Zalando explicitly forbids it in their RESTful API Guidelines, and I would say their argument is a very good one.

    Basically, if you want to provide more fine-grained semantics, use dedicated types for that purpose, rather than hoping every API consumer is going to faithfully adhere to the subtle distinctions you’ve created.

    • masterspace@lemmy.caEnglish
      62·
      6 months ago

      They’re not subtle distinctions.

      There’s a huge difference between checking whether a field is present and checking whether it’s value is null.

      If you use lazy loading, doing the wrong thing can trigger a whole network request and ruin performance.

      Similarly when making a partial change to an object it is often flat out infeasible to return the whole object if you were never provided it in the first place, which will generally happen if you have a performance focused API since you don’t want to be wasting huge amounts of bandwidth on unneeded data.

      • expr
        11·
        6 months ago

        The semantics of the API contract is distinct from its implementation details (lazy loading).

        Treating null and undefined as distinct is never a requirement for general-purpose API design. That is, there is always an alternative design that doesn’t rely on that misfeature.

        As for patches, while it might be true that JSON Merge Patch assigns different semantics to null and undefined values, JSON Merge Patch is a worse version of JSON Patch, which doesn’t have that problem, because like I originally described, the semantics are explicit in the data structure itself. This is a transformation that you can always apply.

        • masterspace@lemmy.caEnglish
          31·
          6 months ago

          No there isn’t.

          Tell me how you partially change an object.

          Object User :

          { Name: whatever, age: 0}

          Tell me how you change the name without knowing the age. You fundamentally cannot, meaning that you either have to shuttle useless information back and forth constantly so that you can always patch the whole object, or you have to create a useless and unscalable number of endpoints, one for every possible field change.

          As others have roundly pointed out, it is asinine to generally assume that undefined and null are the same thing, and no, it flat out it is not possible to design around that, because at a fundamental level those are different statements.

          • expr
            12·
            6 months ago

            As I already said, it’s very simple with JSON Patch:

            [
  { *op": "replace", "path": "/Name™, "value": "otherName"}
]

            Good practice in API design is to permissively accept either undefined or null to represent optionality with same semantics (except when using JSON Merge Patch, but JSON Patch linked above should be preferred anyway).

            • masterspace@lemmy.caEnglish
              34·
              6 months ago

              I.e. waste a ton of bandwidth sending a ridiculous amount of useless data in every request, all because your backend engineers don’t know how to program for shit.

              Gotcha.

              • expr
                2·
                6 months ago

                It’s about making APIs more flexible, permissive, and harder to misuse by clients. It’s a user-centric approach to API design. It’s not done to make it easier on backend. If anything, it can take extra effort by backend developers.

                But you’d clearly prefer vitriol to civil discourse and have no interest in actually learning anything, so I think my time would be better spent elsewhere.