SyntaxError: unterminated string literal with JSON array

Question

So I have...

 var newfavz = 'Array (


[0] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]
[1] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]
[2] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]

)
';

on my console.log, but somehow there is a syntax error: unterminated string literal. I've looked around and tried methods such as str_replace "/" with "//" or a regex like .replace(/^/+/g, ''); because it seems like JavaScript don't allow strings to be broken into multiple lines or something like that.

This all started with an SQL query like so...

$favurl = [];
$favquery = "SELECT * FROM userfavs WHERE users = '$username'";
$favresult = mysqli_query($conn, $favquery);

while($row = mysqli_fetch_assoc($favresult)) {

array_push($favurl, $row['fav_id']);

Afterwards, I did

var newfavz = <?php print_r ($favurl); ?>

which led to the above.

Is there any way I could use to solve the syntax error? Thanks!

echo result of json_encode instead of using print_r. The printed string is useless in JavaScript environment, Check stackoverflow.com/questions/4885737/… — Ram
– Ram, Commented Aug 5, 2017 at 20:55
ah sorry I was using print_r when I was doing the debugging :) — HotPotatos
– HotPotatos, Commented Aug 6, 2017 at 2:22

Jonas Wilms · Accepted Answer · 2017-08-05 20:57:10Z

1

Cause multiline strings are not allowed in js:

"A
 B"

Is a syntax error. You might remove all newlines and replace them with \n, or you use template literals:

`A
 B`

In your code:

var newfavz =` <?php print_r ($favurl); ?>`;

So much about the error. However the string is still unusable, it needs to be parsed. Have a look at JSON or write your own little parser.

edited Aug 5, 2017 at 20:57

answered Aug 5, 2017 at 20:55

Jonas Wilms

139k20 gold badges164 silver badges164 bronze badges

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

2 Comments

Adrian Over a year ago

To add to this, on the server you should encode the array to a json object using json_encode() method before passing to JS.

HotPotatos Over a year ago

Hey, I forgot to add this to the question, but I did use a JSON_encode method before I inserted the data into the SQL database. So far, the only way I've found to remove this unterminated string literal error is to do another JSON_encode which is pretty silly IMO.

DYZ · Accepted Answer · 2017-08-05 20:57:08Z

0

You must put a backslash at the end of each line to have a multiline string:

var newfavz = 'Array (\
[0] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]\
[1] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]\
[2] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]\
)\
';

answered Aug 5, 2017 at 20:57

DYZ

57.3k10 gold badges73 silver badges101 bronze badges

Comments

Sven · Accepted Answer · 2017-08-05 21:01:57Z

0

Either use ES6 template literals:

var newfavz = `Array (


[0] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]
[1] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]
[2] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]

)
`;

Or with a backslash:

var newfavz = 'Array (\n\
\n\
\n\
[0] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]\n\
[1] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]\n\
[2] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]\n\
\n\
)\
';

Or concatenate:

var newfavz = 'Array (\n' +
  '\n' +
  '\n' +
  '[0] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]\n' +
  '[1] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]\n' +
  '[2] => [" 6 "," 1 "," 2 "," 5 "," 3 "," 4 "]\n' +
  '\n' +
  ')\n';

\n is used to represent a newline.

edited Aug 5, 2017 at 21:01

answered Aug 5, 2017 at 20:56

Sven

5,27532 silver badges53 bronze badges

1 Comment

HotPotatos Over a year ago

Thanks man! I used the `` characters and it worked perfectly!

Collectives™ on Stack Overflow

SyntaxError: unterminated string literal with JSON array

3 Answers 3

2 Comments

Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related