Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

JavaScript Build a Simple Dynamic Site with Node.js Creating a Basic Template Engine in Node.js A Simple Merge Utility

Theo Bouwman
Theo Bouwman
2,812 Points
Posted by Theo Bouwman
Theo Bouwman
2,812 Points

I cant solve it. Used previous Workspace as example.

Can someone please explain it to me?

And with easy understandable language please?

Theo Bouwman

index.js 
var utilities = require("./utilities");

var mailValues = {};

mailValues.first_name = "Janet";

var emailTemplate = "Hi %first_name%! Thanks for completing this code challenge :)";

var mergedContent = utilities.merge(emailTemplate, mailValues);

//mergedContent === "Hi Janet! Thanks for completing this code challenge :)";
utilities.js 
function merge(content, values) {

  for(var mailValues in emailTemplate) {
     emailTemplate = emailTemplate.replace("%" + mailValues + "%", emailTemplate[mailValues])
  }

  return content;
}


module.exports.merge = merge;

1 Answer

James Kim
James Kim
8,475 Points
James Kim
James Kim
8,475 Points

Hey Theo, well first i think your for in loop is incorrect because you added mailValues in index.js as a variable for your for in loop. The var in for in loop is a variable for the object property.

for(var prop in mailValues){}// the var in the for in loop is a variable for the object property

... but i had the same problem too.... I think it's something with treehouse or something, but i wrote the same exact code on the workspace with modifications that relevant tot he code challenge and it worked.

function merge(content, values) {
  for(var key in values ){ 
    content =  content.replace("%"+key+"%", values[key]);
  }
  return content;
}


module.exports.merge = merge;

hope that works!.