this post was submitted on 01 Jul 2024
2 points (100.0% liked)

LinkedinLunatics

4997 readers
137 users here now

A place to post ridiculous posts from linkedIn.com

(Full transparency.. a mod for this sub happens to work there.. but that doesn't influence his moderation or laughter at a lot of posts.)

founded 2 years ago
MODERATORS
 
top 15 comments
sorted by: hot top controversial new old
[–] MercuryGenisus@lemmy.world 2 points 1 year ago* (last edited 1 year ago) (3 children)

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.

[–] midori@lemmy.world 1 points 1 year ago

I would like to subscribe to your newsletter.

[–] frezik@midwest.social 1 points 1 year ago (1 children)

Shouldn't boyfriend be a reference to another table?

[–] ArtVandelay@lemmy.world 1 points 1 year ago (1 children)

BF has no referential integrity

That's (part of) why it should be a separate table to map the relation "Relationship". People can have more than one (polyamory, infidelity), and you could track fields like the start, end, status (e.g. flirting, dating, committed, engaged, married, ended) in there.

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. 🤣

[–] heavy@sh.itjust.works 1 points 1 year ago

At least he's wearing the red flag on the outside.

[–] coffinwood@discuss.tchncs.de 1 points 1 year ago (1 children)

Guy with a belly asks for girl with a small waist. The half-assed ugly shirt will do it.

Instant woman repellant.

[–] jqubed@lemmy.world 1 points 1 year ago

I feel confident in assuming the guy who would wear this shirt seeking “girls” between the ages of 18 and 26 is himself no younger than 45.

[–] fibojoly@sh.itjust.works 1 points 1 year ago* (last edited 1 year ago) (1 children)

Should be age > (my_age / 2) +7

[–] JackbyDev@programming.dev 1 points 1 year ago (1 children)

Why would there be an age and my_age column on the table GIRLS?

[–] lennivelkant@discuss.tchncs.de 1 points 1 year ago (1 children)

Good point.

Should be age > (@my_age / 2) +7

FTFOP - now my age is some value defined outside the immediate query.

More likely, the GIRLS would be a view of some table persons and you could query my_age from that table too.

[–] fibojoly@sh.itjust.works 1 points 1 year ago* (last edited 1 year ago) (1 children)

Thank you. I assumed the reader would be educated enough to guess I meant a variable. But yeah, should used @my_age

Pretty sure "People who know enough about SQL to know about variables" is a subset of "People who know enough about SQL to be pedantic about it" :p

[–] barsquid@lemmy.world 1 points 1 year ago

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