3

Здравствуйте!

Скажите, пожалуйста, как можно исправить вот такую ошибку: Uncaught TypeError: y is not a function

function test($t, y) {
	y($.trim($($t).val()));
}
function validateTest(z){
	alert(z);
}
$('.requed').on('change', function() {
  test('#'+$(this).attr('id'), 'validate'+$(this).attr('data-var'));
});
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<p>
  <label for="signin-login">Логин</label>
  <input class="full-width requed" id="signin-login" data-var="Test" type="text" placeholder="Логин" name="login">
  <span class="cd-error-message"></span>
</p>

Я так понимаю ошибка возникает при попытке вызова значения как функции. Мне же нужно именно так. Полей у меня множество и исходя из значения ('validate'+значение из data-var) пришедшего из поля, будет вызываться определенная функция.

2 ответа 2

3

В зависимости от контекста исполняемой функции. Если все находится в контексте window, то можно написать так:

function test($t, y) {
    var func = window[y];
    func($.trim($($t).val()));
}
function validateTest(z){
    alert(z);
}
$('.requed').on('change', function() {
  test('#'+$(this).attr('id'), 'validate'+$(this).attr('data-var'));
});
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<p>
  <label for="signin-login">Логин</label>
  <input class="full-width requed" id="signin-login" data-var="Test" type="text" placeholder="Логин" name="login">
  <span class="cd-error-message"></span>
</p>

В ином случае нужно заменить window на имя объекта-предка, содержащего эту функцию

4
  • Спасибо! То, что нужно:)
    – Дима
    20 мая 2017 в 19:29
  • @Дима, прими один из ответов нажатием на галочку слева от него. минус этого ответа в помещении функций в глобальную облась, а возможный плюс в помещении выбора функции по имени внутрь test. UPDATE: о, пока я писал комментарий, ты уже принял :)
    – Qwertiy
    20 мая 2017 в 19:35
  • @Алексей Шиманский А как Вам вариант у Qwertiy ? Может его лучше использовать?)
    – Дима
    20 мая 2017 в 19:43
  • @Дима да, мне нравится его вариант. О недостатке, кстати, варианта с windows - он написал. 20 мая 2017 в 19:44
2

function test($t, y) {
  y($.trim($t.val()));
}

var validators = {
  Test: function validateTest(z) {
    console.log(z);
  }
}

$('.requed').on('change', function() {
  test($(this), validators[$(this).attr('data-var')]);
});
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<p>
  <label for="signin-login">Логин</label>
  <input class="full-width requed" id="signin-login" data-var="Test" type="text" placeholder="Логин" name="login">
  <span class="cd-error-message"></span>
</p>

8
  • Спасибо!:) Тоже отличный вариант!
    – Дима
    20 мая 2017 в 19:32
  • Кстати, в Вашем варианте я так понял функция вызывается не по имени из data-var. А как быть, если 5 функций таких будет (и 5 полей с разным соответственно значением в data-var) ?
    – Дима
    20 мая 2017 в 19:57
  • @Дима, по имени: validators[$(this).attr('data-var')] - берётся поле объекта validators с именем из атрибута data-var. В соседнем ответе делается примерно то же самое, но в другом месте и из глобального скоупа: window[y].
    – Qwertiy
    20 мая 2017 в 19:59
  • 1
    @Дима, var validators = { Test: function (z) { ... }, Name: function (z) { ... } }
    – Qwertiy
    20 мая 2017 в 20:14
  • 1
    @Дима, это у тебя баг в том, что id одинаковые. И вообще, нет абсолютно никакого смысла искать элемент заново, когда он уже есть - улучшил скрипт. jsfiddle.net/f425pwq1/6
    – Qwertiy
    20 мая 2017 в 20:48

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.