363
submitted 1 year ago* (last edited 1 year ago) by drekly@lemmy.world to c/programming@programming.dev

"UPDATE table_name SET w = $1, x = $2, z = $4 WHERE y = $3 RETURNING *",

does not do the same as

"UPDATE table_name SET w = $1, x = $2, y = $3, z = $4 RETURNING *",

It's 2 am and my mind blanked out the WHERE, and just wanted the numbers neatly in order of 1234.

idiot.

FML.

you are viewing a single comment's thread
view the rest of the comments
[-] jjjalljs@ttrpg.network 26 points 1 year ago

You all run queries against production from your local? Insanity.

[-] elvith@feddit.de 17 points 1 year ago

Everyone has a production system. Some may even have a separate testing environment!

[-] Tarte@kbin.social 17 points 1 year ago* (last edited 1 year ago)

The distinctions get blurry if you’re the sole user.

[-] drekly@lemmy.world 6 points 1 year ago* (last edited 1 year ago)

My only education is a super helpful guy from Reddit who taught me the basics of setting up a back end with nodejs and postgres. After that it's just been me, the references and stack overflow.

I have NO education about actual practises and protocol. This was just a tool I made to make my work easier and faster, which I check in and update every few months to make it better.

I just open vscode, run node server.js to get started, and within server.js is a direct link to my database using the SQL above. It works, has worked for a year or two, and I don't know any other way I should be working. Happy to learn though!

(but of course this has set me back so much it would have been quicker not to make the tool at all)

[-] jjjalljs@ttrpg.network 2 points 1 year ago

With that amount of instruction you've done well

There's probably lots of stuff you don't even know you don't know.

Automated testing is a big part of professional software development, for example, and helps you catch things like this issue before they go live.

[-] drekly@lemmy.world 1 points 1 year ago* (last edited 1 year ago)

I'm up to 537 lines of server code, 2278 lines in my script, and 226 in my API interfacing, I'm actually super proud of it haha.

But you're totally right, there are things I read that I just have no clue what they even mean or if I should know it, and probably use all the wrong terminology. I feel like I should probably go back to the start and find a course to teach me properly. I've probably learned so many bad habits. It doesn't help that I learned JS before ES6 so I need to force myself not to use var and force myself to understand and use arrow functions.

I absolutely know that the way I've written the program will make some people cringe, but I don't know any better. There are a few sections where I'm like "would that actually be what a real, commercial web app would do, or have I convoluted everything?"

For example, the entire thing is just one 129-line html file. I just hide and unhide divs when I need a new page or anything gets changed. I'm assuming that's a bad thing, but it works, it looks good, and I don't know any better!

[-] max@feddit.nl 2 points 1 year ago

Have a look at an ORM, if you are indeed executing plain SQL like I’m assuming from your comment. Sequelize might be nice to start with. What it does is create a layer between your application and your database. Using which, you can define the way a database object looks (like a class) and execute functions on that. For instance, if you’re creating a library, you could do book.update(), library.addBook(), etc. Since it adds a layer in between, it also helps you prevent common vulnerabilities such as SQL injection. This is because you aren’t writing the SQL queries in the first place. If you want to know more, let me know.

[-] drekly@lemmy.world 2 points 1 year ago

Thanks, I'll look into it! I'm interested in why you got downvoted though! 😅

[-] jjjalljs@ttrpg.network 3 points 1 year ago

I didn't downvote but some people like ideologically dislike orms. The reasons I've heard are usually "I can write better SQL by hand", "I don't want to use/learn another library", "it has some limitations"

Those things can be true. Writing better SQL by hand definitely is a big "it depends", though.

[-] max@feddit.nl 2 points 1 year ago

I can see why people might dislike them. Adds some bloat perhaps. But at the same time, I like the idea that my input is definitely sanitised since the ORM was written by people who know what they’re doing. That’s not to say it won’t have any vulnerabilities at all, but the chance of them existing is a lot lower than when I write the queries by hand. A lapse of judgement is all it takes. Even more relevant for beginning developers who might not be aware of such vulnerabilities existing.

[-] drekly@lemmy.world 1 points 1 year ago

For a personal tool that runs locally I can handle some bloat in the name of safety!

[-] max@feddit.nl 1 points 1 year ago

Mostly just safety from yourself/your own little errors in input, but it can’t hurt for sure! Input sanitation is mostly relevant to fend off script kiddies. Relevant xkcd

[-] max@feddit.nl 2 points 1 year ago* (last edited 1 year ago)

Short story, haters gonna hate ¯\_(ツ)_/¯ Long story, see my comment to the commenter below you. :)

[-] ck_@discuss.tchncs.de 4 points 1 year ago

If coffee simply does not get you out of bed in the mornings any more, you need to find your fix elsewhere 💁

this post was submitted on 11 Oct 2023
363 points (97.6% liked)

Programming

17691 readers
217 users here now

Welcome to the main community in programming.dev! Feel free to post anything relating to programming here!

Cross posting is strongly encouraged in the instance. If you feel your post or another person's post makes sense in another community cross post into it.

Hope you enjoy the instance!

Rules

Rules

  • Follow the programming.dev instance rules
  • Keep content related to programming in some way
  • If you're posting long videos try to add in some form of tldr for those who don't want to watch videos

Wormhole

Follow the wormhole through a path of communities !webdev@programming.dev



founded 2 years ago
MODERATORS