조건부 URL에 Javascript를 추가하거나 window.location.href를 기반으로 리디렉션

Question

클릭하면 북 탭에 'char1'및/또는 'char2'가 포함되어 있는지 확인하기 위해 현재 탭/창 URL을 확인합니다. '(주어진 문자). 두 문자가 모두있는 경우 다른 URL로 리디렉션됩니다. 다른 두 문자는 현재 URL을 각각 추가합니다.

나는 (지금까지는 완벽하게 작동 한) 다음보다 진보 된 방법이 있어야한다고 생각하지만 자바에 대한 지식은 충분하지 않습니다. 내 (다루기 힘든 & 반복) 작업 코드 (사과) :

if (window.location.href.indexOf('char1') != -1 && 
    window.location.href.indexOf('char2') != -1) 
{ 
    window.location="https://website.com/"; 
} 
else if (window.location.href.indexOf('char1') != -1) 
{ 
    window.location.assign(window.location.href += 'append1'); 
} 
else if (window.location.href.indexOf('char2') != -1) 
{ 
    window.location.assign(window.location.href += 'append2'); 
} 

내가하지만, 음 ... 아주 우아하지 않는 이상 말을 할 필요 정확히 무엇을합니까.

vars 또는 의사 객체를 사용하면이 작업을 수행하는 간단한 방법이 있습니까? 아니면 더 나은 코드?

Answer 1

A (정렬의) dthorpe의 제안의 리팩토링 :

var hasC1 = window.location.href.indexOf('char1')!=-1 
var hasC2 = window.location.href.indexOf('char2')!=-1 
var newLoc = hasC1 
       ? hasC2 ? "https://website.com/" : window.location.href+'append1' 
       : hasC2 ? window.location.href+'append1' : ''; 

if (newLoc) 
    window.location = newLoc; 

가 assign 호출이 값을 할당과 동일 window.location, 어쨌든 메서드에서 연산자 +=을 더한 값으로 두 작업을 수행하고 있습니다.

window.location.assign(window.location.href+='append2') 

실제로 할당 방법을 호출하기 전에 window.location.href 끝에 "append2"가 할당되어 중복됩니다.

window.location을 var로 설정하여 DOM 조회를 줄일 수도 있습니다.

Answer 2

필자가 볼 수있는 유일한 감소점은 vars로 중복되는 indexof 호출을 제거한 다음 vars를 테스트하는 것입니다. 그것은 비록 성능에서 상당한 차이를 만들지 않을 것입니다.

var hasChar1 = window.location.href.indexOf('char1') != -1; 
var hasChar2 = window.location.href.indexOf('char2') != -1; 
if (hasChar1) 
{ 
    if (hasChar2) 
    { 
     window.location="https://website.com/"; 
    } 
    else 
    { 
     window.location.assign(window.location.href+='append1'); 
    } 
} 
else if (hasChar2) 
{ 
    window.location.assign(window.location.href+='append2'); 
} 

Answer 3

VAR 위치 = window.location.href

if (location.indexOf('char1')!=-1 && location.indexOf('char2')!=-1) 
{window.location="https://website.com/";} 
else if (location.href.indexOf('char1')!=-1) {window.location.assign(location+='append1');} 
else if (location.indexOf('char2')!=-1) {window.location.assign(location+='append2');} 

Answer 4

확장 가능한 코드의 종류. 내가 미쳤나요?

var loc = window.location.href; 
var arr = [{ 
    url: "https://website.com/", 
    chars: ["char1", "char2"] 
}, { 
    url: loc + "append1", 
    chars: ["char1"] 
}, { 
    url: loc + "append2", 
    chars: ["char2"] 
}]; 

function containsChars(str, chars) 
{ 
    var contains = true; 
    for(index in chars) { 
    if(str.indexOf(chars[index]) == -1) { 
     contains = false; 
     break; 
    } 
    } 
    return contains; 
} 

for(index in arr) { 
var item = arr[index]; 
if(containsChars(loc, item.chars)) { 
    window.location.href = item.url; 
    break; 
} 
}