• Главная
  • » JavaScript
  • » Осваиваю JS написал полезный скрипт обсудите плис

#1 2011.02.13 20:16

TLENS
Moderator
Откуда: Украина
Зарегистрирован: 2009.04.05
Сообщений: 2402
Карма: 14
Профиль

Осваиваю JS написал полезный скрипт обсудите плис

Я только начинаю подскажите что я не так сделал и как можно лучше?
Скрипт ниже спрашивает хочет ли пользователь перейти по ссылке если она ведет на другой ресурс.
полезно для форумов чатов и т.п.
Не срабатывает если ссылка ведет на под-домен или выводит с него.
Гемор рой оцени по 10 бальной шкале

Код:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
window.onload = function() {
searchLink();
}
function searchLink()
{
//var myDomain = document.domain;
var myDomain = 'test.pp.ua';
var domainLength = myDomain.length;
var a = document.getElementsByTagName('a');
var l = a.length;
for(i=0; i<l; i++)
{
var href = a.item(i).href;
//console.log(href.replace(/.+:\/\/([^\/]*?)\/.*/,'$1'));
var host = href.replace(/.+:\/\/([^\/]*?)\/.*/,'$1');
if(host.toLowerCase().substr(-domainLength) != myDomain)
{
a.item(i).onclick = locationLink;
}
}
}
 
 
function locationLink()
{
if(confirm('Данная ссылка ведет на другой ресурс.\nВы уверенны что хотите перейти по адресу?\n'+this.href))
{
//window.open("/location.php?url="+encodeURIComponent(this.href));
window.open(this.href);
}
return false;
}

Отредактировано TLENS (2011.02.13 20:18)

Неактивен

#2 2011.02.13 20:55

TLENS
Moderator
Откуда: Украина
Зарегистрирован: 2009.04.05
Сообщений: 2402
Карма: 14
Профиль

Re: Осваиваю JS написал полезный скрипт обсудите плис

Да насчет регулярки к каждой ссылке это не очень
Думал привязать функцию searchLink к клику и обрабатывать по одному адресу, но просто у меня уже к своим применены функции на клик, и я думаю будут мешать один одному

Отредактировано TLENS (2011.02.13 20:56)

Неактивен

#3 2011.02.13 22:54

Gemorroj
Administrator
Откуда: Белоруссия
Зарегистрирован: 2007.11.03
Сообщений: 6538
Карма: 108
Профиль Веб-сайт

Re: Осваиваю JS написал полезный скрипт обсудите плис

Код:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
window.onload = function() {
var myDomain = 'test.pp.ua', domain;
var domainLength = myDomain.length;
 
var a = document.getElementsByTagName('a');
var l = a.length;
 
for (var i = 0; i < l; i++) {
domain = a.item(i).href.split('/')[2].toLowerCase();
 
if (domain != myDomain) {
a.item(i).onclick = function () {
return confirm("Данная ссылка ведет на другой ресурс.\nВы уверенны что хотите перейти по адресу?\n" + domain);
}
}
}
}

Неактивен

#4 2011.02.13 23:43

TLENS
Moderator
Откуда: Украина
Зарегистрирован: 2009.04.05
Сообщений: 2402
Карма: 14
Профиль

Re: Осваиваю JS написал полезный скрипт обсудите плис

Стопудово за split забыл
Добавлено спустя   5 минут  12 секунд:

Gemorroj

P.S твой вариант будет спрашивать если ссылка будет на поддомен.

Отредактировано TLENS (2011.02.14 00:00)

Неактивен

#5 2011.02.13 23:55

TLENS
Moderator
Откуда: Украина
Зарегистрирован: 2009.04.05
Сообщений: 2402
Карма: 14
Профиль

Re: Осваиваю JS написал полезный скрипт обсудите плис

Но это можно исправить!

Код:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
window.onload = function() {
searchLink();
}
function searchLink()
{
var myDomain = 'test.pp.ua';
var domainLength = myDomain.length;
var a = document.getElementsByTagName('a');
var l = a.length;
for(i=0; i<l; i++)
{
if(a.item(i).href.split('/')[2].toLowerCase().substr(-domainLength) != myDomain)
{
a.item(i).onclick = function()
{
if(confirm('Данная ссылка ведет на другой ресурс.\nВы уверенны что хотите перейти по адресу?\n'+this.href))
{
window.open(this.href);
}
return false;
};
}
}
}

Отредактировано TLENS (2011.02.14 00:06)

Неактивен

#6 2011.02.14 00:08

TLENS
Moderator
Откуда: Украина
Зарегистрирован: 2009.04.05
Сообщений: 2402
Карма: 14
Профиль

Re: Осваиваю JS написал полезный скрипт обсудите плис

Кстати после a.item(i).onclick = function () {} надо поставить точку с запятой?
Ведь обфускатор ошибок наделает

Неактивен

#7 2011.02.14 01:57

TLENS
Moderator
Откуда: Украина
Зарегистрирован: 2009.04.05
Сообщений: 2402
Карма: 14
Профиль

Re: Осваиваю JS написал полезный скрипт обсудите плис

Я имел ввиду в целом оценить даже насчет обфускатора

Неактивен

#8 2011.02.14 10:48

Gemorroj
Administrator
Откуда: Белоруссия
Зарегистрирован: 2007.11.03
Сообщений: 6538
Карма: 108
Профиль Веб-сайт

Re: Осваиваю JS написал полезный скрипт обсудите плис

TLENS, да надо. убери searchLink и сделай как в моем примере.

Неактивен

#9 2011.02.14 11:05

TLENS
Moderator
Откуда: Украина
Зарегистрирован: 2009.04.05
Сообщений: 2402
Карма: 14
Профиль

Re: Осваиваю JS написал полезный скрипт обсудите плис

А ну я же убрал выше
Добавлено спустя   2 минуты  21 секунду:
Да это так для тренировок себе на jquery со всякими анимациями

Отредактировано TLENS (2011.02.14 11:06)

Неактивен

  • Главная
  • » JavaScript
  • » Осваиваю JS написал полезный скрипт обсудите плис

Дополнительно

forum.wapinet.ru

PunBB Mod v0.6.1
0.021 s