Node объединяет значения расслоение плотной основной

Question

реализации внешних Я пытаюсь сделать что-то вроде ниже, но объединить функции имеет проблемы с линией

content = content.replace(content, "Hi" + values.first_name + "! Thanks for completing this code challenge :)");

Имя файла: app.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 :)"; 

Filename: utilities.js

function merge(content, values) { 

    content = content.replace(content, "Hi" + values.first_name + "! Thanks for completing this code challenge :)"); 
    return content; 
} 


module.exports.merge = merge; 

Answer 1

Ваша функция слияния не является хорошей, она делает странную вещь и, несомненно, вернет странную строку, если вы передадите другой шаблон. Вы заменяете всю строку ввода строкой «Hi ....», но с первым именем, вставленным так в конце, ваша функция слишком специфична и не может обрабатывать дополнительные параметры или другой шаблон/строку.

function merge(content, values) { 
 
    // loop over all the keys in the values object 
 
    Object.keys(values).forEach(function(key) { 
 
    // look for the key surrounded by % in the string 
 
    // and replace it by the value from values 
 
    content = content.replace('%' + key + '%', values[key]); 
 
    }); 
 
    return content; 
 
} 
 

 
var mailValues = {}; 
 
mailValues.first_name = "Janet"; 
 
mailValues.last_name = "Doe"; 
 
var emailTemplate = "Hi %first_name% %last_name%! Thanks for completing this code challenge :)"; 
 
var mergedContent = merge(emailTemplate, mailValues); 
 

 
document.write(mergedContent);

Попробуйте кнопку "Выполнить код сниппета".

for ... in версия, для информации, лучше использовать предыдущую версию.

function merge(content, values) { 
 
    // loop over all the keys in the values object 
 
    for (var key in values) { 
 
    // look for the key surrounded by % in the string 
 
    // and replace it by the value from values 
 
    content = content.replace('%' + key + '%', values[key]); 
 
    } 
 
    return content; 
 
} 
 

 
var mailValues = {}; 
 
mailValues.first_name = "Janet"; 
 
mailValues.last_name = "Doe"; 
 
var emailTemplate = "Hi %first_name% %last_name%! Thanks for completing this code challenge :)"; 
 
var mergedContent = merge(emailTemplate, mailValues); 
 

 
document.write(mergedContent);