• Serinus@lemmy.world
    link
    fedilink
    arrow-up
    35
    arrow-down
    4
    ·
    2 days ago

    OOP is great, and can be much simpler than what you’ve seen.

    It’s Java culture that’s terrible. The word “Factory” is a code smell all on its own.

    Just like any tool, don’t overuse it. Don’t force extra layers of abstraction.

      • xigoi@lemmy.sdf.org
        link
        fedilink
        English
        arrow-up
        1
        arrow-down
        1
        ·
        edit-2
        1 hour ago

        It means you’re compensating for the lack of optional/named parameters in your language.

        • Traister101@lemmy.today
          link
          fedilink
          arrow-up
          1
          ·
          17 hours ago

          Eh? How’s that work. I’m not going to sit here and say there isn’t too many factories in Java but as a concept it’s extremely useful. You hand off a “factory” to something which actually creates the object. This is really useful in for example serialization. How so? You could register factories (mapped to some sort of ID) which get passed the serialized data and return some sort of created object. Now the core serialization code doesn’t know nor care how exactly any particular type gets serialized. Pretty nifty huh?

          Some languages have better ways to encapsulate this functionality but that’s what the factory concept is

    • GissaMittJobb@lemmy.ml
      link
      fedilink
      arrow-up
      9
      ·
      2 days ago

      I’ve realized that Factories are actually kind of fine, in particular when contexualized as being the equivalent of partials from the world of functionals.

      • Kache@lemm.ee
        link
        fedilink
        arrow-up
        5
        ·
        edit-2
        1 day ago

        IMO factory functions are totally fine – I hesitate to even give them a special name b/c functions that can return an object are not special.

        However I think good use cases for Factory classes (and long-lived stateful instances of) are scarce, often being better served using other constructs.

      • Serinus@lemmy.world
        link
        fedilink
        arrow-up
        7
        ·
        2 days ago

        I have never seen them used well. I expect there IS some use case out there where it makes sense but I haven’t seen it yet. So many times I’ve seen factories that can only return one type. So why did you use a factory? And a factory that returns more than one type is 50/50 to be scary.

        Yeah, I went through the whole shape examples thing in school. The OOP I was taught in school was bullshit.

        Make it simpler. Organizing things into classes is absolutely fine. Seven layers of abstraction is typically not fine.

        • GissaMittJobb@lemmy.ml
          link
          fedilink
          arrow-up
          8
          ·
          2 days ago

          Consider the following: You have a class A that has a few dependencies it needs. The dependencies B and C never change, but D will generally be different for each time the class needs to be used. You also happen to be using dependency injection in this case. You could either:

          • Inject the dependencies B and C for any call site where you need an instance of A and have a given D, or
          • Create an AFactory, which depends on B and C, having a method create with a parameter D returning A, and then inject that for all call sites where you have a given D.

          This is a stripped example, but one I personally have both seen and productively used frequently at work.

          In this case the AFactory could practically be renamed PartialA and be functionally the same thing.

          You could also imagine a factory that returns different implementations of a given interface based on either static (B and C in the previous example) or dynamic dependencies (D in the previous example).

      • skulbuny@sh.itjust.works
        link
        fedilink
        English
        arrow-up
        1
        ·
        edit-2
        2 days ago

        I always call my little helper higher order functions (intended to be partially applied) factories :)