[].map и String.prototype.trim

Question

[Константин Китманов]

Часто приходится делать что-нибудь такое:

function clearTags (rawTags) {
	return rawTags
		.split(',')
		.map(
			function (tag) {
				return tag.trim()
			}
		);
}

Смущает, что приходится каждый раз писать лямбду, и есть интуитивное ощущение, что можно обойтись чем-то вроде .map(String.prototype.trim.call) (не работает), но у меня не получилось. Поделитесь, если кто знает?

P.S. Решение вида:

function clearTags (rawTags) {
	var trim = function (arg) { return String.prototype.trim.call(arg); };
	return rawTags
		.split(',')
		.map(trim);
}

тривиально, не так красиво и не лишено недостатков.

Answer 1

[egorinsk]

Можно проще.

var x = string.split(/\s*,\s*/g);

Comments

[Константин Китманов]

Гениально, спасибо!

Answer 2

[Александр Кит]

Лучшим же решением, конечно же, является комбинация предложений от egorinsk и Aingis. Добавлю просто как выкрутится с прототипами:

function clearTags(str){
    var trim = String.prototype.trim.call.bind(String.prototype.trim);
    return str
        .split(',')
        .map(trim);
}

Но в данном случае лучше так не делать.

Comments

[Константин Китманов]

Да, нашлось лучшее решение, но зато ваш ответ действительное отвечает на мой вопрос:)

[Александр Кит]

k12th, вспомнил ваш вопрос, и стало интересно, насколько map+trim будут отставать от регекспа — оказывается существенно — (на случай если и вам интересно) — jsperf

[Константин Китманов]

Спасибо!

Answer 3

[Сергей Протько]

function clearTags (rawTags) {
    var trim = function (arg) { return String.prototype.trim.call(arg); };
    return rawTags
        .split(',')
        .map(trim);
}

Эквивалентно

function clearTags (rawTags) {
    var trim = function (tag) { return tag.trim(); };
    return rawTags
        .split(',')
        .map(trim);
}

Тоесть вы просто замыкание присвоили переменной, на одну переменную больше. Никакого профита. Более того добавился вызов функции через прототип и через call (который сам по себе медленнее обычного вызова).

Comments

[Сергей Протько]

Замыкания писать нормально. Ежели вас смущает именно сам процесс, что ради строчки приходится объявлять функцию — используйте CoffeeScript, там это занимает меньше символов и не так сильно бросается в глаза.

Но это если говорить вообще об обходе чего-то в map. А так соглашусь с egorinsk, split + regexp как-то проще.

[Константин Китманов]

Я специально указал решение, которое мне не нравится, потому что знал, что рано или поздно оно появится в ответах.

Замыкания писать нормально и кофе экономит символы — тут не поспоришь.

[Сергей Протько]

я к тому что это не решение а просто переиначивание, да еще и избыточное.

Answer 4

[Олег Торбасов]

В ECMAScript 6 писать ламбды гораздо проще:

str.split(",").map(function(str) str.trim());//Fx3+

или вообще так:

str.split(",").map(str=>str.trim());//Fx22+

Comments

[Константин Китманов]

Это здорово, но до этого еще далеко.

[Олег Торбасов]

Кому далеко, а у кого Firefox — целевой браузер, а другие вообще не требуются; тут мне повезло.