Returning Empty Json Object

Question

I am trying to return a Json object in C#. I am new to MVC controller and using Json first time, I return this object, and its empty.

public class A
{
    private string name;
    public void set(string data)
    {
        name = data;
    }
    public string get()
    {
        return name;
    }
}
public JsonResult Hello()
{
    A obj = new A();
    obj.set("Abc");
    JavaScriptSerializer js = new JavaScriptSerializer();
    string jsonVar = js.Serialize(obj);
    return Json(jsonVar, JsonRequestBehavior.AllowGet);
}

Dustin Kingen · Accepted Answer · 2013-04-09 00:46:14Z

4

In C# we have properties which (in C# 3+) can be automatically created.

public class A
{
    public string Name { get; set; }
}

Secondly you will need to return your object wrapped in a new object in order for the JSON to return correctly. You do not need to serialize the object yourself (since you could just return it as an ActionResult otherwise).

public JsonResult Hello()
{
    A obj = new A();
    obj.Name = "Abc";
    return Json(new { obj }, JsonRequestBehavior.AllowGet);
}

This will create a new Json object {"obj":{"Name":"Abc"}}

answered Apr 9, 2013 at 0:46

Dustin Kingen

21.4k7 gold badges55 silver badges94 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Simon Whitehead · Accepted Answer · 2013-04-08 23:54:49Z

3

You are assuming that the framework can deduce that get and set set the private variable name's value.. it doesn't.

Instead, make name a public property, and it should work:

public class A {
    public string Name { get; set; }
}

A obj = new A() { Name = "Abc" };
/* ...etc... */

Think about this from the framework's point of view. How can it determine what get or set are doing? Are they accessing the same variable? Who knows.. its runtime after all. This is why methods can't be serialized the way you're assuming.

answered Apr 8, 2013 at 23:54

Simon Whitehead

65.3k10 gold badges120 silver badges140 bronze badges

2 Comments

PhantomM Over a year ago

Thank you. If I don't want to make my class member public, Is there other way around ??

Mathieu Guindon Over a year ago

You can make the setter private or protected, but I think serialization requires a public getter (and deserialization would require a public setter).

Mathieu Guindon · Accepted Answer · 2013-04-09 00:54:12Z

1

You shouldn'd be naming methods Get and Set - not even GetName and SetName, because if I remember my C# readings right, the CLR does that by convention when "translating" properties into CIL code (the getter becomes a "get" method and the setter becomes a "set" method). The "right" way is to use properties with meaningful names - you'll thank yourself later:

private string _name;
public string Name { get { return _name; } set { _name=value; } }

answered Apr 9, 2013 at 0:54

Mathieu Guindon

71.5k8 gold badges113 silver badges250 bronze badges

Collectives™ on Stack Overflow

Returning Empty Json Object

3 Answers 3

Comments

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related