Minor QuerySelect code improvements, QuerySelect documented. (#69383)
Minor QuerySelect code improvements, QuerySelect documented. (#69383)

* Minor QuerySelect improvements coded from the porcelain throne.

We don't handle bad values being given in the query list well enough. This normally won't matter, runtimes are runtimes, but if mixed with real queries, it can lead to inconsistent state where we have query datums that have been ran, and query datums that have not been ran.

pre-checking is also an option, so that it can just refuse to run any of them if one is bad by checking before, but my main goal was to prevent runtimes from bad inputs leading to undeleted queries spam while being more clear about where the bug is, not try to perfectly handle the side effects of bad code.

(To be clear, i started intending to just add codedoc but now it uses as anything and typechecks because that was just eating at me while typing up the codedoc.)

* Update code/controllers/subsystem/dbcore.dm

Co-authored-by: Mothblocks <c52edec2bd3a27a0b2b5cccb4d09c5385b416d7e@users.noreply.github.com>
Bawhoppen/-tg-station,tgstation/tgstation,tgstation/tgstation,Bawhoppen/-tg-station,MicroWorldwide/SS13-TG-station,MicroWorldwide/SS13-TG-station,Bawhoppen/-tg-station,Bawhoppen/-tg-station,MicroWorldwide/SS13-TG-station,Bawhoppen/-tg-station,Bawhoppen/-tg-station,tgstation/tgstation,ShizCalev/tgstation,Bawhoppen/-tg-station,ShizCalev/tgstation,ShizCalev/tgstation,MicroWorldwide/SS13-TG-station,ShizCalev/tgstation,MicroWorldwide/SS13-TG-station,tgstation/tgstation,tgstation/tgstation,tgstation/tgstation,Bawhoppen/-tg-station,tgstation/tgstation,tgstation/tgstation,ShizCalev/tgstation,MicroWorldwide/SS13-TG-station,ShizCalev/tgstation,MicroWorldwide/SS13-TG-station,ShizCalev/tgstation,MicroWorldwide/SS13-TG-station,ShizCalev/tgstation