• chunes@lemmy.world
    link
    fedilink
    arrow-up
    4
    arrow-down
    1
    ·
    8 hours ago

    A lot of the arguments people use against vibe coding sounds a lot like “You shouldn’t use a high-level programming language unless you understand the assembly it compiles to.”

    • pinball_wizard@lemmy.zip
      link
      fedilink
      arrow-up
      2
      ·
      8 hours ago

      Yes. It’s the same thing, except when I joined the market knowing fuck all about computers, while coding with my cute little BASIC commands; there wasn’t some tech bro asshole telling all the C programmers that their skills would no longer be required. Or at least he (implicated in Epstein files billionaire, Bill Gates) wasn’t saying it quite so loudly (as accused rapist billionaire Sam Altman).

      Incidentally, I still work with that same C programmer, I’m still learning from them, but have since taught them a trick or two; and we both now charge sinful amounts of money for our time.

  • Thetechloop@programming.dev
    link
    fedilink
    English
    arrow-up
    1
    ·
    edit-2
    7 hours ago

    AI is corpo_slob theft. AI is used to kill kids. AI is a exponential decay feedback loop deskilling method brought to by the capitalist class. It will be used to lower wages. It will create security traps where the corpos can gain… ohhh the drama!!! We need regulation META cries…The lady doth protest too much, methinks, dolla dolla bills ya’ll. It will be used to spy on you and take away your free speech. It is a Poisonous cup of pee. We need more than cheese buger farts and violence. I want to live in a real culture not the techno bro hoedown. After the magic trick, they will pull the rug and you will be left with nothing, without a job. Lower the standards, corner the domestic market, lose the war and claim victory… seems fimilar too me. We are past the point of regulation. Wait til you see the effects of digital ID. They got black sites in the USA were the keep kids in cages… the tech bro and his slave trackers, thats how they get you. Bugs everywhere sold to you as tools and as toys. Language is Language and it changes over time. Empires are built on the ashes of others. Innovation and creation is communication and cooperation. The kings love to pick their fruit of the peasants labor as the knights protect the castles walls. We are entering stagflation and corpotechbro fuedalism. Slop be ass far ass the eye can see. The war in ukraine and the genocide in gaza are connected. The iceburg not fully revealed… the capitalist class is a trashy family… but they stick together and run their games. Look up 7 mountain mandate… now thats some Manchurian candidate

  • el_abuelo@programming.dev
    link
    fedilink
    arrow-up
    8
    arrow-down
    1
    ·
    15 hours ago

    I know I’m gonna get flamed for this, but here goes.

    I have vibe coded a system at work that has enabled me to deliver value in a fraction of the time I expected. My verification steps have been around whether it does the things I told it I wanted it to do. I’m not maintaining the syntax and I’m not expecting anyone else to. Ever.

    That said, our teams that deliver products that touch customer data or financial records…they shouldnt (and dont) engineer this way. The tech isnt there (yet).

    Let the flaming commence.

    • MonkeMischief@lemmy.today
      link
      fedilink
      arrow-up
      1
      ·
      8 hours ago

      “…enabled me to deliver value in a fraction of the time I expected.”

      I don’t wanna flame you for it.

      This is kinda where I see vibe coding taking off the most: The same phenomenon as using LLMs to summarize and reply to pithy business emails.

      Business gets so unbearably businessy, people are using machines to bypass the inherently un-human “professionalism performance” for them.

      It might not be work anybody can claim to be proud of, but right now success for the professional-managerial class runs on looking busy and circling back to synergizing the workflow with best practices for optimal returns regarding key performance metrics 🤢 or whatnot.

      I don’t blame people for it, in a work context. Looking impressive with AI tools is now a survival tactic when working under normies holding the purse strings and investing stupid amounts of money into the bubble. They’ve forced people to be part of a circular, self-reinforcing imaginary reality they’re profiting from.

      So, I don’t wanna flame you. I just hope you’re doing something fundamentally human to keep your soul alive on the side, at least. Something you as a unique human being on this earth can be proud of. <3

      • el_abuelo@programming.dev
        link
        fedilink
        arrow-up
        1
        arrow-down
        1
        ·
        4 hours ago

        Honestly im proud of the problems I’m able to solve today now that I can use AI to do it so quickly.

        Even when I stop and manually check all the work the AI has done, it’s still much faster and I’m having to rely on my knowledge and experience to do so. When I’m vibe coding I’m relying on that same knowledge and experience to architect the solution rather than deliver it.

        Ive seen the other side- the colleagues who all of a sudden think theyre engineers and are unknowingly exposing critical commercial data to our competitors. That side is scary…and gives me hope that my role while changing, is far from dead.

        Also I have hobbies - i only work to pay for them. If “the man” wants to pay me to spend tokens…fine by me. I’ll be here to clean it up if it all goes wrong.

    • JackbyDev@programming.dev
      link
      fedilink
      English
      arrow-up
      3
      ·
      14 hours ago

      I once heard someone describe legacy systems as systems without test coverage. I think it’s not the best description, but it’s certainly an interesting perspective.

      It’s part of why it bothers me when folks say they use LLMs to make unit tests. If anything, you should be writing tests by hands to get a solid specification then let the AI make the code. Of course this is a false dichotomy, but I’m just saying if you have to choose between those two options in some weird hypothetical bizarro world.

      • Shanmugha@lemmy.world
        link
        fedilink
        arrow-up
        2
        arrow-down
        1
        ·
        10 hours ago

        Not the best description doesn’t cut it, it’s just false description. I can easily write a system with 100% test coverage, bugged as hell and containing so many weird abstraction tricks that it takes a significant effort to figure out even what is happening in the simplest scenario. And yes, part of that hundred percent coverage is going to be llm style: test that something does what you already know it does

  • ☂️-@lemmy.ml
    link
    fedilink
    arrow-up
    35
    arrow-down
    1
    ·
    edit-2
    1 day ago

    actually, be exactly like bill. boss wants slop, boss gets slop.

    boss wants incomprehensible buggy code? that’s right, boss gets incomprehensible buggy code.

    • dream_weasel@sh.itjust.works
      link
      fedilink
      arrow-up
      5
      ·
      18 hours ago

      Be sure to ask Claude to remove all comments, tests, and dry runs, then ask if there’s a good way you can make the code more complicated without increasing runtime more than 40%.

    • psud@aussie.zone
      link
      fedilink
      English
      arrow-up
      5
      ·
      1 day ago

      Sorry, you were hired as a prompt engineer, your prompt wasn’t good enough to make perfect code, so you are fired

      • Shanmugha@lemmy.world
        link
        fedilink
        arrow-up
        1
        ·
        10 hours ago

        Nah. A bunch of llm code review agents say that produced code is of high quality, fire anyone who said it would have taken for them to implement same functionality more time than I have spent

    • Shanmugha@lemmy.world
      link
      fedilink
      arrow-up
      2
      ·
      10 hours ago

      He better not have his spine poked by anything sharp :)

      (in cade I failed at it: reference to Midra)

  • MoonRaven@feddit.nl
    link
    fedilink
    arrow-up
    53
    arrow-down
    2
    ·
    2 days ago

    I’ve seen the comments AI adds and yeah… No… It’s often pointing out the obvious or even in some cases just misleading.

    • miridius@lemmy.world
      link
      fedilink
      arrow-up
      1
      ·
      1 hour ago

      Out of the box yes this is true, but:

      1. Custom instructions go a long way
      2. Review the code yourself, tell it what to fix, and it will fix it. For me it often takes like 5 rounds of this before the code is fully polished to the point I’m proud of it. And you know what? It’s still MULTIPLE times faster than typing code by hand. And at least for me, the quality is higher, because I have 12 different agents that review the work too and they catch additional issues that even I missed.

      If you or others ship shitty code don’t blame the LLM, the issue is entirely the engineer using it wrong

    • Enkrod@feddit.org
      link
      fedilink
      arrow-up
      10
      ·
      1 day ago

      I told my boss I wouldn’t use AI to help write code and sign off on the commits with my name. He told me to use it to write the documentation… it was bad. Essential concepts were not mentioned and obvious shit was explained five times in slightly different phrasing. I am now writing our documentation as an obsidian vault myself again.

      I only have it compile a change log from the commit messages of commits and merges on the main and development branch. I know our commits are well written (because I established the standards for them in our repo myself) and that’s concrete and rigid enough that it can’t fuck it up enough to matter.

      But honestly there are build tools that could do that for me, I don’t need to buy tokens for that.

      • pinball_wizard@lemmy.zip
        link
        fedilink
        arrow-up
        6
        ·
        20 hours ago

        honestly there are build tools that could do that for me, I don’t need to buy tokens for that.

        Yes. This is where every “have you tried Agentic AI?” conservation lands, for me.

        They tell me I could pay daily, for worse results, to give up the bash script that has worked perfectly for five years. Oh, gee. Tell me more! Haha.

      • NotAnonymousAtAll@feddit.org
        link
        fedilink
        arrow-up
        16
        ·
        edit-2
        19 minutes ago

        Not just beginners unfortunately. One of the things I have to almost always teach to new colleagues, even experienced ones, is to put at least some minimal effort into making the code itself readable instead of relying on comments as crutches. Just basic things like picking proper names for variables and functions.

    • Jakeroxs@sh.itjust.works
      link
      fedilink
      arrow-up
      4
      arrow-down
      5
      ·
      20 hours ago

      You can add instructions to not comment, you can also have it explain what it does at every step, not everyone just doesn’t care about learning. It can be a very effective teaching tool if you use it that way. 🤷

    • Kindness is Punk@lemmy.ca
      link
      fedilink
      arrow-up
      2
      arrow-down
      1
      ·
      2 days ago

      Agreed it’s why if I’m asking for llm assistance I will generally start with a design for a component and ask it to follow that and comment accordingly usually leads to much better results than blanket asking it to do something for you

  • Thorry@feddit.org
    link
    fedilink
    arrow-up
    40
    arrow-down
    2
    ·
    2 days ago

    Asking an LLM to add comments is actually pretty much the worst thing you can do. Comments aren’t meant to be documentation and LLMs have a habit of writing documentation in the comments. Documentation is supposed to be in the documentation, not in the code. LLMs are often trained on things like tutorials, where super obvious statements are commented to allow people to learn and follow along. In actual code you absolutely do not do this, obvious statements should be obvious by themselves. At best it’s extra work to read and maintain the comments for obvious statements, at worst they are incorrect and misleading. I’ve worked on systems where the comments and the code weren’t in line with each other and it was a continual guess if the comment is the way it was supposed to work, or if the code is correct and the comment wrong.

    So when do you actually add comments? That’s actually very hard, something people argue about all the time and a bit of an art form to get right. For example if I have some sort of complex calculation, but it’s based on a well known algorithm, I might comment the name of that algorithm. That way I can recognize it myself right away and someone that doesn’t know it can look it up right away. Another good indicator for comments are magic numbers. It’s often smart to put these in constants, so you can at least name them, but a small little comment to indicate why it’s there and the source can be nice. Or when there is a calculation and there’s a +1 for example in there somewhere, one might ask why the +1, then a little comment is nice to explain why.

    Comments should also serve like a spidey sense for developers. Whenever you are writing comments or have the urge to add some comments somewhere, it might be an indicator the code is messy and needs to be refactored. Comments should be short and to the point, whenever you start writing sentences, either start writing documentation or look at the code why it’s required to explain so much and how to fix that.

    Another good use for comments is to warn away instincts for future devs. For example in a system I worked on there is a large amount of code that seems like it’s duplicate. So a new dev might look at it and see a good place to start refactoring and remove the duplicated code. However the duplication was intentional for performance reasons, so a little comment saying the dupe is intentional is a good idea.

    I’ve also seen comments used to describe function signatures, although most modern languages have official ways of doing that these days. These also might border on documentation, so I’d be careful with that.

    LLMs also have a habit of writing down responses to prompts in the comments. For example the LLM might have written some code, you say: Hey that’s wrong, we shouldn’t set x to y, we should set it to z. And the LLM writes a comment like // X now set to Z as requested. These kinds of comments make no sense to people reading the code in the future.

    Keep in mind comments are there to make it easier for the next guy to work on the code, and often that next guy is you. So getting it right is important and hard, but very much worth while. What I like to do is write code one day and then go back and read it the next day or a few days later. And not the commit, with the diff and the description, the actual files beginning to end. When I think something is weird or stands out, I’ll go back and edit the code and perhaps add comments.

    IMHO LLMs are terrible at writing code, it’s often full of mistakes and oversights, but one of the worst parts is the comments. I can tell code was AI generated right away by the comments and those comments being present are a good indicator the “dev” didn’t bother to actually read and correct the code.

    • jtrek@startrek.website
      link
      fedilink
      arrow-up
      31
      ·
      2 days ago

      There’s also the class of comments that explain strange business decisions.

      # Product guy said to return January 1st if the user doesn't have a birthday on record

      Kind of arbitrary but someone with decision making power decreed it.

      • baines@lemmy.cafe
        link
        fedilink
        English
        arrow-up
        14
        ·
        2 days ago

        this is among the most important comment types that exist imo

        it may not seem like it but 20 years and 5million lines later when shit behaves odd because of some non standard code practice this type of comment is a life saver

      • Skullgrid@lemmy.world
        link
        fedilink
        arrow-up
        8
        ·
        2 days ago

        Kind of arbitrary but someone with decision making power decreed it.

        Here, under protest is this code. Every time the user doesn’t have a birthday, we return January 1st. Do we really mean that?

    • NotAnonymousAtAll@feddit.org
      link
      fedilink
      arrow-up
      8
      ·
      2 days ago

      Excellent comment and I fully agree with almost everything. Just one tiny nitpick:

      For example if I have some sort of complex calculation, but it’s based on a well known algorithm, I might comment the name of that algorithm.

      Unless you really need this complex calculation to be inline (e.g. for performance reasons) it would be better to move it into a function or method and include the algorithm in the name instead of adding a comment.

      • Thorry@feddit.org
        link
        fedilink
        arrow-up
        3
        ·
        2 days ago

        Very good! Your spidey senses are working perfectly. Hey I want to comment this calculation, why don’t I move it into a function so the name can explain what it does. Good call!

        Sometimes the algorithm is inlined for performance, sometimes it’s a class with a bunch of functions that as a whole is primarily based on an algorithm, so comments might make sense in those cases. Most of the times it’s a library, so the name of the library kinda gives it away and hopefully has good documentation as well.

      • baines@lemmy.cafe
        link
        fedilink
        English
        arrow-up
        3
        ·
        2 days ago

        i want it in a function

        fully include the pre implementation long hand in comments and the book / expert reference

        the number of times i’ve found longer algos not doing as advertised is scary

        improperly used statistical algos given the data sets and hand waving results are so common

        gee i wonder why testing shows no improvement

    • Dumhuvud@programming.dev
      link
      fedilink
      English
      arrow-up
      3
      arrow-down
      2
      ·
      1 day ago

      Comments aren’t meant to be documentation

      Documentation is supposed to be in the documentation, not in the code.

      Some tooling generates documentation from comments. Like rustdoc or LDoc.

      I’ll be honest, I mostly skimmed through your comment. Sorry if it’s something you touch upon later on.

    • stingpie@lemmy.world
      link
      fedilink
      arrow-up
      2
      ·
      2 days ago

      I disagree that comments are an indication of messy code. The purpose of comments are to 1) translate low level concepts into higher ones for improved readability and 2) to maintain information in the design process that can’t be represented by the code itself.

      Obviously I don’t have to comment mathematical operations, but it makes sense to comment the algebra of how I derived an equation and it’s use in the code. Now, I could refactor that section of code into a function in order to give it a name, but that would make it more difficult to read as the programmer would then have to find the definition of that function somewhere else in the file. It is objectively more spaghetti-like to pull out single-use code into a function rather than just label the block of code with a comment.

  • slazer2au@lemmy.world
    link
    fedilink
    English
    arrow-up
    26
    ·
    2 days ago

    Bill doesn’t understand liability clauses in contracts when he sells his vibe coded shit as a SaaS platform and gets popped because AI forgot about SQL injection.

  • kibiz0r@midwest.social
    link
    fedilink
    English
    arrow-up
    21
    ·
    2 days ago

    “Did you just have Claude ship all this code?”

    “Yep”

    “You shouldn’t ship code you don’t understand…”

    “Good point. Claude, go understand this code for me!”

    • sekki@lemmy.world
      link
      fedilink
      arrow-up
      5
      ·
      2 days ago

      If Claude understand code, Claude ship code just fine. If Claude understand but you don’t, you no ship code, let Claude do!

    • CluckN@lemmy.world
      link
      fedilink
      arrow-up
      1
      ·
      2 days ago

      Got pulled into a meeting where a web dev contractor we hired stated, “I can’t promise these changes, it depends on how we can get CoPilot to do them”. I don’t think my bosses care because they want their bosses to know that they are, “working with AI”.

  • jaybone@lemmy.zip
    link
    fedilink
    English
    arrow-up
    18
    arrow-down
    1
    ·
    2 days ago

    A lot of big companies are basically forcing you to be Bill. They try to track AI usage, and make sure everyone is using it. Because they invested so much in AI, they need to force it, to make it a self fulfilling prophecy. It’s truly one of the dumbest things to behold.

    • sobchak@programming.dev
      link
      fedilink
      arrow-up
      1
      ·
      1 day ago

      If the company also trains AI, I’m guessing they’re also forcing use to gather training data. I.e. tracking feedback, corrections, etc.

    • kindnesskills@literature.cafe
      link
      fedilink
      arrow-up
      5
      ·
      2 days ago

      Nightmarish. Can you make the AI write up new documentation every time you want to push a change, so it looks like you’re using it frequently but you still get to write the code yourself?

      I would love letting LLM deal with documentation. It’s the bane of my existence.

      • Enkrod@feddit.org
        link
        fedilink
        arrow-up
        1
        ·
        1 day ago

        Tried that, result was shitty documentation and as someone who reads other peoples documentation for a living I will NOT subject other people or my future self to that.

  • tiredofsametab@fedia.io
    link
    fedilink
    arrow-up
    4
    ·
    1 day ago

    I recently got moved to another team after my company restructured. Several repos have zero documentation. Most of those don’t even have comments on the functions. LinkXxxx(some args) like Xxxx to /what?!/ It’s also an over-engineered mess with multiple layers of abstraction. I can’t wait to finish figuring out what everything does and re-write (and document it) like a sane person. This code presumably had no AI involvement which I’d argue is even worse since real humans made these shit decisions. Don’t get me starting on their testing (and mostly lack thereof)…

    Worker suggested using AI to write some documentation. Another coworker did. I immediately spotted a bunch of hallucinated shit. Good times. I want to know in my head what a thing does, how it works, and how it fits into the architecture. I can’t do that if an AI is just deciding stuff; it’s like a quiz back in school where I would memorize shit for about one week before 99% of it left my brain forever.

    • psud@aussie.zone
      link
      fedilink
      English
      arrow-up
      2
      ·
      1 day ago

      Sometimes documentation doesn’t help. My workplace used to be waterfall and we still have systems analysts producing documentation

      One piece of work came to my team as “this calculation fails in this case and we can’t tell why”

      Looking at it there were two updates in the documentation showing updates ten years ago the first fixing that corner case with the second fixing a side effect of the fix

      Why didn’t it work? It was never built, or perhaps built but never merged

      On the good side we have an excellent plan for how to fix that corner case, on the bad side we aren’t funded to do that amount of work so we have to half arse a solution

      • tiredofsametab@fedia.io
        link
        fedilink
        arrow-up
        1
        ·
        11 hours ago

        The documentation I mean here should be in the repo and checked as a part of the PR process. I don’t mean it should be a manual, I just mean a readme should at least describe what the service does (it has such a generic name in my case that it’s not self-evident and is so broad in scope I don’t know what happened). The functions also have terrible naming and no comments to describe what they do. So, as a guy just coming into this team after re-org with no idea what all this stuff does, I was completely lost. They also changed which services which teams owned, further making it difficult to get knowledge.

  • 🌞 Alexander Daychilde 🌞@lemmy.world
    link
    fedilink
    English
    arrow-up
    7
    arrow-down
    2
    ·
    2 days ago

    Woever made this image should learn the difference between line spacing and paragraph spacing…

    This is an example
    of line spacing which
    clearly shows it’s together

    And now this is a new
    bit that is separate from
    the above bit

    Readability is important.