jhh1117 / json-simple

Automatically exported from code.google.com/p/json-simple
Apache License 2.0
0 stars 0 forks source link

Suggestion about JSONValue.toJSONString #7

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Hello, It would be useful to include 

        if(value instanceof Object[])
            return JSONArray.toJSONString(Arrays.asList((Object[])value));

I hope you can include this two simple lines

Thanks a lot

Original issue reported on code.google.com by bare...@gmail.com on 6 Oct 2009 at 6:01

GoogleCodeExporter commented 8 years ago
Yes, we were aware that some users may need this kind of convenient array 
support. We
are considering adding it to the future release. Thanks a lot for your 
suggestion.

Original comment by fangyid...@gmail.com on 7 Oct 2009 at 12:30

GoogleCodeExporter commented 8 years ago
Issue 14 has been merged into this issue.

Original comment by fangyid...@gmail.com on 7 Dec 2009 at 5:22

GoogleCodeExporter commented 8 years ago
json-simple was unable to serialize the result of request.getParameterMap() 
because
it returns Map<String,String[]>. In fact, the result is not even valid JSON. If 
it's
just calling toString(), I would expect it to at least be quoted.

*** Code:
protected void doPost(HttpServletRequest request, HttpServletResponse response)
throws ServletException, IOException {
  String json = JSONValue.toJSONString(request.getParameterMap());
  doSomethingWith(json);
}

*** Test URL:
/myservlet?test=foo

*** Result:
{"test":[Ljava.lang.String;@2afa2afa}

*** Expected Result:
{"test":["foo"]}

Original comment by jeoffwilks@gmail.com on 10 Feb 2010 at 8:11

GoogleCodeExporter commented 8 years ago
Issue 28 has been merged into this issue.

Original comment by fangyid...@gmail.com on 14 Jul 2010 at 1:14

GoogleCodeExporter commented 8 years ago
Looks like the outcome from 
http://code.google.com/p/json-simple/issues/detail?id=48 is pretty similar to 
this, maybe should be merged?

Original comment by michael....@gmail.com on 3 Apr 2012 at 8:07

GoogleCodeExporter commented 8 years ago
This is done in r211.

Original comment by jon.cham...@gmail.com on 10 Aug 2013 at 2:58