Отмеченные Email-ы #7
Conversation
|
Добавляю преподавателя (@ShGKme) для код-ревью. |
|
Сейчас только понял, что неправильно использовал computed. Он должен вычислять на основании реактивных переменных, а у меня видоизменяет. Так подумал из за того, что о нем идет речь в задании. Тут нужно было указать watch либо watchEffect верно? |
| const emailMarked = computed(() => { | ||
| return email.value.map((mail) => { | ||
| if (mail.indexOf(search.value) !== -1 && search.value !== "") { | ||
| return { | ||
| mail: mail, | ||
| marked: true, | ||
| } | ||
| } | ||
| return { | ||
| mail: mail | ||
| } | ||
| }) | ||
| }) |
There was a problem hiding this comment.
Сейчас только понял, что неправильно использовал computed. Он должен вычислять на основании реактивных переменных, а у меня видоизменяет. Так подумал из за того, что о нем идет речь в задании. Тут нужно было указать watch либо watchEffect верно?
computed действительно должен вычислять результат на основе других переменных и ни в коем случае не изменять другие данные.
Но ваш computed корректный. Он вычисляет новый массив объектов на основе email и search переменных, не изменяя в процессе вычисления ни одну из них.
There was a problem hiding this comment.
По мелочи:
mail.indexOf(search.value) !== -1в современном JS можно заменить наmail.includes(search.value)- Если создаётся массив объектов, лучше сохранять его структуру, и во втором случае вместо
{ mail }указывать явно marked:{ mail, marked: false }
No description provided.