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. The Weird and The Wonderful
  4. Quantity FTW!

Quantity FTW!

Scheduled Pinned Locked Moved The Weird and The Wonderful
announcementdatabasesql-servertoolshelp
6 Posts 4 Posters 1 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.
  • B Offline
    B Offline
    Brady Kelly
    wrote on last edited by
    #1

    Or, "How not to do your release notes.":

                str3 = str3 + "<tr><td align='center'><b> Version 2.1.101 • 15 Sep 2007 </b></td></tr>" +
                       "<tr><td>&nbsp;</td></tr>" +
                       "<tr><td>&nbsp;</td>• Resolved script error on Reports Viewed List Report</tr>" +
                       "<tr><td>&nbsp;</td>• Resolved error when adding level using MSSQL</tr>" +
    

    ... [snip]
    "<tr><td> Corrected Directory Listing query to display Extension Number of Parent Extension </td></tr>" +
    "<tr><td> </td></tr>";

    730 lines have been removed from the string literal to protect everyone, not just the innocent.

    You really gotta try harder to keep up with everyone that's not on the short bus with you. - John Simmons / outlaw programmer.

    T I J 3 Replies Last reply
    0
    • B Brady Kelly

      Or, "How not to do your release notes.":

                  str3 = str3 + "<tr><td align='center'><b> Version 2.1.101 • 15 Sep 2007 </b></td></tr>" +
                         "<tr><td>&nbsp;</td></tr>" +
                         "<tr><td>&nbsp;</td>• Resolved script error on Reports Viewed List Report</tr>" +
                         "<tr><td>&nbsp;</td>• Resolved error when adding level using MSSQL</tr>" +
      

      ... [snip]
      "<tr><td> Corrected Directory Listing query to display Extension Number of Parent Extension </td></tr>" +
      "<tr><td> </td></tr>";

      730 lines have been removed from the string literal to protect everyone, not just the innocent.

      You really gotta try harder to keep up with everyone that's not on the short bus with you. - John Simmons / outlaw programmer.

      T Offline
      T Offline
      Tristan Rhodes
      wrote on last edited by
      #2

      Wow... just wow. As me and my brother debate about regularly, at what point would a 'normal' person stop, take a step back, and say "really... wtf am i doing?". And what should we do with the non-normal people that either keep going for a REEEEEAAAALLLLYY long time, or worse yet, never hit that mark? From both our experience, it would appear to be - make them CTO.

      ------------------------------- Carrier Bags - 21st Century Tumbleweed.

      1 Reply Last reply
      0
      • B Brady Kelly

        Or, "How not to do your release notes.":

                    str3 = str3 + "<tr><td align='center'><b> Version 2.1.101 • 15 Sep 2007 </b></td></tr>" +
                           "<tr><td>&nbsp;</td></tr>" +
                           "<tr><td>&nbsp;</td>• Resolved script error on Reports Viewed List Report</tr>" +
                           "<tr><td>&nbsp;</td>• Resolved error when adding level using MSSQL</tr>" +
        

        ... [snip]
        "<tr><td> Corrected Directory Listing query to display Extension Number of Parent Extension </td></tr>" +
        "<tr><td> </td></tr>";

        730 lines have been removed from the string literal to protect everyone, not just the innocent.

        You really gotta try harder to keep up with everyone that's not on the short bus with you. - John Simmons / outlaw programmer.

        I Offline
        I Offline
        Ian Shlasko
        wrote on last edited by
        #3

        So not only did he use over 700 unnecessary string concatenations, but he couldn't even put the text INSIDE the table cells? I must say I'm impressed... It takes real talent to make a coding horror that spans two languages! (Three, if you count English -- Extension Number of Parent Extension? Is that from the Department of Redundancy Department?)

        1 Reply Last reply
        0
        • B Brady Kelly

          Or, "How not to do your release notes.":

                      str3 = str3 + "<tr><td align='center'><b> Version 2.1.101 • 15 Sep 2007 </b></td></tr>" +
                             "<tr><td>&nbsp;</td></tr>" +
                             "<tr><td>&nbsp;</td>• Resolved script error on Reports Viewed List Report</tr>" +
                             "<tr><td>&nbsp;</td>• Resolved error when adding level using MSSQL</tr>" +
          

          ... [snip]
          "<tr><td> Corrected Directory Listing query to display Extension Number of Parent Extension </td></tr>" +
          "<tr><td> </td></tr>";

          730 lines have been removed from the string literal to protect everyone, not just the innocent.

          You really gotta try harder to keep up with everyone that's not on the short bus with you. - John Simmons / outlaw programmer.

          J Offline
          J Offline
          Jorgen Sigvardsson
          wrote on last edited by
          #4

          That's highly inefficient too. This is much more efficient:

          str3 = "<tr><td align='center'><b> Version 2.1.101 • 15 Sep 2007 </b></td></tr>"
          "<tr><td> </td></tr>"
          ...
          "<tr><td> </td></tr>";

          That way, the compiler will be able to do the string joins at compile time, rather than at runtime using the overloaded + operator. I'm assuming this is C++ and not C#. The initial str + would not have been needed in C#...

          -- Kein Mitleid Für Die Mehrheit

          B 1 Reply Last reply
          0
          • J Jorgen Sigvardsson

            That's highly inefficient too. This is much more efficient:

            str3 = "<tr><td align='center'><b> Version 2.1.101 • 15 Sep 2007 </b></td></tr>"
            "<tr><td> </td></tr>"
            ...
            "<tr><td> </td></tr>";

            That way, the compiler will be able to do the string joins at compile time, rather than at runtime using the overloaded + operator. I'm assuming this is C++ and not C#. The initial str + would not have been needed in C#...

            -- Kein Mitleid Für Die Mehrheit

            B Offline
            B Offline
            Brady Kelly
            wrote on last edited by
            #5

            It's decompiled C#.

            You really gotta try harder to keep up with everyone that's not on the short bus with you. - John Simmons / outlaw programmer.

            J 1 Reply Last reply
            0
            • B Brady Kelly

              It's decompiled C#.

              You really gotta try harder to keep up with everyone that's not on the short bus with you. - John Simmons / outlaw programmer.

              J Offline
              J Offline
              Jorgen Sigvardsson
              wrote on last edited by
              #6

              Whoa.. It didn't even optimize the literals when it was compiled? Did someone forget to turn on the optimizer? :~

              -- Kein Mitleid Für Die Mehrheit

              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