Форум

Newstory · #1 (**permalink**) 14.02.2013, 20:14

Здравствуйте, возникла пробема с построением регулярки. Есть переменная content, в которую я из файла считываю шаблон вида
var content='<div style="clear: both;"> </div>
<script type="text/javascript">// <![CDATA[
jQuery(document).ready(function(){

jQuery("#he5").css('display','none');
jQuery('#hb5').on("click",function () {
jQuery('#he5').toggle('slow');
});
})
// ]]></script>';

Нужно получить в отдельную переменную весь контетнт до тега <script> и в другую - содержиое тега <script> (то есть все то, что находится между открывающим и закрывающим тегами)

Написанная регулярка не работает - выдает null( Хотя ожидался массив из 3 элементов.
var htmlJsRe = new RegExp('([\w,\W]*)<script type="text\\/javascript">([\w,\W]*)<\\/script>', 'mg');

var htmlJsReArray = htmlJsRe.exec(content);

Подскажите пожалуйста как ее исправить

Deff · #2 (**permalink**) 14.02.2013, 20:43

var content='<div style="clear: both;">&nbsp;</div>\
<script type="text/javascript">([\w,\W]*)<\\/script>\', \'mg\');\
 jQuery(document).ready(function(){\
\
 jQuery("#he5").css("display","none");\
 jQuery("#hb5").on("click",function () {\
 jQuery("#he5").toggle("slow");\
 });\
 })\
 // ]]><'+'/'+'script>';
var script = (' '+content).split(eval('/(<script[^>]*?>|<\\/script>)/igm'));
var contentDoScrpt=script[0];
var teg1=script[1];
var teg2=script[3];
var contentteg=script[2];

alert(contentDoScrpt)
alert(contentteg)

рони · #3 (**permalink**) 14.02.2013, 22:34

Сообщение от Newstory

Написанная регулярка не работает

работает, нужно только экранировать \w\W и запятая там ненужна.

var content='<div style="clear: both;">&nbsp;</div>\
<script type="text/javascript">// <![CDATA[\
 jQuery(document).ready(function(){\
\
 jQuery("#he5").css("display","none");\
 jQuery("#hb5").on("click",function () {\
 jQuery("#he5").toggle("slow");\
 });\
 })\
 // ]]><\/script>';
var htmlJsRe = new RegExp('([\\w\\W]*)<script type="text\/javascript">([\\w\\W]*)<\/script>', 'mg');
var htmlJsReArray = htmlJsRe.exec(content);
alert(htmlJsReArray.join('\n\n\n'));

rgl · #4 (**permalink**) 15.02.2013, 10:53

Ошибки в выражении в исходном сообщении:
1. [\w,\W] - соответствует словному символу, запятой, несловному символу, запятая тут лишняя, она и так несловный символ. Заменяем на [\w\W] (хотя в этом случае принято использовать [\s\S])
2. Звездочка после закрывающей квадратной скобки говорит - найти сколько угодно повторений, причем чем больше, тем лучше. Поэтому сначала выражение пройдет по всей строке, а потом посимвольно будет откатываться назад. Чтобы этого не было, для ускорения, добавляем вопрос, т.е. пишем так [\w\W]*?
3. Прямые слэши в текстовых строчках маскировать нет необходимости.
заменяем text\/javascript на text/javascript и <\/script> на </script>
4. Собственно, из-за чего не работало. Обратные слэши маскоровать в текстовых строках надо обязательно, т.е. [\\w\\W]
После всех этих исправлений:
var htmlJsRe = new RegExp('([\\w\\W]*?)<script type="text/javascript">([\\w\\W]*?)</script>', 'mg');
5. Вызывает сомнение нужность флагов 'mg', а вот флаг 'i' возможно и нужен, вдруг тэги будут написаны большими буквами.
6. <script type="text/javascript"> возможно стоит заменить на <script[^>]*> (кстати, тут после звездочки вопрос ни к чему :-) )

рони · #5 (**permalink**) 15.02.2013, 11:08

rgl,
Вы сами проверяли свои размышления?

rgl · #6 (**permalink**) 15.02.2013, 11:22

Что конкретно?

рони · #7 (**permalink**) 15.02.2013, 11:27

rgl,

<!DOCTYPE HTML>
<html>
<head>
  <title></title>
</head>
<body>
<script language="JavaScript" type="text/javascript">
var content='<div style="clear: both;">&nbsp;</div>\
<script type="text/javascript">// <![CDATA[\
 jQuery(document).ready(function(){\
\
 jQuery("#he5").css("display","none");\
 jQuery("#hb5").on("click",function () {\
 jQuery("#he5").toggle("slow");\
 });\
 })\
 // ]]><\/script>';
//var htmlJsRe = new RegExp('([\\w\\W]*)<script type="text\/javascript">([\\w\\W]*)<\\/script>', 'mg');
var htmlJsRe = new RegExp('([\\w\\W]*?)<script type="text/javascript">([\\w\\W]*?)</script>', 'mg');
var htmlJsReArray = htmlJsRe.exec(content);
alert(htmlJsReArray.join('\n\n\n'));
</script>
</body>
</html>

rgl · #8 (**permalink**) 15.02.2013, 11:42

Ошибся, в <\/script> маскировка нужна, но это уже не регулярные выражения и не JavaScript, этого требует html, а я тестирую скрипты без html, вот и ошибся. Если поместить мое выражение в отдельный файл и включить с помощью <script src= то все работает.

Newstory · #9 (**permalink**) 15.02.2013, 19:54

Спасибо всем огромное за примеры и подробный разбор моего регулярного выражения! Проблема решена)

Форум

Справочник

Discord чат

Курсы javascript