• BougieBirdie@lemmy.blahaj.zone
    link
    fedilink
    English
    arrow-up
    71
    arrow-down
    1
    ·
    1 day ago

    Most of my college coursework was around OOP. That said, they actually did a pretty lousy job of explaining it in a practical sense, so since we were left to figure it out ourselves a lot of our assignments ended up looking like this.

    At the end of the program, our capstone project was to build a full stack app. They did a pretty good job simulating a real professional experience: we all worked together on requirements gathering and database design, then were expected to build our own app.

    To really drive home the real world experience, the professor would change the requirements partway through the project. Which is a total dick move, but actually a good lesson on its own.

    Anyway, this app was mostly about rendering a bunch of different views, and something subtly would change that ended up affecting all views. After the fact, the professor would say something to the effect of “If you used good objects, you’ll only have to make the change in one place.”

    This of course is only helpful if you really appreciated the power of OOP and planned it from the start. Most of us were flying by the seat of our pants, so it was usually a ton of work to get those changes in

    • Kache@lemm.ee
      link
      fedilink
      arrow-up
      12
      ·
      11 hours ago

      If you used good objects, you’ll only have to make the change in one place

      IMO that’s generally a retroactive statement because in practice have to be lucky for that to be true. An abstraction along one dimension – even a good one, will limit flexibility in some other dimension.

      Occasionally, everything comes into alignment and an opportunity appears to reliably-ish predict the correct abstraction the future will need.

      Most every other time, you’re better off avoiding the possibility of the really costly bad abstraction by resisting the urge to abstract preemptively.

      • BougieBirdie@lemmy.blahaj.zone
        link
        fedilink
        English
        arrow-up
        1
        ·
        37 minutes ago

        That’s a fair assessment. It’s kind of like the rule for premature optimization: don’t.

        With experience you might get some intuition about when it’s good to lean into inheritance. We were definitely lacking experience at that point though.

        OOP is a pretty powerful paradigm, which means it’s also easy to provide enough rope to hang yourself with. See also just about any other meme here about OOP

    • Solemarc@lemmy.world
      link
      fedilink
      arrow-up
      6
      ·
      13 hours ago

      Same, I always remember this with interfaces and inheritance, shoehorned in BS where I’m only using 1 class anyway and talking to 1 other class what’s the point of this?

      After I graduated as a personal project i made a wiki for a game and I was reusing a lot of code, “huh a parent class would be nice here”.

      In my first Job, I don’t know who’s going to use this thing I’m building but these are the rules: proceeds to implement an interface.

      When I have to teach these concepts to juniors now, this is how I teach them: inheritance to avoid code duplication, interfaces to tell other people what they need to implement in order to use your stuff.

      I wonder why I wasn’t taught it that way. I remember looking at my projects that used this stuff thinking it was just messy rubbish. More importantly, I graduated not understanding this stuff…

      • pfm@scribe.disroot.org
        link
        fedilink
        arrow-up
        5
        arrow-down
        1
        ·
        11 hours ago

        I wouldn’t say that inheritance is for avoiding code duplication. It should be used to express “is a” relationship. An example seen in one of my projects: a mixin with error-handling code for a REST service client used for more than one service has log messages tightly coupled to a particular service. That’s exactly because someone thought it was ok to reuse.

        In my opinion, inheritance makes sense when you can follow Liskov’s principle. Otherwise you should be careful.

        • Solemarc@lemmy.world
          link
          fedilink
          arrow-up
          2
          ·
          9 hours ago

          You’re not wrong but I think when you’re teaching someone just having 1 parent and 1 child class makes for a bad example I generally prefer to use something with a lot of different children.

          My go-to is exporters. We have the exporter interface, the generic exporter, the accounting exporter and the payroll exporter, to explain it.

          At school, the only time I used inheritance was 1 parent (booking) and 1 child (luxury) this is a terrible example imo.

          • pfm@scribe.disroot.org
            link
            fedilink
            arrow-up
            1
            ·
            4 hours ago

            Maybe that example was made terrible because the author couldn’t think of a good ways to show how great this can be. I’m obviously a fan of SOLID, and OCP is exactly why I don’t worry if I have only one class at the beginning. Because I know eventually requirements would change and I’d end up with more classes.

            Some time ago I was asked by a less experienced coworker during a code review why I wrote a particularly complex piece of code instead just having a bunch of if statements. Eventually this piece got extended to do several other things, but because it was structured well, extending it was easy with minimum impact for the code-base. This is why design matters.

            Above claims are based on nearly 2 decades of writing software, 3/4 of it in big companies with very complex requirements.

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

        inheritance to avoid code duplication

        What no, inheritance is not for code sharing

        Sound bite aside, inheritance is a higher level concept. That it “shares code” is one of several effects.

        The simple, plain, decoupled way to share code is the humble function call, i.e. static method in certain langs.