Error handling inbouwen: goed idee.

Error handling inbouwen: goed idee.

Error handling inbouwen en niet testen: geen goed idee.

System.Runtime.Serialization.IFormatter formatter = new System.Runtime.Serialization.Formatters.Binary.BinaryFormatter();
if (stream!=null
&& stream.Length > 0)
  {
   
try 
{ settings=(Hashtable)formatter.Deserialize(stream); }
   
catch
{ }
  }
  if (settings==null) return new
Hashtable();
  stream.Close();
  isoFile.Close();
  return
settings;
}

Slim hé? Ja, nog naast de kwestie dat ik niets echt constructiefs aanvang met die try/catch: als er geen settings zijn stuur ik een nieuwe hashtable terug. En als er wel zijn, dan sluit ik de stream en de file, en stuur ik de ingelezen en gedeserialiseerde settings terug.

Aaargh! Die stream en die file moet in alle geval hoedanook gesloten worden!! Grrr.

Zeg uw gedacht

Navigatie

Vorige entry:

Volgende entry:

» homepagina, archief

Vriendjes

<insert standard disclaimer>

Alles wat hier staat is mijn eigen opinie. Het wordt niet nagelezen of goedgekeurd door mijn werkgever voor het on-line komt, en ik bied geen enkele garantie voor kwaliteit of correctheid.

Mijn werkgever is het niet noodzakelijk eens met wat ik schrijf, en het spreekt vanzelf dat hij dan ook op geen enkele wijze aansprakelijk kan zijn voor wat ik hier publiceer.