Как улучшить/исправить этот код?

Question

[godsplane]

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

https://codesandbox.io/s/dry-cache-een4q6?file=/sr...

Спасибо.

Answer 1

[maksam07]

Я в JS слаб, но пометку сделаю:

validation(target) {
    return this.validate(target)
  }

Comments

[Gary_Ihar]

Голосую, что это вообще бесполезный метод в классе Form

[maksam07]

Gary_Ihar, имеешь в виду, что можно напрямую обращаться к validate ? Я тоже об этом думал, но чет не решился дописать)

[Gary_Ihar]

maksam07, ну да, смысл тогда наследования. Раз уж поделил форму и валидацию, то топи до конца

[Gary_Ihar]

ну мне лично еще не нравится, что форма работает с CSS-селекторами внутри себя. Корми ей просто данные и все, получай эти данные не внутри формы. Ну это если планируешь переиспользовать код, если это разовая форма на чисто так, то можно в принципе и не делать такой код формы независимым(ну зависимым ток от типа входных данных)

[WbICHA]

Gary_Ihar, я бы лучше спросил почему класс формы в принципе наследует класс валидации.

[Gary_Ihar]

WbICHA, ну это да, ну тут типа вся соль мне показалось попользоваться наследованием