Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • World
  • Users
  • Groups
Skins
  • Light
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse
Code Project
  1. Home
  2. Other Discussions
  3. Buzzword Bingo
  4. If it ain't broken don't fix it!

If it ain't broken don't fix it!

Scheduled Pinned Locked Moved Buzzword Bingo
performancehelpquestion
5 Posts 3 Posters 7 Views 1 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • Z Offline
    Z Offline
    zgonja
    wrote on last edited by
    #1

    Well, I'm going through this programmable data pump and trying to get a hang of it, but the butcher that put it together's really got me goin' nuts, cause whenever I think some piece of code has secret purpose behind it and put some time into investigating what the heck it's doin' I end up looking at something like this: if(sNewMaintStatus==null) { Handler.execute(ixhandler, ixEvent, sSourceConnectionName, sDestinationConnectionName,sNewSpirnum, sSourceUser, sDestUser); } else if((sNewMaintStatus.equals("PARTIAL")) || sNewMaintStatus.equals("") || sNewMaintStatus.equals(null)) { Handler.execute(ixhandler, ixEvent, sSourceConnectionName, sDestinationConnectionName,sNewSpirnum, sSourceUser, sDestUser); } Which translates to: if (A=true) Execute block 1 else if (B=true || A=true || A=true) Execute block 1 Which in normal language translates to if (A=true || B=true) Execute block 1 So I called my boss, and I told him, listen man, this's a discrase, client's complainin' bout' the performance and reliablility and so on and he said IF IT AIN'T BROKEN WELL DON'T FIX IT :omg: ;P :wtf: Now does that sound familiar to you? Well, I had to share the pain!

    P P 2 Replies Last reply
    0
    • Z zgonja

      Well, I'm going through this programmable data pump and trying to get a hang of it, but the butcher that put it together's really got me goin' nuts, cause whenever I think some piece of code has secret purpose behind it and put some time into investigating what the heck it's doin' I end up looking at something like this: if(sNewMaintStatus==null) { Handler.execute(ixhandler, ixEvent, sSourceConnectionName, sDestinationConnectionName,sNewSpirnum, sSourceUser, sDestUser); } else if((sNewMaintStatus.equals("PARTIAL")) || sNewMaintStatus.equals("") || sNewMaintStatus.equals(null)) { Handler.execute(ixhandler, ixEvent, sSourceConnectionName, sDestinationConnectionName,sNewSpirnum, sSourceUser, sDestUser); } Which translates to: if (A=true) Execute block 1 else if (B=true || A=true || A=true) Execute block 1 Which in normal language translates to if (A=true || B=true) Execute block 1 So I called my boss, and I told him, listen man, this's a discrase, client's complainin' bout' the performance and reliablility and so on and he said IF IT AIN'T BROKEN WELL DON'T FIX IT :omg: ;P :wtf: Now does that sound familiar to you? Well, I had to share the pain!

      P Offline
      P Offline
      PIEBALDconsult
      wrote on last edited by
      #2

      Fix it anyway, that's what I do.

      1 Reply Last reply
      0
      • Z zgonja

        Well, I'm going through this programmable data pump and trying to get a hang of it, but the butcher that put it together's really got me goin' nuts, cause whenever I think some piece of code has secret purpose behind it and put some time into investigating what the heck it's doin' I end up looking at something like this: if(sNewMaintStatus==null) { Handler.execute(ixhandler, ixEvent, sSourceConnectionName, sDestinationConnectionName,sNewSpirnum, sSourceUser, sDestUser); } else if((sNewMaintStatus.equals("PARTIAL")) || sNewMaintStatus.equals("") || sNewMaintStatus.equals(null)) { Handler.execute(ixhandler, ixEvent, sSourceConnectionName, sDestinationConnectionName,sNewSpirnum, sSourceUser, sDestUser); } Which translates to: if (A=true) Execute block 1 else if (B=true || A=true || A=true) Execute block 1 Which in normal language translates to if (A=true || B=true) Execute block 1 So I called my boss, and I told him, listen man, this's a discrase, client's complainin' bout' the performance and reliablility and so on and he said IF IT AIN'T BROKEN WELL DON'T FIX IT :omg: ;P :wtf: Now does that sound familiar to you? Well, I had to share the pain!

        P Offline
        P Offline
        Pete OHanlon
        wrote on last edited by
        #3

        zgonja wrote:

        IF IT AIN'T BROKEN WELL DON'T FIX IT

        The consultancy corrolary is: If it ain't broke, break it and charge massive amounts to remove your foul ups.

        Deja View - the feeling that you've seen this post before.

        My blog | My articles

        Z P 2 Replies Last reply
        0
        • P Pete OHanlon

          zgonja wrote:

          IF IT AIN'T BROKEN WELL DON'T FIX IT

          The consultancy corrolary is: If it ain't broke, break it and charge massive amounts to remove your foul ups.

          Deja View - the feeling that you've seen this post before.

          My blog | My articles

          Z Offline
          Z Offline
          zgonja
          wrote on last edited by
          #4

          Hahaha, yes, that's sad but very very true indeed! All too true if you ask me. :((

          1 Reply Last reply
          0
          • P Pete OHanlon

            zgonja wrote:

            IF IT AIN'T BROKEN WELL DON'T FIX IT

            The consultancy corrolary is: If it ain't broke, break it and charge massive amounts to remove your foul ups.

            Deja View - the feeling that you've seen this post before.

            My blog | My articles

            P Offline
            P Offline
            PIEBALDconsult
            wrote on last edited by
            #5

            Consulting[^]

            1 Reply Last reply
            0
            Reply
            • Reply as topic
            Log in to reply
            • Oldest to Newest
            • Newest to Oldest
            • Most Votes


            • Login

            • Don't have an account? Register

            • Login or register to search.
            • First post
              Last post
            0
            • Categories
            • Recent
            • Tags
            • Popular
            • World
            • Users
            • Groups