this post was submitted on 27 Dec 2023
547 points (96.8% liked)

Programmer Humor

19712 readers
106 users here now

Welcome to Programmer Humor!

This is a place where you can post jokes, memes, humor, etc. related to programming!

For sharing awful code theres also Programming Horror.

Rules

founded 1 year ago
MODERATORS
 
top 50 comments
sorted by: hot top controversial new old
[โ€“] [email protected] 148 points 11 months ago* (last edited 11 months ago) (3 children)

Honestly more readable than a lot of SQL I've read. It even has hierarchical grouping.

[โ€“] [email protected] 52 points 11 months ago* (last edited 11 months ago) (4 children)

I was disgusted by the XML at first, but it's a readable query returning a sane JSON object.

Meanwhile, I'm mantaining Java code where the SQL is a perfectly square wall of text, and some insane mofo decided the way to read the resulting list of Object[] ๐Ÿคฎ is getting each column by index... so I'd switch to SQXMLL in a heartbeat.

[โ€“] cmdrkeen 22 points 11 months ago

React basically figured out how to make XML work.

Remember, XML was actually designed for use cases like this, thatโ€™s why it came with XPath and XSLT, which let you make it executable in a sense by performing arbitrary transformations on an XML tree.

Back in the day, at my first coding job, we had an entire program that had a massive data model encoded in XML, and we used a bunch of XSL to programmatically convert that into Java objects, SQL queries, and HTML forms. Actually worked fairly well, except of course that XSL was an awful language to do that all in.

React simply figured out how to use JavaScript as the transformation language instead.

[โ€“] [email protected] 4 points 11 months ago

it's a readable query returning a sane JSON object.

No it's not. What table is the data supposed to be coming from..?

load more comments (2 replies)
[โ€“] [email protected] 4 points 11 months ago (2 children)

true, but having it look like a component might get annoying. since this is likely to stay at the top, having an island of non components between two components might make it hard to see where functions start and end. and if this isn't used directly inside a component it'll just look dumb and inefficient (this also looks like it'll take way more to edit once you change something)

load more comments (2 replies)
load more comments (1 replies)
[โ€“] cmdrkeen 83 points 11 months ago (1 children)

Honestly not the worst thing Iโ€™ve seen.

[โ€“] [email protected] 14 points 11 months ago (3 children)

I'd like you to think for a moment about CTEs, the HAVING clause, window functions and every other funky and useful thing you can do in SQL ... Now just think, do you think that this syntax supports all those correctly?

[โ€“] [email protected] 33 points 11 months ago

sql syntax doesn't support even itself correctly i fail to see your point

[โ€“] cmdrkeen 16 points 11 months ago

Probably no better or worse than any other ORM written in a more traditional language. Worst comes to worst, you can always escape to plain SQL.

load more comments (1 replies)
[โ€“] [email protected] 40 points 11 months ago

Ah yes. That's what the kids call "sqlx" right?

NGL, if it has real time code completion and compile time SQL checks, this is fine.

[โ€“] expr 38 points 11 months ago (4 children)

Not only is this really gross, it's also straight up wrong. It's missing a from clause, and it makes no sense for a where clause to be nested under the select. The select list selects columns from rows that have already been filtered by the where clause. Same for the limit.

Also just gonna go ahead and assume the JSX parser will happily allow SQL injection attacks...

load more comments (4 replies)
[โ€“] [email protected] 38 points 11 months ago (2 children)

I actually like this. This would allow reuse of all the infrastructure we have around XML. No more SQL injection and dealing with query parameters? Sign me up!

[โ€“] [email protected] 8 points 11 months ago

Assuming it's built well. As someone else pointed out, it doesn't look quite right here.

[โ€“] [email protected] 5 points 11 months ago (3 children)

So you mean like parameterized queries, which exist?

load more comments (3 replies)
[โ€“] [email protected] 37 points 11 months ago (2 children)

I want to hate this. I really do. But the problem isโ€ฆ I think I like it.

[โ€“] naonintendois 13 points 11 months ago (2 children)

But how do I know if the WHERE clause is AND or OR?

[โ€“] [email protected] 5 points 11 months ago (1 children)

We can say default is and and add an Or node for or. Similar to SoP notation, you only write +.

load more comments (1 replies)
load more comments (1 replies)
load more comments (1 replies)
[โ€“] [email protected] 31 points 11 months ago* (last edited 11 months ago)

When you are assigned to write database queries at work and your academical background is that online react bootcamp

[โ€“] [email protected] 29 points 11 months ago (21 children)

if you don't believe that adding more structure to the absolute maniacal catastrophe that is sql is a good thing then i'm going to start to have doubts about your authenticity as a human being

[โ€“] [email protected] 25 points 11 months ago

If you think this is more structured than traditional SQL, I really disagree. Is this a select * query, it's ambiguous. Also what table is being queried here there's no from or other table identifier.

load more comments (20 replies)
[โ€“] [email protected] 25 points 11 months ago (1 children)

Remember kids, JSX is just function calls. It can't hurt you.

[โ€“] [email protected] 18 points 11 months ago

Oh it can and it did.

[โ€“] [email protected] 21 points 11 months ago (2 children)

The most offensive thing here is the amount={5} attribute. What is it? It's not XML.

[โ€“] [email protected] 24 points 11 months ago (1 children)

It's a react component and that would be the proper way to give a numerical value in jsx

[โ€“] [email protected] 6 points 11 months ago

JSX has grown beyond react, so without further context it doesn't have to be react.

[โ€“] [email protected] 14 points 11 months ago (1 children)

It's JSX. It's used to embed markup into javascript

[โ€“] [email protected] 5 points 11 months ago

It's to embed Javascript into embedded markup in Javascript

[โ€“] [email protected] 20 points 11 months ago (7 children)

please kindly send all javascript into the sun and explode it

load more comments (7 replies)
[โ€“] [email protected] 14 points 11 months ago (3 children)

This idea is bad and whoever came up with it should feel bad.

load more comments (3 replies)
[โ€“] [email protected] 14 points 11 months ago
[โ€“] [email protected] 14 points 11 months ago

still more readable than sqlalchemy exceptions

[โ€“] [email protected] 13 points 11 months ago

I haven't been this pissed off since LINQ started allowing syntax switches in random-ass places.

[โ€“] [email protected] 12 points 11 months ago
[โ€“] [email protected] 12 points 11 months ago

Omg that's terrible! Link?

[โ€“] [email protected] 11 points 11 months ago

"HTTP and the Web is a totally reliable and easy to use internet protocol"

[โ€“] [email protected] 7 points 11 months ago (1 children)

Is that select * ?

I expect it looks more cumbersome with joins and multiple columns from different tables.

load more comments (1 replies)
[โ€“] [email protected] 5 points 11 months ago

Of course not... where's the damn tag..?

[โ€“] [email protected] 5 points 11 months ago

Joins must be a pain in the ass with hooks

load more comments
view more: next โ€บ