Arrays in php & javascript syntax issues

Question

I have got my self confused on how to construct arrays correctly in PHP prior to my json encode to check them in javascript.

I'm trying to store an array of objects with their grid reference (x,y)

So i do this in php:

    $get = mysql_query("SELECT x,y,sid FROM $table WHERE uid='1'") or die(mysql_error());
    while($row = mysql_fetch_assoc($get)) {
        $data[$row['x'].$row['y']] = $row['sid'];
    }
//print_r($data);
$data = json_encode($data);

In javascript i then try to check if a object exists on a given co-ordinate so i try this:

 for (i=0;i<tilesw;i++){ //horizontal   
  for (j=0;j<tilesh;j++){ // vertical

        if(sdata[i.j]){
            alert(sdata[i.j][sid]); 
        }
          }
        }

sdata is my array after json encode.

My json encode looks like this:

 {"44":"0","21":"0"}

Problem is i get : Uncaught SyntaxError: Unexpected token ] on the alert line.

Also is my approach correct or is there a better way to construct my array?

What is i.j attempting to accomplish? This is invalid javascript syntax. — Quintin Robinson
– Quintin Robinson, Commented Mar 8, 2012 at 22:05
i and j are two variables created in the for loop so then i check if: sdata[10] is true.. is so echo the value of sdata[10]. This is an example if i was 1 and j was 0 — Sir
– Sir, Commented Mar 8, 2012 at 22:07
Ahh okay so you are using the PHP string concatenation syntax. In javascript you would want to do sdata[i.toString() + j.toString()] (or some reasonable facsimile.) — Quintin Robinson
– Quintin Robinson, Commented Mar 8, 2012 at 22:10
Wouldn't if(sdata[i+j]) be sufficient ? Or will that create a sum ? — Sir
– Sir, Commented Mar 8, 2012 at 22:12
correct if they are both numeric types then it would apply the addition operation you could do sdata[i + '' + j] ..any addition operation with a string will cause all subsequent addition operations to be string concatenations regardless of type. — Quintin Robinson
– Quintin Robinson, Commented Mar 8, 2012 at 22:15

jprofitt · Accepted Answer · 2012-03-08 22:31:13Z

1

You have a JavaScript syntax error in your code. There is an extra ] on your alert() line.

alert(sdata[i.j][sid]]);

Should be

alert(sdata[i.j][sid]);

If you're actually trying to concatenate the values i and j you also need to be using + rather than ., so you would use i.toString()+j.toString() as the key rather than i.j.

Example of using this with two-dimensional arrays:

PHP

$arr = array();
while($row = mysql_fetch_assoc($get)) {
    if(!isset($arr[$row['x']])) $arr[$row['x']] = array();
    $arr[$row['x']][$row['y']] = $row['sid'];
}

$data = json_encode($arr);

JavaScript

for(var x in sdata) {
    for(var y in sdata[x]) {
        alert('Object found at coordinates ' + x + ',' + y + ' ' + sdata[x][y]);
    }
}

edited Mar 8, 2012 at 22:31

answered Mar 8, 2012 at 22:05

jprofitt

11k4 gold badges38 silver badges46 bronze badges

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

6 Comments

Sir Over a year ago

Ah yeh i fixed that mistake :) thanks - though it still does not seem to alert me once i and j meet an array =/

Sir Over a year ago

Ok i have edited that yet no alert occurs =/ the json encode suggests there is one at i 4 and j 4 but doesn't seem to create the alert =/

jprofitt Over a year ago

@Dave I neglected to take into account that i and j were integers. I've updated my answer once more.

Sir Over a year ago

Thanks jprofitt ill tick you're answer as it is now working... on a side note - is my method most efficient to the best of you're knowledge?

jprofitt Over a year ago

I would potentially use a two-dimensional array like array($row['x'] => array($row['y'])); as the return object so you can simply iterate over the values in the array to determine which cells have objects rather than brute forcing it.

|

Collectives™ on Stack Overflow

Arrays in php & javascript syntax issues

1 Answer 1

6 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

6 Comments

Your Answer

Sign up or log in

Post as a guest

Related