Всем привет, ребят, это нормально? https://github.com/vuejs/test-utils/blob/main/src/constants/dom-events.ts#L110-L975 Я не понимаю, почему они не

Question

Vue.js — русскоговорящее сообщество

Даниил И.

Всем привет, ребят, это нормально? https://github.com/vuejs/test-utils/blob/main/src/constants/dom-events.ts#L110-L975 Я не понимаю, почему они не

создали класс или хук для того, чтобы сократить код и не нарушать принцип DRY?

#frontend #javascript #programming #russian #vue.js

0

08.03.2022

4 ответов

17 просмотров

Даниил И. Автор вопроса

Grigorii K. Shartsev
Здесь не нарушен DRY, в чём повторение? Зачем тут ...

Я думал, что здесь подойдет нечто подобное, ибо здесь возвращается один и тот же объект с одинаковыми свойствами, но разными значениями этих свойств. Таким образом можно сократить код, да и не повторять написание вот этих одних и тех же свойств множество раз.

0

08.03.2022

Grigorii K. Shartsev

Даниил И.

Я думал, что здесь подойдет нечто подобное, ибо зд...

Видимо, нет цели делать код короче, просто чтобы он был короче. Да, в таком решении не повторяются свойства, но зато есть повторение названия функции. При этом значения описания событий нет изначально, чтобы оно появилось, требуется выполнять код. А тому, кто его читает, надо помнить, что вот второй true - это bubbles, а третий true - это cancelable + помнить значение по умолчанию. В таком месте, которое почти никогда не будут редактировать, нет цели сделать как можно меньше строчек кода, и чаще хотят простой и явный код.

0

08.03.2022

Artyom Tuchkov

потому что там константный объект (as const), который имеет литеральные значения, а твой вариант с функцией не будет константным в общем, ради типизации

0

08.03.2022

Grigorii K. Shartsev · Accepted Answer

Grigorii K. Shartsev

Здесь не нарушен DRY, в чём повторение? Зачем тут классы, когда нужна просто константа - не понятна. А хуки - ещё более не понятно.

0

08.03.2022

169 похожих чатов

Всем привет, ребят, это нормально? https://github.com/vuejs/test-utils/blob/main/src/constants/dom-events.ts#L110-L975 Я не понимаю, почему они не

4 ответов

Похожие вопросы