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. Try Catch - at least it won't break

Try Catch - at least it won't break

Scheduled Pinned Locked Moved The Weird and The Wonderful
asp-net
4 Posts 4 Posters 0 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.
  • R Offline
    R Offline
    RCoate
    wrote on last edited by
    #1

    Just found this piece of quality in our production code.

        try
        {
            Class1.NullableInt1 = int.Parse(txtSomeUserInput.Text);
        }
        catch
        {
            Class1.NullableInt1 = null;
        }
        ...
        //repeat for each nullable int in the class.
    

    NB: Class, property and input names changed to protect the inocent. And people here wonder why our web forms can take 1 minute or more to post back!

    L B 2 Replies Last reply
    0
    • R RCoate

      Just found this piece of quality in our production code.

          try
          {
              Class1.NullableInt1 = int.Parse(txtSomeUserInput.Text);
          }
          catch
          {
              Class1.NullableInt1 = null;
          }
          ...
          //repeat for each nullable int in the class.
      

      NB: Class, property and input names changed to protect the inocent. And people here wonder why our web forms can take 1 minute or more to post back!

      L Offline
      L Offline
      leppie
      wrote on last edited by
      #2

      Sadly, the sucker never heard of the Convert class...

      ((λ (x) `(,x ',x)) '(λ (x) `(,x ',x)))

      G 1 Reply Last reply
      0
      • L leppie

        Sadly, the sucker never heard of the Convert class...

        ((λ (x) `(,x ',x)) '(λ (x) `(,x ',x)))

        G Offline
        G Offline
        GibbleCH
        wrote on last edited by
        #3

        or TryParse ...

        1 Reply Last reply
        0
        • R RCoate

          Just found this piece of quality in our production code.

              try
              {
                  Class1.NullableInt1 = int.Parse(txtSomeUserInput.Text);
              }
              catch
              {
                  Class1.NullableInt1 = null;
              }
              ...
              //repeat for each nullable int in the class.
          

          NB: Class, property and input names changed to protect the inocent. And people here wonder why our web forms can take 1 minute or more to post back!

          B Offline
          B Offline
          BillW33
          wrote on last edited by
          #4

          Seeing far too much code like that is what made me choose my signature line.

          Just because the code works, it doesn't mean that it is good code.

          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