Javascript Bookmarklet If-Else

Question

Я пытаюсь написать букмарклет, который обнаруживает, что пользователь находится на уже проанализированной странице, и если он не является значением, должен передаваться вместе с API.

if (window.location.indexOf("thequeue.org") >= 0) { 
    alert("Drag the Bookmarklet in your Brookmarks Bar!"); 
} else { 
    location.href = 'http://thequeue.org/r?id=' + encodeURIComponent(location.href) + '&title=' + document.title; 
} 

Есть несколько учебников по Букмарклетам в целом, но я нашел этот инструмент преобразования: http://jasonmillerdesign.com/Free_Stuff/Instant_Bookmarklet_Converter, который дал мне это:

javascript:(function(){if(window.location.indexOf(%22queue.org%22)%20%3E%3D%200)%0A%20%20%20%20%20%20%20%20%20%20%20%20%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20alert(%22your%20url%20contains%20the%20name%20franky%22)%3B%0A%20%20%20%20%20%20%20%20%20%20%20%20%7D%20else%20%7B%0Alocation.href%3D'http%3A%2F%2Fthequeue.org%2Fr%3Fid%3D'%2BencodeURIComponent(location.href)%2B'%26title%3D'%2Bdocument.title%3B%0A%7D}()); 

Однако букмарклет перестает работать полностью. Я могу подтвердить, что это работает: javascript:location.href='http://thequeue.org/r?id='+encodeURIComponent(location.href)+'&title='+document.title;

Что я делаю неправильно?

Answer 1

Это должно работать для вас:

javascript:(function(){if(location.href.indexOf("thequeue.org/")>=0){alert("Drag the Bookmarklet in your Brookmarks Bar!");}else{location.href='http://thequeue.org/r?id='+encodeURIComponent(location.href)+'&title='+document.title;}})(); 

Обратите внимание, что я изменил, если предложение немного, чтобы сделать его более устойчивым.

Answer 2

Возможно, вы захотите попробовать сначала удалить все пробелы. Я не знаю, если это ваша проблема, но это, по крайней мере, сделает ваш букмарклет намного менее уродливым.