Блог

#javascript #typescript #design-patterns #anti-patterns #architectural-patterns

Вопрос:

    [
       {
           "is there a problem": "yes"
       },
       {
           "what is it?": "product is damaged"
       },
       {
           "do you want a refund?": "yes"
       }
   ]
 

 productDamaged
productLost
 

 "isThereProblem?YesWhatisIt?ProductDamagedDoyouwantaRefund?Yes" : 
"productDamaged"
 

Комментарии:

1. Лучший по мнению кого/чего?

2. » что делать, если вопросы и ответы изменятся или будут добавлены дополнительные вопросы » — ну, что вы хотите сделать в этом случае? Должна ли «уникальная строка» измениться или нет? Как вы определяете, какие вопросы имеют отношение к выбору шаблона электронной почты, а какие нет?

3. Хэш — таблица здесь не кажется хорошим решением-она работает только в том случае, если вы можете перечислить все возможные входные данные. Если вы когда-нибудь добавите свободное текстовое поле в свой вопросник, это взорвется. Вместо этого напишите функцию , которая получает значения формы и возвращает строку для выбора шаблона электронной почты. Затем поместите в эту функцию любой алгоритм, который вы хотите, например, проверьте ответ на вопрос «в чем проблема «.

4. @Bergi нет возможности ввода свободного текста в анкету. Вопросы/ответы могут измениться, но затем нам придется вручную обновить сопоставление customStringGenerated -> templateId. В настоящее время у нас есть доступ к массиву вопросов/ответов. Вы предлагаете мне взять этот массив в функцию и выполнить некоторую условную логику для выбранных вопросов, а затем сгенерировать идентификатор шаблона? В этом случае мне даже не нужно другое сопоставление для stringGenerated->templateId. Я просто чувствую, что эта функция потребует тонны условной логики, хотя и это не так просто. Спасибо

5. @motime545 Да, вам всегда придется обновлять таблицу вручную. Если вы используете функцию, она просто вернет соответствующее резервное значение или выберет шаблон каким-либо другим способом. Да, для этого нужна тонна условной бизнес — логики. Начните с малого. Сохраните интерфейс функции. Если функция становится слишком сложной, выполните рефакторинг. Если у вас есть if / else цепочка или switch оператор, который проверяет конкретный вопрос, используйте для этого таблицу поиска, но обрабатывайте случаи, когда вопрос не был задан или ответа нет в таблице. Функция обеспечивает всю необходимую гибкость.

Ответ №1:

 const userSubmission = [{
    "is there a problem": "yes"
  },
  {
    "what is it?": "product is damaged"
  },
  {
    "do you want a refund?": "yes"
  }
]

const userSubmission2 = [{
    "is there a problem": "yes"
  },
  {
    "what is it?": "product is damaged"
  },
  {
    "do you want a refund?": "no"
  }
]

const templates = {
  "is there a problem-yeswhat is it?-product is damageddo you want a refund?-yes": "templateForDamagedAndRefundWanted",
  "is there a problem-yeswhat is it?-product is damageddo you want a refund?-no": "templateForDamagedAndRefundNotWanted"
}

function keyFromSubmission(submission) {
  return submission.reduce((acc, obj) => {
    let [key, value] = Object.entries(obj)[0]
    acc  = `${key}-${value}`
    return acc
  }, "")
}


const key = keyFromSubmission(userSubmission)
console.log(templates[key])

console.log("nOr, with a different user submission...")
const key2 = keyFromSubmission(userSubmission2)
console.log(templates[key2]) 

 const questions = [{
    questionId: "q1",
    text: "what is the problem?",
    answers: [{
      answerId: "a1",
      text: "product was broken"
    }, {
      answerId: "a2",
      text: "product didn't arrive"
    }]
  },
  {
    questionId: "q2",
    text: "do you want a refund?",
    answers: [{
      answerId: "a1",
      text: "yes"
    }, {
      answerId: "a2",
      text: "no"
    }]
  }
]

const userSubmission = [{
    "what is the problem?": "product didn't arrive"
  },
  {
    "do you want a refund?": "yes"
  }
]

function userSubmissionAsSymbols(submission) {
  return submission.map(s => {
    let [key, value] = Object.entries(s)[0]
    let question = questions.find(q => q.text === key)
    let answer = question.answers.find(a => a.text === value)
    return `${question.questionId}-${answer.answerId}`
  })
}

console.log(userSubmissionAsSymbols(userSubmission)) 

Комментарии:

1. Спасибо, я думаю, что у меня было что-то подобное в голове с моим решением, и я склоняюсь к этому. Проблема, которая меня беспокоила, заключалась в необходимости вручную обновлять отображение, если вопросы/ответы все равно были изменены. Из того, что я понимаю, нам это не нужно, но вот почему я подумал о том, чтобы, возможно, использовать функцию для ручного выбора определенных вопросов из ответов, а затем генерировать строку.

2. @motime545, независимость от текста может быть достигнута косвенным путем. См.правку.

3. Спасибо, да, я подумывал об использовании функции поиска, но подумал, что у нее могут возникнуть некоторые проблемы с производительностью. Как вы думаете, безопасно ли сейчас использовать самое простое решение, т. Е. Просто создать строку, объединив вопросы/ответы, а затем переработать ее позже, чтобы использовать больше логики/найти, есть ли у вопросов/ответов более высокая вероятность изменения? Спасибо

4. Да, я думаю, что простой подход прекрасно подойдет даже для сотен вопросов и ответов.

Ответ №2:

 ["is there a problem", "what is it?", "do you want a refund?"]
 

 ["yes", "product is damaged", "yes"]
 

 if(data[0] === "yes" amp;amp; data[1] === "product is damaged") {
   //return email template 1
}
else if(data[0] === "yes" amp;amp; data[1] === "product is lost") {
   //return email template 2
}
 

Комментарии:

1. Вставил уточнение в исходный вопрос. Спасибо

2. Поэтому я хочу кое-что прояснить. Допустим, это единственные три вопроса, являются ли они списком ответов ИЛИ закрытыми вопросами, на которые вы знаете все возможные ответы? Второй вопрос заключается в том, повлияют ли ответы на шаблон, т. Е. существует ли фактор количества вопросов, количество шаблонов? Я думаю, что вы могли бы использовать оператор switch или оператор if, если не слишком много шаблонов. Пример, который, я думаю, вы могли бы найти похожим на вашу проблему, — это условный рендеринг react. Существуют различные «представления», которые вы можете отображать в зависимости от конкретных условий, которые очень похожи на ваши шаблоны.

3. Да, мы знаем все возможные ответы. Ответы на эти вопросы определяют, какой шаблон мы будем использовать. Сейчас у нас не так много шаблонов, но они могут вырасти. На самом деле я мог бы добавить еще одну строку в сопоставление, т. е. questionResonsesString: templateId : Причина контакта Эта причина контакта может быть чем-то, что шаблон может принять в виде текста и отобразить. По сути, нам вручную нужно сопоставить строку questionResponses с идентификатором шаблона, а также добавить reasonForContact. Я чувствовал, что на данный момент это хорошее и простое решение.

4. Хорошо, я обновил свой ответ теперь, когда я полностью понимаю вашу проблему, и это должно решить проблему, с которой вы столкнулись. Дай мне знать. 🙂

5. Спасибо за ответ. Причина, по которой я не хочу хранить эту логику в бэкэнде, заключается в том, что она специфична для конкретного клиента. Некоторые другие клиенты могут использовать эти шаблоны по другим причинам, поэтому мы хотим сохранить логику того, какой шаблон использовать на стороне клиента.