Алгоритм поиска подстроки

[zilker]

function subStr(_substring, _string) {
	var substring = _substring.split(''),
		string = _string.split(''),
		found = false, k = 1;
	for(var i = 0; i < string.length; i++) {
		if(string[i] == substring[0]) {
			for(var j = 1, l = i + 1; j < substring.length; j++, l++) if (string[l]) {
				if(string[l] == substring[j]) {
					k++;
				} else {
					k = 1;
					break;
				}
			}
			if(k == substring.length) {
				found = true;
				break;
			} else {
				i = l;
			}				
		}
	}
	return found;
}

Можно ли сделать эффективнее? Интересует чисто с академической, так сказать, стороны

[devote]

а чем вам indexOf не угодил?

var string = 'qwe';
var substring = 'we';

alert(string.indexOf(substring) !== -1 ? 'found' : 'not found');

[zilker]

devote,
ну про стандартные конструкции языка я в курсе, интересует именно со стороны эффективности алгоритма. Ну представим на минуту, что в javascript`e отсутствуют стандартные способы поиска подстроки и необходимо его реализовать.
Я вот тут ещё что подумал - к символу строки ведь можно обратиться по индексу, правильно ли я понимаю, что движок в этом случае при каждом подобном обращении преобразует строку к массиву? Целесообразно ли я в начале функции сам преобразовываю строки в массивы

[devote]

и необходимо его реализовать.

легко:

function strpos(substring, string) {
    return string.split(substring).length === 2;
}
alert(strpos('привет', 'привет мир!'));

вариант 2:

function strpos(substring, string) {
    var index = 0, subindex = 0;
    while(subindex < substring.length && index < string.length) {
        if (substring[subindex] === string[index]) {
           subindex++;
        } else {
           index -= subindex;
           subindex = 0;
        }
        index++;
    }
    return subindex > 0;
}
alert(strpos('привет', 'привет мир!'));

[danik.js]

Чет ты перемудрил.

function stringContains(string, substring) {
    if (substring.length === 0)
        return false;
    for (var i = 0; i < string.length; i++) {
        if (string[i] === substring[0]) {
            for (var j = 1; j < substring.length; j++) {
                if (string[i + j] !== substring[j])
                    return false;
            }
            return true;
        }
    }
    return false;
}

Может я тоже)

[devote]

Чет ты перемудрил.

да вы оба мудрите странно))) без обид

P.S. конечно название я не совсем правильное функций сделал, но это сами придумывайте.

[devote]

danik.js,
хотя не, у тебя тоже вполне компактно

[devote]

Дзен-трансгуманист,
ну тогда релиз:

function strpos(substring, string) {
    var index = 0, subindex = 0;
    var subStrLen = substring.length;
    var strLen = string.length;
    while(subindex < subStrLen && index < strLen) {
        if (substring.charCodeAt(subindex) === string.charCodeAt(index)) {
            subindex++;
        } else {
            index -= subindex;
            subindex = 0;
        }
        index++;
    }
    return subindex > 0;
}
alert(strpos('привет', 'привет мир!'));

частое обращение к свойству .length тоже влияет на производительность.

[danik.js]

Хром 28, разница чуть меньше: x8

[рони]

zilker,

function subStr(_substring, _string) {
   var found =  _string.split(_substring)[0].length
   return found  <  _string.length
}
alert(subStr('ty', 'qwerty'))