Concatenate values in JavaScript?

Question

I have table row that I'm building in JavaScript. This row has an onClick function. I have to pass some values in this function but I'm getting an error. I think that my single and double quotes do not match. Here is my code:

insRow.innerHTML = "<tr><td><img src='../images/delete.png' border='0px' alt='Delete' title='Delete' onclick='pgDelete('"+fnObj.DATA+"','"+dType+"','"+tblID+"');' /></td></tr>";

Error message:

SyntaxError: expected expression, got '}'

If I inspect the element in dev tools this is what I get:

<img src="../images/delete.png" alt="Delete" title="Delete" onclick="pgDelete(" 8739','att','dba');'="" border="0px">

If anyone can help with this problem please let me know. Thank you.

Is this HTML injected with Javascript? If so, what does that code look like? — Lennholm
– Lennholm, Commented May 11, 2017 at 18:48
Well, look at it like a parser; you get to onclick=', so you're thinking, "they're using the single quote as a delimiter, so I'll stop at the next one". The next one is right after the opening parenthesis. The rest is garbage. — Heretic Monkey
– Heretic Monkey, Commented May 11, 2017 at 18:48

Lennholm · Accepted Answer · 2017-05-11 19:16:49Z

1

You need to change

onclick='pgDelete('"+fnObj.DATA+"','"+dType+"','"+tblID+"');'

into

onclick='pgDelete(\""+fnObj.DATA+"\",\""+dType+"\",\""+tblID+"\");'

answered May 11, 2017 at 19:16

Lennholm

7,5301 gold badge23 silver badges32 bronze badges

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

Comments

Falci · Accepted Answer · 2017-05-11 18:50:43Z

0

As a personal rule, I use "double quotes" for HTML and 'single quotes' for JS.

It means:

<img src="../images/delete.png" border="0px" alt="Delete" title="Delete" onclick="pgDelete(fnObj.DATA + ',' + dType + ',' + tblID);" />

However, there is a tip for concatenation using ',': use Array.join()

<img src="../images/delete.png" border="0px" alt="Delete" title="Delete" onclick="pgDelete([fnObj.DATA, dType, tblID].join());" />

answered May 11, 2017 at 18:50

Falci

1,8734 gold badges29 silver badges56 bronze badges

Comments

BrandonFlynn-NB · Accepted Answer · 2017-05-11 18:56:31Z

0

I'm not certain what you're trying to concatenate but it seems like you have your parameters with unnecessary quotes and pluses. Does pgDelete() accept 3 parameters?

Try:
onclick='pgDelete(fnObj.DATA,dType,tblID);'

Instead of:
onclick='pgDelete('"+fnObj.DATA+"','"+dType+"','"+tblID+"');'

It would be helpful to see the origin of fnObj.DATA , dType and tblID; pgDelete() too.

answered May 11, 2017 at 18:56

BrandonFlynn-NB

4203 silver badges15 bronze badges

1 Comment

espresso_coffee Over a year ago

All three arguments I will get once my process is successful. Ajax call send s parameters to the server. Then I get response in JSON format. I loop through object and populate values in each table row.

Collectives™ on Stack Overflow

Concatenate values in JavaScript?

3 Answers 3

Comments

Comments

1 Comment

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

1 Comment

Linked

Related