JS syntax error while building object

Question

I need to parse multiple email bodies that look like:

Name: Bob smith
Email: [email protected]
Phone Number: 4243331212

As part of a larger program I have the following function to parse the page based on Efficiently parsing email body in javascript:

function parse (i, body) {
 var obj = {}; 
 body.split('\n').forEach(v=>v.replace(/\s*(.*)\s*:\s*(.*)\s*/, (s,key,val)=>{obj[key]=isNaN(val)||val.length<1?val||undefined:Number(val);}));

 var objArr = Object.values(obj);

 var res = [];
 res[0] = i
 res.push(objArr)

 return res

}

when I run this I get a syntax error in:

 body.split('\n').forEach(v=>v.replace(/\s*(.*)\s*:\s*(.*)\s*/, (s,key,val)=>{obj[key]=isNaN(val)||val.length<1?val||undefined:Number(val);}));

what am I doing wrong ?

What's the exact error? Also, you haven't done yourself any favors by sticking everything on one line. Line numbers would have be able to help you narrow it down. I'm going to guess it's a misplaced brace problem. — Carcigenicate
– Carcigenicate, Commented Aug 26, 2017 at 19:37
In the apps script editor it just says "Syntax error. (line 43, file "Code") (that line). I ran the code in jslint and the output said "Wrap the parameter in parens." — user1592380
– user1592380, Commented Aug 26, 2017 at 19:45
Trying splitting it over a few lines so you can get a better location. Also, running this is a debugger would probably work as well. — Carcigenicate
– Carcigenicate, Commented Aug 26, 2017 at 19:46
Cannot reproduce. parse(0, "Name: Bob smith\nEmail: [email protected]\nPhone Number: 4243331212") gives back an array of "0,Bob smith,[email protected],4243331212". Not sure what that prefixed "0" is, but I didn't get any errors. Tested in the Edge console. — Carcigenicate
– Carcigenicate, Commented Aug 26, 2017 at 20:00
yes that's what it should so. The code from the previous question did work. I'm trying to run it as apps script and that's where the problem occurs. I've added a screenshot. The error is occuring on line 44 — user1592380
– user1592380, Commented Aug 26, 2017 at 20:22

Anton Dementiev · Accepted Answer · 2017-08-27 00:43:01Z

1

Google Apps Script is based on the ECMA Script version that doesn't support arrow functions. Replace

array.forEach(element => element.replace(expression))

with

array.forEach(function(element) {

return element.replace(expression);

});

answered Aug 27, 2017 at 0:43

Anton Dementiev

5,7344 gold badges23 silver badges36 bronze badges

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

2 Comments

user1592380 Over a year ago

I;ve changed it to : body.split('\n').forEach(function(v) { return v.replace(/\s*(.*)\s*:\s*(.*)\s*/, (s,key,val)=>{obj[key]=isNaN(val)||val.length<1?val||undefined:Number(val);})}) - However I'm still getting a syntax error. does ' (s,key,val)=>...' also need to be replaced?

Anton Dementiev Over a year ago

Everything with => in it must be replaced with proper functions (with parameters and return statement). For example function (a, b, c) { return a + b + c; }

Collectives™ on Stack Overflow

JS syntax error while building object

1 Answer 1

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related