indexof в функцию

[kolyas]

У меня часто используется indexOf и для сокращения я выкинул в фунцию часть кода:

function istut(where, who){
	if (where.indexOf(who) != -1) return true;
	return false;
}

вопрос по производительности: без разницы использовать функцией или не сокращая. или на одном проходе это не заметно?

[danik.js]

вопрос по производительности: без разницы использовать функцией или не сокращая. или на одном проходе это не заметно?

На одном проходе точно разницы нет. Дайте нормальное имя функции, уберите дибильный if () - зачем он нужен, если результат сравнения и есть true/false

function contains(where, who){
    return where.indexOf(who) !== -1;
}

ES Harmony вроде как может появиться String.prototype.contains

[kolyas]

спасибо. и код у вас чище.

Про

ES Harmony вроде как может появиться String.prototype.contains

не очень понял

если про

var contained = str.contains(searchString [, position]);

то мало где поддерживается.

оказывается я плохо искал.

if(!('contains' in String.prototype))
  String.prototype.contains = function(str, startIndex) { return -1!==this.indexOf(str, startIndex); };

[nerv_]

return ~where.indexOf(who);

[рони]

nerv_,

return !!~where.indexOf(who);

[nerv_]

рони, если нужно вернуть булево, то да