Вызов определяемой пользователем функции javascript для элемента jQuery

Question

Вызов определяемой пользователем функции javascript для элемента jQuery

0

Я пытался вызвать определенную пользователем функцию JavaScript в элементе jQuery, и хотя я видел здесь несколько тем, у меня все еще есть немного проблем с применением их логик к моей ситуации. Вот что я пробовал до сих пор:

Javscript & jQuery:

    $(document).ready(function() {

// JavaScript function     
     nl2br = function (str, is_xhtml) {
            var breakTag = (is_xhtml || typeof is_xhtml === 'undefined') ? '<br ' + '/>' : '<br>';
            return (str + '').replace(/([^>\r\n]?)(\r\n|\n\r|\r|\n)/g, '$1' + breakTag + '$2');
        }



    // jQuery Call
        if ( ! $('.mediaDesc:contains("p")').length || ! $('.mediaDesc:contains("div")').length || ! $('.mediaDesc:contains("span")').length ) {
                nl2br(this);
            }

    }); //end document ready

HTML:

<div class="mediaDesc">
Hello World!
I am text on a new line soon to have a br tag before me!    
</div>

Моя цель - динамически найти div с классом mediaDesc, чтобы узнать, содержит ли их внутренний контент html-теги, такие как p, span или div. Если нет, то примените к нему функцию JavaScript nl2br. Любая помощь будет оценена по достоинству. Заранее спасибо.

Tower 22 фев. 2014, в 23:09

Источник

1

Что происходит, когда вы отлаживаете это? Есть ли ошибки на консоли? Код jQuery работает вообще? Условие if выполнено? Селекторы находят элементы, которые вы ожидаете найти? Ваша функция вызывается? Пройдите через это в отладчике, вы сможете найти полезную информацию, которая поможет нам помочь вам.
David 22 фев. 2014, в 21:26
2

вы передаете ссылку на объект this для nl2br но аргумент выглядит , как это требуется строка (содержание вы пытаетесь повлиять). IOW кажется, что, возможно, вы хотите что-то более похожее на это nl2br($(this).html());
Crayon Violent 22 фев. 2014, в 21:26
0

Это полное расширение вашего javascript? Есть несколько вещей, которые, вероятно, вызывают проблему: 1) Должен присутствовать в функции готовности документа, поэтому он запускается после того, как весь html будет представлен: jQuery(function($) { ... put your script in here ... }); - Кроме того, я не уверен, но я считаю, что вам нужно изменить с nl2br(this) на nl2br($(this)) - есть разница ... - плюс, как указывает @CrayonViolent, nlbr () настроена на синтаксический анализ строки, но вы передаете ей объект элемента ...
cale_b 22 фев. 2014, в 21:26
0

@cale_b Это действительно функция для работы с документами, и я также использовал селектор $ (this) в nl2br, и код все еще не выполняется должным образом.
Tower 22 фев. 2014, в 21:28
0

Тогда, скорее всего, проблема с передачей объекта, когда функция ожидает строку. Дело в том, что вам нужно изменить функцию для принятия объекта или изменить способ его $(this).val(nlbr($(this).val())); : $(this).val(nlbr($(this).val()));
cale_b 22 фев. 2014, в 21:32
0

но в дополнение к этому, nl2br надеется просто вернуть измененный контент. Но в вашем первом фрагменте, где вы его называете, вы не назначаете измененный контент обратно чему-либо. так что, возможно , что вам на самом деле действительно нужно что - то больше , как это: $(this).html(nl2br($(this).html()));
Crayon Violent 22 фев. 2014, в 21:33
0

Вам не нужно передавать 2 параметра в вашей функции?
Alessandro Gomes 22 фев. 2014, в 21:33
0

Пожалуйста, добавьте больше окружающего кода; как написано, не похоже, что он может работать вообще.
Collin Grady 22 фев. 2014, в 21:34
0

В частности, $ (". MediaDesc") не будет магически запускать nl2br для каждого найденного элемента.
Collin Grady 22 фев. 2014, в 21:35
0

@CrayonViolent Я обновил свой код, чтобы отразить то, как я его полностью использую.
Tower 22 фев. 2014, в 21:49

Показать ещё 8 комментариев

Теги:

javascript

jquery

2 ответа

0

.contains предназначен для поиска текста внутри узла. Вы хотите:

$('.mediaDesc').not(':has(p, div, span)').each(nl2br)

если вам нужно обработать узел и только вызвать nl2br с text или html то выполните:

$('.mediaDesc').not(':has(p, div, span)').each(function(){
    nl2br( $(this).html() )
})

martypdx 22 фев. 2014, в 19:12

Ещё вопросы

Что происходит, когда вы отлаживаете это? Есть ли ошибки на консоли? Код jQuery работает вообще? Условие if выполнено? Селекторы находят элементы, которые вы ожидаете найти? Ваша функция вызывается? Пройдите через это в отладчике, вы сможете найти полезную информацию, которая поможет нам помочь вам.
вы передаете ссылку на объект this для nl2br но аргумент выглядит , как это требуется строка (содержание вы пытаетесь повлиять). IOW кажется, что, возможно, вы хотите что-то более похожее на это nl2br($(this).html());
Это полное расширение вашего javascript? Есть несколько вещей, которые, вероятно, вызывают проблему: 1) Должен присутствовать в функции готовности документа, поэтому он запускается после того, как весь html будет представлен: jQuery(function($) { ... put your script in here ... }); - Кроме того, я не уверен, но я считаю, что вам нужно изменить с nl2br(this) на nl2br($(this)) - есть разница ... - плюс, как указывает @CrayonViolent, nlbr () настроена на синтаксический анализ строки, но вы передаете ей объект элемента ...
@cale_b Это действительно функция для работы с документами, и я также использовал селектор $ (this) в nl2br, и код все еще не выполняется должным образом.
Тогда, скорее всего, проблема с передачей объекта, когда функция ожидает строку. Дело в том, что вам нужно изменить функцию для принятия объекта или изменить способ его $(this).val(nlbr($(this).val())); : $(this).val(nlbr($(this).val()));
но в дополнение к этому, nl2br надеется просто вернуть измененный контент. Но в вашем первом фрагменте, где вы его называете, вы не назначаете измененный контент обратно чему-либо. так что, возможно , что вам на самом деле действительно нужно что - то больше , как это: $(this).html(nl2br($(this).html()));
Вам не нужно передавать 2 параметра в вашей функции?
Пожалуйста, добавьте больше окружающего кода; как написано, не похоже, что он может работать вообще.
В частности, $ (". MediaDesc") не будет магически запускать nl2br для каждого найденного элемента.
@CrayonViolent Я обновил свой код, чтобы отразить то, как я его полностью использую.

Collin Grady · Accepted Answer · 2014-02-22T19-07-00.000Z

1

Лучший ответ

Основываясь на текущей доступной информации, я считаю, что вам нужно что-то вроде этого:

$(".mediaDesc").each(function () {
    if ($(this).find("p").length == 0 || $(this).find("div").length == 0 || $(this).find("span").length == 0)
    {
        $(this).html(nl2br($(this).html()));
    }
});

edit: исправлено неправильное использование .contains

http://jsfiddle.net/s8eNh/

Collin Grady 22 фев. 2014, в 19:07

0

Я попробовал ваше предложение как напечатанное, но оно не сработало. Я обновил свой код, чтобы отразить полную структуру его использования, чтобы обеспечить лучшее пояснение.
Tower 22 фев. 2014, в 21:46
0

Вы можете console.log несколько строк console.log чтобы увидеть, в чем дело; console.log($(this)); например, чтобы посмотреть, правильно ли он ударил по всем элементам .mediaDesc. Я бы, вероятно, зарегистрировал выходные .contains() вызовов .contains() и посмотрел, возвращают ли они ожидаемые значения.
Collin Grady 22 фев. 2014, в 21:50
0

Ack, я неправильно использую содержит; подожди, я исправлю свой пример
Collin Grady 22 фев. 2014, в 21:54
0

Это сработало! Спасибо! Хотя, глядя на это, ваш код отличался от моего не с точки зрения вызова $ (this), а с точки зрения оператора if? По сути, это тот же эквивалент, что и $ ("element: Содержит ('P')");
Tower 22 фев. 2014, в 21:59
0

Это совсем не одно и то же - $(".mediaDesc") захватывает все элементы этого типа, а не только один. Ты тогда ничего с ними не делаешь. Вы проверяете это, но затем вызываете nl2br() для документа (к чему this относится в вашем коде), которому нечего конвертировать.
Collin Grady 22 фев. 2014, в 22:03
0

Мой код захватывает каждый элемент .mediaDesc , но затем запускает цикл над ним (.each) и проверяет каждый отдельный элемент на необходимость запуска nl2br
Collin Grady 22 фев. 2014, в 22:03
0

О, я вижу, это действительно имеет смысл. Спасибо за ваши усилия в этом, я ценю это!
Tower 22 фев. 2014, в 22:05

Показать ещё 5 комментариев