04.12.2014, 21:04
|
Профессор
|
|
Регистрация: 09.11.2014
Сообщений: 610
|
|
Сообщение от devote
|
регулярки кушают
|
ага, отличный повод создавать стены лапши на каждую мизерную проверку. В оптимизации тоже меру надо знать
|
|
04.12.2014, 21:14
|
что-то знаю
|
|
Регистрация: 24.05.2009
Сообщений: 5,176
|
|
Сообщение от krutoy
|
создавать стены лапши
|
где ты увидел лапшу?
Сообщение от krutoy
|
В оптимизации тоже меру надо знать
|
вот когда пойдешь работать туда где крупный проект и каждая мс на счету, я тогда на тебя посмотрю. Там где я сейчас работаю, у нас оптимизация одна из важнейших частей кодинга. Потому что приложения объемные, функционалом обширны и вот такие части кода что предлагаешь ты, будут сжирать не мало проц. времени, что для наших проектов ни в коем случае не приветствуется...
Конечно для мелких сайтиков эта оптимизация не значительна, и не особо требуется. Но все же нужно уметь и знать как обходится без регулярных выражений. И при возможности их не использовать.
|
|
04.12.2014, 21:20
|
Профессор
|
|
Регистрация: 09.11.2014
Сообщений: 610
|
|
devote,
К тому же, во первых, по поводу перфоманса еще можно поспорить. Во вторых, код с регулярками тоже может быть оптимизирован
var url = 'file:///d:/w.o.r.d.doc';
i=1000000
console.time('my stupid')
while(i--){
url.match(/\.([a-z]+)$/i)[1]
}
console.timeEnd('my stupid')
i=1000000
re=/\.([a-z]+)$/i
console.time('my better')
while(i--){
url.match(re)[1]
}
console.timeEnd('my better')
i=1000000
console.time('yours')
while(i--){
var ext = (ext = url.split('.')).length > 1 ? ext.pop() : '';
}
console.timeEnd('yours')
//в ноде:
// my stupid: 423ms
// my better: 401ms
// yours: 425ms
В третих, во большинстве случаев можно обойтись тестом, а матчить только в поседнюю очередь. А так по любому будет быстрей.
Сообщение от devote
|
где ты увидел лапшу?
|
она накапливается незаметно.
Сообщение от devote
|
каждая мс на счету
|
Думаю, это говорит о фатальных недостатках архитектуры, в подавляющем большинстве случаев, если речь не идет о специализированном проекте, заточенном на максимальную производительнось.
Последний раз редактировалось krutoy, 04.12.2014 в 21:24.
|
|
04.12.2014, 21:26
|
|
Профессор
|
|
Регистрация: 23.12.2013
Сообщений: 1,856
|
|
devote,
Цитата:
|
Удивительно. Похоже исправили метод. А то ранее он работал иначе. Разбивал лишь по первому найденному вхождению... либо я что то путаю.
|
Путаешь, наверно, с этим http://habrahabr.ru/post/231071/#comment_7810751
|
|
04.12.2014, 21:27
|
Аспирант
|
|
Регистрация: 30.06.2014
Сообщений: 56
|
|
вот никогда бы не додумался делать split'ом
|
|
04.12.2014, 21:30
|
|
Профессор
|
|
Регистрация: 23.12.2013
Сообщений: 1,856
|
|
TimohaP,
постоянно так делаю, чтобы регулярки не писать
|
|
04.12.2014, 21:44
|
Профессор
|
|
Регистрация: 09.11.2014
Сообщений: 610
|
|
TimohaP,
Ага, выручает в примитивных случаях, типа вашего, или, например,
alert("1111/2222/3333/4444".split("/").reverse().join("/"))
Но шаг вправо -- наг влево,
alert("1111222233334444".replace(/(1+)(2+)(3+)(4+)/, "$4$3$2$1"))
и уже облом.
и даже вот это 1111/2222/3333/4444/ вы за*тесь реверсить сплитом
Так что особо не расчитывайте на него, в плане работы с текстом. Хотя в js split -- мощная, и юзабельная конструкция, она много че может, это да.
Последний раз редактировалось krutoy, 04.12.2014 в 21:50.
|
|
04.12.2014, 21:46
|
|
Профессор
|
|
Регистрация: 22.03.2012
Сообщений: 3,744
|
|
devote,
var url = "file:///d:/w.o.r.d.doc";
console.time("time1");
var ext1 = url.split('.').pop();
console.log(ext1);
console.timeEnd("time1");
console.time("time2");
var ext2 = url.slice(url.lastIndexOf(".") + 1);
console.log(ext2);
console.timeEnd("time2");
Цитата:
|
chrome 39
doc
time1: 3.577ms
doc
time2: 0.144ms
|
var url = "file:///d:/w.o.r.d.doc";
console.time("time2");
var ext2 = url.slice(url.lastIndexOf(".") + 1);
console.log(ext2);
console.timeEnd("time2");
console.time("time1");
var ext1 = url.split('.').pop();
console.log(ext1);
console.timeEnd("time1");
Цитата:
|
chrome 39
doc
time1: 3.499ms
doc
time2: 0.124ms
|
в ff34 всегда разница в 0.01-0.02 ms
|
|
04.12.2014, 22:06
|
что-то знаю
|
|
Регистрация: 24.05.2009
Сообщений: 5,176
|
|
Сообщение от krutoy
|
К тому же, во первых, по поводу перфоманса еще можно поспорить.
|
ладно согласен, первый мой вариант не удачны, а вот этот обойдет регулярки:
var url = "file:///d:/w.o.r.d.doc";
alert((ext = url.lastIndexOf('.')) !== -1 ? url.substr(ext + 1) : '');
bes, уже предложил его выше)
var url = "file:///d:/w.o.r.d.doc";
i=1000000
re=/\.([a-z]+)$/i
console.time('my better')
while(i--){
var ext = url.match(re)[1]
}
console.log(ext);
console.timeEnd('my better')
i=1000000
console.time('yours')
while(i--){
var ext = (ext = url.lastIndexOf('.')) !== -1 ? url.substr(ext + 1) : ''
}
console.log(ext);
console.timeEnd('yours')
|
|
04.12.2014, 22:13
|
Профессор
|
|
Регистрация: 09.11.2014
Сообщений: 610
|
|
Сообщение от devote
|
этот обойдет регулярки:
|
Дело не в том, обойдет или нет, а в том, что не стоит экономить на копейках. Первоначально вообще надо писать не задумываясь о скорости, и лишь потом, если она недостаточна, приступать к оптимизации.
Парсинг текста без регулярок может стать хорошим интеллектуальным упражнением, (правда, тут больше подходит не сахар языка, а что нибудь более суровое, например, обработка и перебор списков), но практического значения это не имеет.
|
|
|
|