• GBU_28@lemm.ee
    link
    fedilink
    English
    arrow-up
    80
    arrow-down
    2
    ·
    11 months ago

    No closing semicolon, anyone got any extras to throw on this thing?

    • Moops@lemmy.world
      link
      fedilink
      English
      arrow-up
      17
      arrow-down
      1
      ·
      edit-2
      11 months ago

      At the very least I’d try to clean up that fuzzy condition on behavior to anticipate any bad or inconsistent data entry.

      WHERE UPPER(TRIM(behavior)) = ‘NICE’

      Depending on the possible values in behavior, adding a wildcard or two might be useful but would need to know more about that field to be certain. Personally I’d rather see if there was a methodology using code values or existing indicators instead of a string, but that’s often just wishful thinking.

      Edit: Also, why dafuq we doing a select all? What is this, intro to compsci? List out the values you need, ya heathen ;)

      (This is my favorite Xmas meme lol)

    • pruwyben@discuss.tchncs.de
      link
      fedilink
      arrow-up
      7
      ·
      edit-2
      11 months ago

      Need to normalize the database. I would add a join to a BehaviorTypes table.

      Edit: or, if the only options are naughty or nice, make it a boolean.

    • krotti@sh.itjust.works
      link
      fedilink
      arrow-up
      4
      ·
      11 months ago

      Honest question, which ones wouldn’t it work with? Most add a semicolon to the end automatically or have libraries and interfaces saved me a million times?

      • GBU_28@lemm.ee
        link
        fedilink
        English
        arrow-up
        6
        arrow-down
        2
        ·
        11 months ago

        Other reply s accurate but it’s always a good practice to include the semicolon else you can get

        “Bobby tables’ed” look that xkcd comic up

        • Doc Avid Mornington@midwest.social
          link
          fedilink
          English
          arrow-up
          8
          ·
          11 months ago

          I’m not sure how including a final semicolon can protect against an injection attack. In fact, the “Bobby Tables” attack specifically adds in a semicolon, to be able to start a new command. If inputs are sanitized, or much better, passed as parameters rather than string concatenated, you should be fine - nothing can be injected, regardless of the semicolon. If you concatenate untrusted strings straight into your query, an injection can be crafted to take advantage, with or without a semicolon.

          • GBU_28@lemm.ee
            link
            fedilink
            English
            arrow-up
            3
            arrow-down
            1
            ·
            edit-2
            11 months ago

            Yep it would only work if you didn’t sanitize a user input string in this case ‘nice’

            They could write ‘’; drop table blah;

      • jaybone@lemmy.world
        link
        fedilink
        arrow-up
        1
        ·
        11 months ago

        Usually with libraries like jdbc or whatever and prepared statements you don’t need the semicolon.

    • takeda@lemmy.world
      link
      fedilink
      arrow-up
      4
      ·
      11 months ago

      You need semicolons if it is a script with multiple commands to separate them. It is not needed for a single statement, like you would use in most language libraries.

      • mellejwz@lemmy.world
        link
        fedilink
        English
        arrow-up
        2
        ·
        11 months ago

        If you don’t use a semicolon directly in MySQL it won’t do anything until you add it.

        • takeda@lemmy.world
          link
          fedilink
          arrow-up
          2
          ·
          11 months ago

          In the MySQL client console where you can run multiple commands.

          If you add semicolon in language library commands such as fetch() you will get an error.

  • Truck_kun@beehaw.org
    link
    fedilink
    English
    arrow-up
    53
    ·
    11 months ago

    That SELECT and WHERE are all caps, but from is not is bugging me.

    I don’t care if you choose to uppercase keywords or lowercase, but consistency please.

    Also, great, love it.

    • katy ✨@lemmy.blahaj.zone
      link
      fedilink
      arrow-up
      6
      ·
      11 months ago

      it also implies that naughty or nice is an either or thing and not a weighted thing from an incidents table. the good place lied to us.

      • ursakhiin@beehaw.org
        link
        fedilink
        arrow-up
        4
        ·
        11 months ago

        It could be a materialized view that is generated off of a weighting where you are nice until you have a certain number of incidents.

  • guy@lemmy.world
    link
    fedilink
    arrow-up
    36
    ·
    11 months ago

    Guess that settles the debate, we got to pronounce it “sequel” then to optimally match syllables

    • _danny@lemmy.world
      link
      fedilink
      arrow-up
      5
      arrow-down
      5
      ·
      11 months ago

      The only people I know who actually call it ess queue ell are either too new to know the “sequel” pronunciation, or the type of person you generally smell before you see.

      • SpeakinTelnet@programming.dev
        link
        fedilink
        arrow-up
        9
        arrow-down
        1
        ·
        11 months ago

        I say ess cue ell for the sake of uniformity because it’s not Mysequel nor Postgresequel and the language changed from Sequel to the acronym SQL in the 70s so not really in the “too new” ballpark anymore.

        • _danny@lemmy.world
          link
          fedilink
          arrow-up
          2
          arrow-down
          1
          ·
          11 months ago

          I think those make sense as deviations. I’ve heard “my sequel” but you’re absolutely right about postgresql.

          The name is kinda irrelevant like hard vs soft g in gif. People know what you mean when you say either.

          But in that same vein, the creator of the “graphics interchange format” says the pronunciation is soft g, but basically everyone says hard g… So “official” pronunciation is kinda irrelevant.

          I don’t judge anyone who uses whichever term they want, but I’ve just noticed the general trend in my smallish interaction bubble.

      • relevants@feddit.de
        link
        fedilink
        arrow-up
        6
        arrow-down
        2
        ·
        11 months ago

        Here in Germany everyone I know pronounces the letters individually – as German letters that is, which means the Q is pronounced “coo” rather than “cue”. I don’t mind it, it’s not quite as clunky as in English.

        I do say sequel when speaking English though.

      • cm0002@lemmy.world
        link
        fedilink
        arrow-up
        3
        ·
        11 months ago

        I’m neither, I refuse to pronounce acronyms if it doesn’t make sense to do so.

        Same thing with ‘gooey’ for GUI, except I hate that even more because that straight up elicits feelings of disgust, I don’t want anything gooey anywhere near any electronics

        • _danny@lemmy.world
          link
          fedilink
          arrow-up
          2
          ·
          11 months ago

          I’ve literally never heard GUI said as “gee ewe eye” before.

          You could just say UI, avoids the gooey phobia and sounds less weird than g u i.

      • rufus@discuss.tchncs.de
        link
        fedilink
        arrow-up
        2
        ·
        11 months ago

        Come they told me, pa rum pum pum pum A new born King to see, pa rum pum pum pum Our finest gifts we bring, pa rum pum pum pum To lay before the King, pa rum pum pum pum, rum pum pum pum, rum pum pum pum,

        So to honor Him, pa rum pum pum pum, When we come.

        Little Bobby, pa rum pum pum pum I am a poor boy too, pa rum pum pum pum I have no gift to bring, pa rum pum pum pum That’s fit to give the King, pa rum pum pum pum, rum pum pum pum, rum pum pum pum,

        Shall I play for you, pa rum pum pum pum, On my unsanitized database inputs?

  • ApexHunter@lemmy.ml
    link
    fedilink
    arrow-up
    27
    ·
    11 months ago

    I can’t be the only one disappointed by the lack of an order by clause after being told the list was being sorted (twice!)…

  • Akrenion@programming.dev
    link
    fedilink
    arrow-up
    14
    arrow-down
    1
    ·
    11 months ago

    Can anyone recommend a cheap receipt printer that takes pictures from a pc or phone? I want to print mtg tokens on the fly.

  • Donkter@lemmy.world
    link
    fedilink
    arrow-up
    11
    ·
    11 months ago

    The beginning maps perfectly to “The Distance” by Cake and I was singing along to that tune as I read.