• UndercoverUlrikHD@programming.dev
      link
      fedilink
      arrow-up
      9
      ·
      8 days ago
      class MyClass:
          def __init__(self, x: int):
              self.whatever: int = x
      
      def foo(x: MyClass) -> int:
          return x.whatevr
      

      Any decent IDE would give you an error for unresolved attribute. Likewise it would warn you of type error if the type of x.whatever didn’t match the return type of foo()

      • Starbuncle@lemmy.ca
        link
        fedilink
        English
        arrow-up
        7
        ·
        8 days ago

        You’re both right. It’s possible to write code that gets linted well in Python, yes, but you’re often not working with just your code. If a library doesn’t use typing properly, not a lot to be done without a ton more effort.

      • FizzyOrange@programming.dev
        link
        fedilink
        arrow-up
        2
        ·
        8 days ago

        Yes because you used static type annotations. This thread was about code that doesn’t use static types (or static type annotations/hints).

        • Eiri@lemmy.ca
          link
          fedilink
          arrow-up
          0
          ·
          7 days ago

          Nope, don’t need to. WebStorm can even detect nonexistent attributes for objects whose format the back-end decides, and tbh I’m not sure what sort of sorcery it uses.

    • Strykker@programming.dev
      link
      fedilink
      arrow-up
      2
      ·
      9 days ago

      It’s python, just use type hinting already and your linter will catch that.

      Also some winters can look at the use of food and see the type being passed in.

      • Ephera@lemmy.ml
        link
        fedilink
        arrow-up
        4
        ·
        9 days ago

        Autocorrect got you pretty bad, there.

        I was very confused, why we’re suddenly talking about rationing food during winter. 🙃

      • FizzyOrange@programming.dev
        link
        fedilink
        arrow-up
        0
        ·
        9 days ago

        Yes you can use static type hinting and the static type checker (Mypy or Pyright) will catch that. Linters (Pylint) won’t.

    • ripcord@lemmy.world
      link
      fedilink
      arrow-up
      0
      ·
      9 days ago

      Not with an example that simple and poor, no.

      If you have done the minimum and at least set a type hint, or if your ide is smart enough to check what calls the function and what it passes, then it’ll be flagged.

      • calcopiritus@lemmy.world
        link
        fedilink
        arrow-up
        1
        ·
        8 days ago

        This is literally a getter function. How is s getter awful code? It’s the simplest function there is. The only function simpler than that is returning the input itself.

        • BrianTheeBiscuiteer@lemmy.world
          link
          fedilink
          arrow-up
          0
          arrow-down
          1
          ·
          8 days ago

          How does “foo” mean “get”? Half the battle of writing correct code is writing code that’s easy to interpret. Do you always look at the guts of every function you’re about to use?

      • Ephera@lemmy.ml
        link
        fedilink
        arrow-up
        0
        ·
        9 days ago

        How would you make it non-awful, without specifying static types?

        I guess, a unit test would catch it, but needing 100% test coverage to catch typos isn’t exactly great…

        • Backlog3231@reddthat.com
          link
          fedilink
          English
          arrow-up
          1
          ·
          edit-2
          7 days ago

          I would do

          class MyClass:
              def __init__(self, value):
                  self._whatever = value
          
              @property
              def whatever(self):
                  return self._whatever
          

          Personally, I would type hint all of that but I’m just showing how you can do it without types. Your linter should be smart enough to say “hey dumbass did you mean this other thing”? Also since we didn’t create a setter you can’t arbitrarily overwrite the value of whatever so thats neat.

          And I’ll just say before I post that I’m on mobile and I’m sorry if the formatting is fucked. I’m not going to fix it.

        • BrianTheeBiscuiteer@lemmy.world
          link
          fedilink
          arrow-up
          0
          ·
          edit-2
          9 days ago

          What’s the purpose of foo? Why an ambiguous single character variable? What if the property was there but the value was null? Why not use (assuming JS) optional chaining?

          I’d approach it more like this:

          function getWhatevrProp(userData) (
            const default = { whatevr: "n/a" };
          
            return { ...default, ...userData }.whatevr;
          }
          

          Sorry, read too fast the first time. It’s more likely Python. I also don’t know Python well enough to give recommendations on that.