Плоха ли функция eval?

Question

[Muvka]

Делаю для себя универсальный обработчик форм. И мне нужно для каждой формы свои цели прописать. Всякие yaCounter или gtag(я плохо в этом разбираюсь). onsubmit не подходит, т.к. формы проверяются на бэкэнде. Хочу сделать дата атрибут, куда буду пихать код целей, а после отправки формы, исполнять этод код с помощью eval. Подскажите, это хороший вариант? Если есть лучше - подскажите.

Comments

[DanKud]

Не совсем понял в чем проблема и зачем eval()? Отправляйте форму AJAX-ом и выполняйте нужный вам код при возврате результата в коллбэк-функции. Или может я что-то не так понял. Само по себе использование eval() при современном развитии JS это, если можно так выразиться, даже уже "незаконно" :)

[Muvka]

DanKud, Ну скажем у меня есть 10 форм. Вы предлагаете для каждой писать отдельный обработчик? Мне кажется это излишним. Я даю класс 'ajax-form' и на все формы с этим классом вешаю один обработчик. Но проблема в том, что у каждой формы свои цели... Вот сейчас пришло в голову хранить только названия целей в data атрибутах... но а вдруг разные будут? А нет, глянул, у яндекса одно название цели, у гугла другое. А 2 дата атрибута давать несколько странно...

[DanKud]

Muvka, так просто добавьте каждой форме скрытое поле с уникальным идентификатором и по нему отдавайте ответ.

[Muvka]

DanKud, Так поле вернет ту же строку. А исполнять то ее как? eval()?

[DanKud]

Muvka, не совсем опять же понимаю, что вы имеете ввиду. Вот у вас есть 10 форм, все эти формы разные, но на них висит один и тот же обработчик, который делает AJAX-запрос к бэкенду и получает ответ. Добавляете уникальное поле к каждой форме и на бэкенде уже обрабатываете данные на основании этого уникального идентификатора. Если вам пришли данные с формы 1, то отдаете такой ответ, если пришли данные с формы 2, то такой, и так далее. Я это имею ввиду.

[Muvka]

DanKud, Мне не на бэкэнде нужно формы различать, а на фронтэнде, после получения ответа от бэка. Т.е. если форма прошла валидацию и успешно отправилась, мне нужно выполнить уникальный для каждой формы код в js. Типа

yaCounterXXXXXXXXX.reachGoal('....'); return true;gtag('event', '....', { 'event_category': '....' });

[DanKud]

Muvka, ну так опять же, вы на бэкенде проверяете валидацию и можете определить форму, после чего отдаете ответ, в котором будет идентификатор формы и успешна ли она прошла валидацию. И на основании этого ответа в коллбэке AJAXа выполняете нужный вам код. Весь код, который нужно исполнить после отправки формы, должен содержаться в коллбэк-функции AJAX-запроса, вот я о чем говорю.

[Muvka]

DanKud, Ну так код то все равно будет в виде строки) И его придется eval() выполнять, разве нет? Я поэтому и спрашиваю, нормально ли в 2019 году использовать eval() в таком случае или есть другие, более правильные способы?

Answer 1

[Сергей Соколов]

В вашей задаче eval() не нужен, вроде бы.

Вызов событий счётчиков отличается друг от друга только названием события и именем счетчика. Как уже верно заметили в комментарии, данные в backend отправляете ajax'ом, и в обработчике вызова получаете ответ сервера.

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

// <script src="https://unpkg.com/axios/dist/axios.min.js"></script>
function submitData( formData, goalName, yandex, google) {
  axios.post( '/submitform.php', formData)
  .then( function(response) {
    if (response.ok) {
      yandex.reachGoal( goalName );
      google(goalName, '...' );
    } else {
      // данные не прокатили
    }
  })
}