при прохождении по объекту. Проблемы 2:
1. Получается очень много if:
if (!element.questions[indexQuestion].questions[indexSubQuestion].multiselect) {
element.questions[indexQuestion].questions[indexSubQuestion].radioSelect = answerInfo.id
element.questions[indexQuestion].questions[indexSubQuestion].disabled = true
} else {
let indexAnswer = element.questions[indexQuestion].questions[indexSubQuestion].answers.findIndex(item => item.id == answerInfo.id)
if (answerInfo.value == 1) {
element.questions[indexQuestion].questions[indexSubQuestion].answers[indexAnswer].checked = true
} else {
element.questions[indexQuestion].questions[indexSubQuestion].answers[indexAnswer].checked = false
}
это только часть. Это вообще нормально, когда куча вложенных проверок? Или можно сделать как-то проще и понятней?
2. Раздражают вот эти цепочки element.questions[indexQuestion].questions[indexSubQuestion].answers[indexAnswer]... можно, конечно, все это дело в переменные запихать и к ним обращаться. Но нельзя сделать это более лаконичным методом?
Надеюсь, понятно объяснил :)
я бы сказал лучше уж весь код на песок а там ревью кода могут сделать а по этому куску сложно что либо посоветовать как ни: работает ни трогай
Обсуждают сегодня