Json function not passing the array

Question

var checkedValues = $('.required:checked').map(function () {
    return this.value;
}).get();

 $.getJSON('@Url.Action("testcata", "home")' +"?Type=" + "@(Session["typ"])" + "&city=" + "@(Session["cit"])" +  "&chkd=" + checkedValues,
 function (data) {
     //code
 })

In controller:

public ActionResult testcata(string Type, int? city, int[] chkd)
{
   //code
}

I am trying to get the values of check-boxes which are checked to pass through json function. why the method in the controller taking null as an argument for more than one selected check values ? I am having a int[] chkd parameter in the function . It shows null.

I am not sure which server side language is this. But, you should only get Type, city and chkd as get parameters, not as passed in function. — Jashwant
– Jashwant, Commented Sep 20, 2014 at 6:08
i m not sure but if you see this fiddle jsfiddle.net/khu5oeLc see in console if checkbox is checked it is returning a array with "on"..then instead of taking array of int on controller side change it array of string instead. — Kartikeya Khosla
– Kartikeya Khosla, Commented Sep 20, 2014 at 6:09
passing so much data in query string is really a bad idea. you can use JsonResult instead of ActionResult which should return Json() response. Try Making Post request to the method. In this sense you need to make some fair adjustments in your code. — Rohit416
– Rohit416, Commented Sep 20, 2014 at 6:13
@Jashwant This is asp.net mvc. On client side its jquery. Please help me — Smoking monkey
– Smoking monkey, Commented Sep 20, 2014 at 6:13
@Rohit416 - If I tick one check-box then its working but more than one its not. I am properly getting the json result in one check-box ticked case. — Smoking monkey
– Smoking monkey, Commented Sep 20, 2014 at 6:15

StackSlave · Accepted Answer · 2014-09-20 07:05:48Z

1

You are trying to put an Array as a String. Try this:

var checkedValues = encodeURIComponent($('.required:checked').map(function(){
  return this.value;
}).get().join());

edited Sep 20, 2014 at 7:05

answered Sep 20, 2014 at 6:55

StackSlave

10.6k2 gold badges20 silver badges35 bronze badges

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

Comments

JJRS · Accepted Answer · 2014-09-20 09:28:17Z

1

Please first turn your method from GET to POST and than follow code below.

Your Existing Code

var checkedValues = $('.required:checked').map(function () {
return this.value;
}).get();

New Code to be added

var Data = new Object();
Data.Type = @(Session["typ"]);
Data.city = @(Session["cit"]);
Data.chkd = checkedValues;


$.ajax({
type: "POST",
url: "/home/testcata",
data: JSON.stringify(Data),
dataType: "json"
}).done(function(data){
// Required Code 
});

C# Code Changes

public class Data
{
  string Type {get; set ;};
  int city {get; set ;};
  int[] chkd {get; set ;};
}


public ActionResult testcata(Data data)
{
   String Type = data.Type;
   int city =  data.city;
   int[] chkd = data.chkd; // Your Array will be successfully received here.
}

Hope this will help you, Thanks :) Happy Coding

answered Sep 20, 2014 at 9:28

JJRS

375 bronze badges

1 Comment

Smoking monkey Over a year ago

Thanks for your kind help but I didn't want to change my getjson method as it is being used at various places in project. I used join method and passed the string as param. now everything's fine. :-)

Rohit416 · Accepted Answer · 2014-09-20 06:33:59Z

0

This is kind of a re-arrangement of your code.

var checkedValues = $('.required:checked').map(function () {
    return this.value;
}).get();
//if you are getting correct values here than,

$.ajax({
    type: "POST",
    url: "/home/testcata",
    data: JSON.stringify({chkd: checkedvalues}),
    dataType: "json"
}).done(function(data){
  // your stuff
});

Decorate your controller method with [HttpPost] if possible. Communicating with JSON data is quite secure though. :)

answered Sep 20, 2014 at 6:33

Rohit416

3,4963 gold badges27 silver badges42 bronze badges

2 Comments

StackSlave Over a year ago

Don't think you need JSON.stringify() on {chkd: checkedvalus}. jQuery sends data just fine without that.

Rohit416 Over a year ago

yep, agree. but i used stringify() for extra surety. :)

Collectives™ on Stack Overflow

Json function not passing the array

3 Answers 3

Comments

1 Comment

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related