Stories
Slash Boxes
Comments

News for nerds, stuff that matters

Slashdot Log In

Log In

Create Account  |  Retrieve Password

IBM Trying To Patent Timed Code Inspection

Posted by kdawson on Sat Apr 26, 2008 09:15 PM
from the walking-through-a-patent-minefield dept.
theodp writes "A just-published IBM patent application for a Software Inspection Management Tool claims to improve software quality by taking a chess-clock-like approach to code walkthroughs. An inspection rate monitor with 'a pause button, a resume button, a complete button, a total lines inspected indication, and a total lines remaining to be inspected indication' keeps tabs on participants' progress and changes color when management's expectations — measured in lines per hour — are not being met."
+ -
story

Related Stories

This discussion has been archived. No new comments can be posted.
The Fine Print: The following comments are owned by whoever posted them. We are not responsible for them in any way.
 Full
 Abbreviated
 Hidden
More
Loading... please wait.
  • by Anonymous Coward on Saturday April 26 2008, @09:17PM (#23210880)
    I know I'm most careful and attentive to detail when I feel like someone's looking over my shoulder timing my progress.
    • I know I'm most careful and attentive to detail when I feel like someone's looking over my shoulder timing my progress.

      Most people are more "careful and attentive to" looking like they're busy, as opposed to actually thinking about the problem at hand, when someone's looking over their shoulder.

      Yes, we need better metrics to determine performance. However, we should do our Jedi Knight code warrior hand-waving thing and say "these are not the metrics you're looking for." This is a sop to cover up inadequacies (in both the people managing, and their methodologies) the previous steps to the development process. Where is the story mod, with a "-1 Fucktarded" option, when we need it?

    • If it means that only IBM employees have to put up with this particular form of pointy-haired manager bullshit.

      -jcr
  • by NialScorva (213763) on Saturday April 26 2008, @09:18PM (#23210884)
    which they'll use to sue anyone who's dumb enough to use this technique
  • by Anonymous Coward

    changes color when management's expectations...are not being met.
    Translation: Your PHB needs it color coded to know what's going on.
  • Note to self: (Score:5, Insightful)

    by Weaselmancer (533834) on Saturday April 26 2008, @09:21PM (#23210906)

    Don't buy any IBM software after this awful thing gets approved.

    And in case any management types happen to be reading this - programming isn't freaking bricklaying. You can't say "well the wall needs 120 bricks, and 1 person can lay 1 brick in one minute, so that's two hours work. Or 1 hour's worth of work for two people."

    Read this book, [wikipedia.org] and then get back to us IBM.

    • I've given you 9 women and 6 weeks...... WHERE IS THE BABY !!!!
        • ... or seek assistance from certain overseas adoption agencies that might be interested in such a trade.
          ...and we are back to outsourcing....

          • It is a Soviet myth. It goes like this, Soviet was a plan economy and the polit bureau told the screw factory manager that they had to produce 10 metric tonnes of screws this year. So they made one (1) REALLY BIG screw.
    • Re: (Score:3, Insightful)

      well the wall needs 120 bricks, and 1 person can lay 1 brick in one minute, so that's two hours work. Or 1 hour's worth of work for two people.

      Unfortunately that is how "executive students" in crackerjack MBA degree programs around the country are taught to think about everyone else except themselves. Is it any wonder then that most of the top managers, the best board members, and the most intelligent CEOs are self made and taught men and women? The person who believes and applies exactly what is taught in business school without intelligent thought and reasonable allowance for the circumstances is not really an MBA, but an employee in manager's

      • by njcoder (657816) on Saturday April 26 2008, @09:52PM (#23211086)
        Do you hear that? That's the loud roar of all the people you just described patting themselves on the back for not being one of the people you just described.
      • When I was an undergraduate engineering student I took courses in management from the institute's management school. I did well in the course gradewise but I recognize now that back then I had absolutely NO context for understanding the philosophies and practices. Only after a bunch of experience and many years could the views be properly integrated into reality. Any MBA student coming to management theory right out of four years of college really can't well apply it no matter how smart or slick they think
        • LAny MBA student coming to management theory right out of four years of college really can't well apply it no matter how smart or slick they think they are.

          Right, so they subsitute technology like this code-review chess clock for actual leadership and management and keep track of useless statistics in spreadsheets in a vain attempt to look smart or at least report something to the ones who are signing their checks. Meanwhile the engineers are pissed off and begin to undermine the new system on purpose (usually in really clever ways that keep it going but twist its purpose) to subtly protest the continued waste of time in a fruitless effort to save time.

      • Re: (Score:3, Insightful)

        The person who believes and applies exactly what is taught in business school without intelligent thought and reasonable allowance for the circumstances is not really an MBA, but an employee in manager's clothing who is [in over his head ===> overhead] and is too arrogant and too foolish to admit it, even to him or herself.

        Fixed that for you.

    • Oh how we've come full circle:

      http://en.wikipedia.org/wiki/The_Mythical_Man-Month [wikipedia.org]

      "Brooks's observations are based on his experiences at IBM while managing the development of OS/360"

    • Don't buy any IBM software after this awful thing gets approved.

      1. IBM is one of the founders of the http://www.patentcommons.org/ [patentcommons.org]

      2. IBM is one of the world's largest consulting operations (THE largest?) and as such they work on an unbelievably wide array of projects, and this could just be one of the more random ones.

      3. If you stopped buying IBM software every time you didn't like some product that one of their divisions thought about, then you'd have stopped buying all of their software around the time you were born.

      Your over-reaction, here, is just a bit theatrical.

    • Caution: I've read Mythical Man Month and would like to disagree with it's applicability to the subject of this article.

      One of the tenets of MMM is that adding warm bodies to a software project that is behind schedule will (counter-productively) make that project fall even further behind schedule because of the time required to train the newbies.

      The article actually sounds like it is talking about Michael Fagan-esque inspection bullcrap which is more closely related to improving software quality. Essen

  • ...like not being able to spend significantly more time on the tricky sections of code than on the routine stuff.
  • by Baldrson (78598) * on Saturday April 26 2008, @09:26PM (#23210938) Homepage Journal
    I get dibs on the patent for measuring whether programmers meet management expectations for lines of code per hour!

    In fact, I think programmers should meet management expectations for keystrokes per second including backspace! I mean backspace means the programmer is correcting his mistakes!!!

    • Some years ago I worked as an engineer for a company in Dearborn, Michigan that makes a major consumer durable good. My supervisor was a rather earnest Eastern European emigre whose affect loosely resembled the "intense" comic personna of Gilbert Godfried.

      We had a PDP-10 timesharing system, and I was keying in a simulation program for a fluid-flow problem into one of those clunky Model 33 Teletype terminals. My supervisor came and stood in the door of the terminal room, cigarette between his lips (this

    • I get dibs on the patent for measuring whether programmers meet management expectations for lines of code per hour!

      OK but I call dibs on the patent for software that measures if managers are alienating and burning out their underlings at the specified rate. Product producing employees need a high turnover rate to keep our product "fresh" and "innovative". Do you like my new tie? It matches my BMW.
    • I want patents on all the software released using this methodology.

      Judge: "Can you prove this software is yours?"

      Me: "Just look at it! It sucks! Any questions?"

      Judge: "OK, Mr. Gates - Pay up."
  • "An inspection rate monitor with 'a pause button, a resume button, a complete button, a total lines inspected indication, and a total lines remaining to be inspected indication' keeps tabs on participants' progress and changes color when management's expectations â" measured in lines per hour â" are not being met."

    Right Wally. so, you write line after line of utterly meaningless horseshit to keep your line count up, and then route the data to a place where it gets commented out... and then follo

    • Right Wally. so, you write line after line of utterly meaningless horseshit to keep your line count up, and then route the data to a place where it gets commented out... and then follow that with line after line of craptastic documentation.

      Bingo, sir. Programmers are excellent at beating the system. That's our job. Figuring out systems.

      You're gonna see a lot of code like this:

      for(
      // i holds the count
      int i=0;
      // The next line makes sure we count to ten
      i<=10;
      // This increments the count
      i++)
      {
      // Let's do our function.
      doWork();
      // And close the loop.
      }

      Sounds like a real blast, IBM. Good thinking.

      • Agreed. There are already too many programmers who have copy/pasted when they should have extracted the common code into something that was re-usable.

        If this IBM method were implemented, I can just see it...

        CTRL-C, CTRL-V

        It's code reuse! It's a performance improvement! The program will be faster since it doesn't have to worry about a function call! And look, I'm at least 20 times more productive than all of the other programmers by the number of lines I check in to the source control.

        Good grief.
        • I think this method is already used by every outsourcing company in existence. They justify their existence and their cost-effectiveness to management by showing the sheer size of the program they created. Surely a program THAT big couldn't be bad, could it? Of course not! And for such great prices per line!

          enum Boolean {
            True,
            False,
            FileNotFound
          }
      • Brilliant! I'll implement this and finally all the code will be written with documentation!
        • Thats been fixed where I work. We get evaluated on lines of code but the code counter excludes comments because, well, they don't do anything.
          • If I may, of course.

            First question: Excluding comments, would my example above count as 7 lines? Does my busted up for loop count for multiple lines?

            Second question: You looking for another job yet? Holy crap but I can't imagine working under those conditions.

          • Obviously you need to write comments as strings.

            "Loops over all the nodes"
            for node in nodes:
                "Checks if the node is valid"
                if node.valid():
                    "Prints the node if it is valid"
                    print node
            "End 'for node in nodes'"
    • by arth1 (260657) on Saturday April 26 2008, @09:50PM (#23211076) Homepage Journal
      Someone I know was told to increase lines-of-code output by 20%, not counting comments. He immediately complied and exceeded expectations by switching to K&R style, declaring function variables on separate lines instead of inline, and placing { on a separate line.
      He was ready to place semicolons on a separate line too, if management wanted even higher efficiency.
    • by snl2587 (1177409) on Saturday April 26 2008, @10:23PM (#23211196)

      If I am understanding this correctly, the article refers to lines inspected, not written. So this is for the quality-control guys and not the main programmers.

      • Invite quality control guys to a code inspection and they will occassionally find coding violations that automated tools should be picking up. Thus, all they will figure out is that you aren't running static analysis tools on your code prior to the inspection.

        Invite programmers to the inspection, and you'll figure out that the interface that you assume you will be using to communicate with another subsystem has actually been completely redone and that you'd better write a wrapper to fix things or else th

  • Quality... (Score:4, Insightful)

    by symbolic (11752) on Saturday April 26 2008, @09:35PM (#23210994)
    claims to improve software quality by taking a chess-clock-like approach to code walkthroughs

    This is one of those patents based clearly on conjecture. Seriously - is there anyone stupid enough to try it, or anyone stupid enough to work in an environment that relies on this kind of QA?
    • Re: (Score:3, Interesting)

      perhaps IBM is not being evil and is merely patenting this to ensure that no one will be able to use this "method".
    • I, Cringely [pbs.org] paints a picture of IBM [pbs.org] that is quite consistent with the spirit of this invention. Such a timer would increase throughput (as distinct from productivity) and generate an audit trail that "proves" a customer's code was "reviewed". What's a sleazy consultancy whose value is in paperwork and billable hours not supposed to love about this?
  • Metropolis.... (Score:3, Interesting)

    by drolli (522659) on Saturday April 26 2008, @09:41PM (#23211016) Journal
    This reminds me of:

    Metropolis (Fritz Lang, 1927). Excellent movie BTW.

    The worker at the power plant collapses during a a shift which was too long and required to many new operations demanded by a clock-like device. The power plant nearly explodes, because he can not keed up with the pace of this clock-like device.
  • Just look at any pice of code that is a bit harder to understand. It will not have been inspected well.

    Lines of code as performance metric is only for those without any clue at all...
  • I've only flicked through the patent application so far, but it doesn't seem very much like what the submitter makes out.

    From what I can see, the implication that this has anything to do with management harassing the developers and testers is completely conjecture on the part of the slashdot submitter. The only context in which the word "manage" appears in the entire application is as part of the phrase "management tool", which to me implies that it's supposed to be entirely to help the testing and development staff. (Okay, there's one occurance which is "inspection process manager".)

    I know that IBM has a famous history of having associated productivity with lines of code, but I really don't think they're being quite so dim-witted with this one. I haven't read the application in detail, but to me it looks more like someone's been developing a tool to help with code inspection. By the looks of it, it has a certain way of displaying the code, it has a method of recording noted defects and comments, and it has a feature of timing how long things are taking and how long a user is spending on certain parts of a code-base.

    I can't see any direct implication in the patent application that this is primarily for management to measure staff performance to compare with pre-defined expectations. On the other hand I can see a lot of references in the patent application to the code inspector themselves using this tool to assist their work. I think it's much more likely that someone running an inspection could use such a tool to help them keep track of the most fragile parts of the code, and which areas are tying up the most of their time. If there was a deadline for inspection, it'd probably also help to highlight if you were spending far too much time in one place without having even reached other areas that might be important.

    Whether it would work or be any use at all it another issue, but if it's a completely wacky idea then it wouldn't be the first that someone tried to patent. Many good patented ideas seemed silly or ridiculous before a working implementation was produced to demonstrate otherwise, but if an inventor had waited until it was clearly useful before patenting it, it'd be a lot harder.

    • (responding to my own post with more ideas) ...

      I think it's much more likely that someone running an inspection could use such a tool to help them keep track of the most fragile parts of the code, and which areas are tying up the most of their time. If there was a deadline for inspection, it'd probably also help to highlight if you were spending far too much time in one place without having even reached other areas that might be important.

      Other areas where such a tool might be useful are:

      • For a QA tea
  • -2000 Lines Of Code (Score:5, Interesting)

    by theodp (442580) on Saturday April 26 2008, @10:51PM (#23211336)
    After optimizing Quickdraw with a rewrite that saved around 2,000 lines of code, Bill Atkinson dutifully filled out the form Apple management used to measure software productivity, completing the lines of code part by writing in the number -2000 [folklore.org].
  • Seen this (Score:3, Insightful)

    by ichbineinneuben (1065378) on Saturday April 26 2008, @11:27PM (#23211500)
    Isn't this straight out of US Govt coding hell in Snowcrash? http://en.wikipedia.org/wiki/Snowcrash [wikipedia.org]
  • I HOPE they deploy this at work.

    I reckon i could type flat out coding in python at about 20 solid lines per minute. this tool of ibm's would make me look great, and when nothing compiles i can use it as proof i was working hard.

    • Posting as AC because I'm not a fan of burning bridges!

      I wouldn't worry about it. I doubt IBM interviewers read /.