Converter for deserializing JSON

Question

I'm writing a custom JavaScript converter to deserialize some JSON. Here's a few lines I'm writing:

public override object Deserialize(IDictionary<string, object> dictionary, Type type, 
      JavaScriptSerializer serializer)
  {
    int NoteID;
    NotesModel TheObject = new NotesModel();

    if (dictionary.ContainsKey("NoteID"))
    {
      if (int.TryParse(serializer.ConvertToType<string>(dictionary["NoteID"]), 
          out NoteID))
      {
         TheObject.NoteID = serializer.ConvertToType<int>(dictionary["NoteID"]);
      }
    }

How can this be improved?

Answer 1

you can do

TheObject.NoteID = NoteID;

Answer 2

It would be like this, you are already converting it to int with the tryParse

public override object Deserialize(IDictionary<string, object> dictionary, Type type, JavaScriptSerializer serializer)
  {
    int noteID;
    NotesModel TheObject = new NotesModel();

    if (dictionary.ContainsKey("NoteID"))
    {
      if (int.TryParse(serializer.ConvertToType<string>(dictionary["NoteID"]), out noteID))
      {
         TheObject.NoteID = noteId;
      }
    }
}

or even shorter

public override object Deserialize(IDictionary<string, object> dictionary, Type type, JavaScriptSerializer serializer)
  {
    int noteID;
    NotesModel TheObject = new NotesModel();

    if (dictionary.ContainsKey("NoteID") && int.TryParse(serializer.ConvertToType<string>(dictionary["NoteID"]), out NoteID))
      {
         TheObject.NoteID = noteId;
      }
    }

Answer 3

I would recommend the following...

public override object Deserialize(IDictionary<string, object> dictionary, Type type, JavaScriptSerializer serializer) 
  { 
    int noteID; 
    NotesModel TheObject = new NotesModel(); 

    object value;

    if (dictionary.TryGetValue("NoteID", out value)) 
    { 
      if (int.TryParse(serializer.ConvertToType<string>(value), out NoteID)) 
      { 
         TheObject.NoteID = noteId; 
      } 
    } 
}

This should cut your dictionary query by half.