• barsquid@lemmy.world
    link
    fedilink
    arrow-up
    1
    ·
    5 months ago

    One of the reasons women will find this repugnant is because they didn’t normalize their tables. Should be boyfriend_id is null.

    • JackbyDev@programming.dev
      link
      fedilink
      English
      arrow-up
      1
      ·
      4 months ago

      Why is there a separate table for men and women in the first place? Shouldn’t there be a person table with a many to many relationship with itself (because polyamory exists)?

      • GBU_28@lemm.ee
        link
        fedilink
        English
        arrow-up
        1
        ·
        5 months ago

        They allowed business logic to pollute the DB table, and “small waist” is a defined range in some confluence doc somewhere.

        • Breve@pawb.social
          link
          fedilink
          arrow-up
          2
          ·
          4 months ago

          Or this is an analytics database where these are well-defined dimensions added for segmentation logic. 🤷

  • MercuryGenisus@lemmy.world
    link
    fedilink
    arrow-up
    1
    ·
    edit-2
    5 months ago

    I would reject this pull request. Why is the indenting all over the place? Why is your keyword capitalisation all over the place? WHY YELLOW?!

    Edit: the more I look at this the more it pisses me off. Wtf is going on with your kerning? Just random number and placement of spaces. Also, why is the table name in caps? Who does that? Select * is lazy. Do you really need every field about a girl? Really? Worst of all, not a limited request. I sware this is just the kind of thing that would return 30 million rows and brick the database for twenty seconds.

    • Black616Angel@discuss.tchncs.de
      link
      fedilink
      arrow-up
      1
      ·
      4 months ago

      You forgot some: Why is there no space after SELECT?

      Why are boyfriend and smallwaist not questions like is_cute and is_crazy? Either all boils are with a verb or none.

      Also why is smallwaist not in snake case? It should be small_waist (or better yet has_small_waist or even better waist = “small”)

      Also also boyfriend should be null not false, this would solve multiple issues.

      And finally the only positive thing is the * itself, because selecting only body would be even worse. 🤣