Letters to the editor: go to statement considered harmful | Communications of the ACM
dl.acm.org

In this letter, Dijkstra talks about readability and maintainability in a time where those topics were rarely talked about (1968). This letter was one of the main causes why modern programmers don’t have to trouble themselves with goto statements. Older languages like Java and C# still have a (discouraged) goto statement, because they (mindlessly) copied it from C, which (mindlessly) copied it from Assembly, but more modern languages like Swift and Kotlin don’t even have a goto statement anymore.

        • atzanteol@sh.itjust.worksEnglish
          1·
          6 个月前

          Following that logic if, else and while are also “pseudo goto” statements.

          There’s nothing wrong with conditional jumps - we couldn’t program without them. The problem with goto specifically is that you can goto “anywhere”.

    • TehPers@beehaw.orgEnglish
      3·
      6 个月前

      In C# at least, goto can take you between case labels in a switch statement (rather than using fallthrough), which I don’t view as being nearly as bad. For example, you can do goto case 1 or goto default to jump to another case.

      The only other use of goto I find remotely tolerable is when paired with a labelled loop statement (like putting a label right before a for loop), but honestly Rust handles that far better with labelled loops (and labelled block expressions).

      • asyncrosaurus@programming.dev
        5·
        6 个月前

        I’ve programmed C# for nearly 15 years, and have used goto twice . Once to simplify an early break from a nested loop, essentially a nested continue. The second was to refactor a giant switch statement in a parser, essentially removing convoluted while loops, and just did a goto the start.

        It’s one of those things that almost should never be used, but the times it’s been needed, it removed a lot of silliness.

  • eveninghere@beehaw.org
    19·
    6 个月前

    For C it makes sense. The point of C is that it can work as a low level language. Basically, everything doable with assembly SHOULD be doable with C, and that’s why we don’t need another low level language that’s basically C with goto.

    Even though almost all of C users should never use goto.

  • Deebster@programming.dev
    141·
    6 个月前

    For such an influential letter, I don’t find his arguement all that compelling. I agree that not using go to will often lead to better structured (and more maintainable) programs, but I don’t find his metric of “indexable process progress” to satisfyingly explain why that is.

    Perhaps it’s because at that time people would be running the programs in their heads before submitting them for processing, so they tended to use more of a computer scientist mindset - whereas now we’re more likely to use test cases to convince ourselves that code is correct.

    • Kissaki@programming.devEnglish
      7·
      6 个月前

      I think it’s convoluted way to describe it. Very technically-practical. I agree it’s probably because of historical context.

      The argument I read out of it is that using goto breaks you being able to read and follow the code logic/run-logic. Which I agree with.

      Functions are similar jumps, but with the inclusion of a call stack, you can traverse and follow them.

      I think we could add a goto stack / include goto jumps in the call stack though? It’s not named though, so the stack is an index you only understand when you look at the code lines and match goto targets.

      I disagree with unit tests replacing readability. Being able to read and follow code is central to maintainability, to readability and debug-ability. Those are still central to development and maintenance even if you make use of unit tests.

      • Deebster@programming.devEnglish
        3·
        6 个月前

        I wasn’t saying that unit tests replaces readability, I was saying that back in the 60s they’d reason and debug using their brains (and maybe pen and paper), with more use of things like formal proofs for correctness. Now that we write more complicated programs in more powerful environments, it’s rare to do this (we’d use breakpoints, unit tests, fuzzing, etc).

    • atzanteol@sh.itjust.worksEnglish
      6·
      6 个月前

      Perhaps it’s because at that time people would be running the programs in their heads before submitting them for processing, so they tended to use more of a computer scientist mindset - whereas now we’re more likely to use test cases to convince ourselves that code is correct.

      This is 1968. You didn’t have an IDE or debugger. Your editor was likely pretty terrible too (if you had one). You may have still been using punch cards. It’s possible the only output you got from your program was printed on green-bar paper.

      “Back in the day” it wasn’t uncommon to sit with a printout of your code and manually walk though it keeping state with a pencil. Being able to keep track of things in your head was very important.

      GOTO existed in part for performance purposes. When your CPU clock is measured in megahertz, your RAM is measured in kilobytes and your compilers don’t do function in-lining it’s quicker and cheaper to just move the program pointer than it is to push a bunch of variables on a stack and jump to another location, then pop things off the stack when you’re done (especially if you’re calling your function inside a loop). Even when I was programming back in the '80s there was a sense that “function calls can be expensive”. It wasn’t uncommon then to manually un-roll loops or in-line code. Compilers are much more sophisticated today and CPUs are so much faster that 99% of the time you don’t need to think about now.

      Oddly enough the arguments against GOTO are less important today as we have much better development tools available to us. Though I certainly won’t recommend it over better flow-control structures.

      • 0x0@programming.dev
        2·
        6 个月前

        When your CPU clock is measured in megahertz, your RAM is measured in kilobytes

        Ah yes, the good ol’ days when developers programmed for efficiency.

        • atzanteol@sh.itjust.worksEnglish
          3·
          6 个月前

          Mostly because they had to. Writing efficient code and easy-to-read code are often at odds with each other. I like being able to create lots of functions that can be called from a loop without needing to worry too much about function call overhead. I can prioritize readability for some time before I ever need to worry about performance.

            • atzanteol@sh.itjust.worksEnglish
              2·
              5 个月前

              People have been complaining about this exact thing forever. Even back when “people cared about efficiency”.

            • BatmanAoD@programming.dev
              1·
              5 个月前

              Does the catchphrase “blazing fast” ring any bells? Some people care.

              (Arguably that’s just the pendulum swinging the other way; Ruby, Python, and Java ruled the software world for a while, and I think that’s a large part of why the Go and Rust communities make such a big deal about speed.)

      • atzanteol@sh.itjust.worksEnglish
        6·
        6 个月前

        It’s not idiotic. You can do a lot of performance optimizations with GOTO so providing it as a “use it if you know what you’re doing” option is fine. And some things are easier to read with GOTO.

        FWIW the Linux source code is full of GOTO statements. Nearly 200,000 of them in fact.

        • huginn@feddit.it
          2·
          6 个月前

          There’s a solid reason for goto in C.

          Bringing goto into Java was (and is) idiotic.

          If you’re trying to squeeze every ounce of performance out of your code then you’ll need those optimizations.

          But any higher level language than C the entire point is to write easily maintainable and useful code that any idiot can go, read and update. A goto is antithetical to readability.

          • ggppjj@lemmy.worldEnglish
            1·
            6 个月前

            It can be, sure. But when used in a limited manner where it makes sense it can be the more readable option. I’ve used it in a try/catch to retry the operation after changing a variable. One label (“reconnect”), one goto, totally easy to understand on a surface level.

    • billybong@lemmynsfw.com
      3·
      6 个月前

      Single exit point is an assembly rule that was mindlessly ported to C/C++. It doesn’t make sense in a structured language.

          • 0x0@programming.dev
            1·
            5 个月前

            A common case is a function that uses resources like opening files, sockets, whatever. If there’s an error you want to ensure a clean exit, so at different parts of said function you goto the end where you check if said resources are being used and release them before exiting.

            Cleaner than constantly checking at every error.

  • Kissaki@programming.devEnglish
    9·
    6 个月前

    Their main argumentation (from page 1) summarized:

    You know the state and progress of a program from the line you are on. A goto breaks that.

    You can index the progress of a program through static line indexes and a dynamic loop index and function call stack. A goto breaks that. Including a “statements/lines since beginning of execution” is infeasible for understanding.

    • abbadon420@lemm.eeOP
      2·
      6 个月前

      The article does say that there are good cases to use goto, but they are rare and most programmers won’t ever encounter such situations. I believe the jist is that it can do nore harm than good.

    • mrkite@programming.dev
      41·
      6 个月前

      And yet it’s still easy to write spaghetti code in Java. Just abuse inheritance. Where is this function implemented? No one knows but the compiler!

      • SatouKazuma@ani.social
        3·
        6 个月前

        And don’t forget to use miracle sort for any sorting needs you have. Why do the work yourself when you can just hope it gets solved for you?!

      • warlaan@lemm.ee
        11·
        5 个月前

        That doesn’t make it spaghetti code though. In well-written OOP code you shouldn’t care where a function is implemented. The problem is a much too high level of abstraction. If your high level code is so abstract that it is only running tasks and handling messages there’s no way to write it in a way that prevents mistakes because you couldn’t possibly know what the actual implementations do.

  • Kissaki@programming.devEnglish
    3·
    6 个月前

    PDF magic… It has grainy text. But the selectable text and displayed text have a 1-character offset.

    I assume they display the original scan so it definitely does not contain errors, while still providing the image-parsed text for searchability, indexability, and select-+copyability?

    screenshot of text + backing text offset

    Unfortunately, the grainy text is hard[er] to read.