0

Se supone que mi función debería tranquilamente reemplazar el contenido de cada elemento detectado por el each, y a su vez, buscar también la aparición de attachment para posteriormente realizar una función con cada attachment buscado, pero no funciona lo que va dentro del while si no pongo return Cualquiercosa, y si pongo return, lo que sigue después del while, se omite quedando como última función el $(this).html(content). No se realmente lo que pasa, o que onda, es como si no leyera algunas lineas...

    $(".x").each(function(){
        content = $(this).text();
        content = content.replace(/\@(.*?)\#([0-9]{5})/g, '<a class="x" x="@$1#$2">$&</a>');
        content = content.replace(/\*{2}(.*?)\*{2}/g, '<text-b>$1</text-b>');
        content = content.replace(/\-{2}(.*?)\-{2}/g, '<text-t>$1</text-t>');
        content = content.replace(/\~{2}(.*?)\~{2}/g, '<text-s>$1</text-s>');
        attachment = /\{{2}attachment:(.*?)\}{2}/g;
        attachmentsCount = (content.match(attachment) || []).length;
        while((image = attachment.exec(content))) {
        $("<img>").attr("src", "url=" + image[1]).appendTo($(this).children("x"));
        }
        var content = content.replace(attachment, '')
        $(this).html(content);
    })

8
  • 1
    Has inentado colocar == en vez de solo uno, en la condición del while? Commented el 6 ago. 2018 a las 0:23
  • @HugoQuiñónez Si querés lo intento, aunque desde mi punto de vista de principiante, creo que eso no influye la verdad en nada, porque no es como si estuviese comparando dos variables distintas, es más como forEach(var AS varible). Commented el 6 ago. 2018 a las 0:26
  • 1
    Me refiero a esta parte while((image = attachment.exec(content))) estoy tratando de entender tu código, pero creo que creas un ciclo infinito... qué contiene "image" ?? Commented el 6 ago. 2018 a las 0:37
  • "image" no contiene nada, lo puse así porque en mdn (si es que así se llamaba) explican que la condición tiene que ser una variable no usada para que luego se convierta en el array con los datos de las ocurrencias de exec, por eso es while((image = attachment.exec(content))), por cada vez que aparezca algo con el regexp del attachment, se creará el ciclo con los datos de la ocurrencia de regexp en la variable image. Es como decir while($users AS $query->fetch()) en php, y sí, lo probé como dijiste y la función deja de servir. Commented el 6 ago. 2018 a las 0:46
  • @HugoQuiñónez while((image = attachment.exec(content))) esto no necesariamente está mal porque uno puede declarar una variable dentro del while y cuando sea undefined, false, null ... el while acaba. Para mi que el ciclo si es infinito pero porque el en ningun momento está cambiando content, lo que hace attachment.exec(content) siempre devuleva algo diferente a null que es lo mismo que la condicion siempre sea true...
    – clay
    Commented el 6 ago. 2018 a las 0:49

1 respuesta 1

1

La condición del while al parece siempre es cierta y se queda en un bucle infinito y lo que parece es que modificas content afuera del ciclo y no adentro.

Este codigo debería funcionar

 $(".x").each(function(){
    content = $(this).text();
    content = content.replace(/\@(.*?)\#([0-9]{5})/g, '<a class="x" x="@$1#$2">$&</a>');
    content = content.replace(/\*{2}(.*?)\*{2}/g, '<text-b>$1</text-b>');
    content = content.replace(/\-{2}(.*?)\-{2}/g, '<text-t>$1</text-t>');
    content = content.replace(/\~{2}(.*?)\~{2}/g, '<text-s>$1</text-s>');
    attachment = /\{{2}attachment:(.*?)\}{2}/g;
    attachmentsCount = (content.match(attachment) || []).length;
    while((image = attachment.exec(content))) {
    $("<img>").attr("src", "url=" + image[1]).appendTo($(this).children("x"));
    content=content.replace(attachment, '');
    }

    $(this).html(content);
})
2
  • Resulta que no era nada de lo discutido arriba, lo siento todo está funcionando bien, mala mía, el elemento del DOM que especifiqué no existía JAJAJAJAJAAJAJAJ Commented el 6 ago. 2018 a las 1:08
  • 1
    ese era el otro posible error $("<img>") porque creo que no va < ni > si ?
    – clay
    Commented el 6 ago. 2018 a las 1:20

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.