Проверка ссылок через JS?

Question

[Дмитрий Полушкин]

Всем привет!

Подскажите пожалуйста в чем проблема, только начинаю в JS .

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

var hrefs = document.getElementsByTagName('a');
for (var i = 0; i < hrefs.length; i++) {
    if (hrefs[i].host == 'нужный линк') {
        console.log('Совпало');
    } else {
        console.log('Не совпало');
    }
}
console.log(hrefs);

Comments

[ProjectSoft]

Дмитрий Полушкин Что представляет из себя "нужный линк"
Данным кодом вы можете проверять только доменное имя.
Если ваш код запустить в консоли на данной странице, то всё сработает

Array.prototype.slice.call(document.getElementsByTagName('a')).forEach(function(el){
	if (el.host == 'toster.ru') {
        console.log('Совпало');
    } else {
        console.log('Не совпало');
    }
})

Answer 1

[Arman]

логируйте подробнее

console.log('Не совпало: '+hrefs[i].host+' != ' + 'нужный линк');

Answer 2

[be_a_man]

var hrefs = document.querySelectorAll('a');
if (hrefs[i].getAttribute('href') == 'нужный линк')

https://jsfiddle.net/se5kbwrx/

UPD: с getElementsByTagName также работает. Решение проблемы именно в получении атрибута .getAttribute('href')

Comments

[Алексей]

В контексте данного вопроса в чем разница с getElementsByTagName кроме того что (querySelectorAll) работает медленнее? В Вашем же примере работает корректно и то и то.

[be_a_man]

Алексей, в контексте данного примера разницы нет.

кроме того что (querySelectorAll) работает медленнее?

Вы действительно где-либо чувствовали что querySelectorAll медленный и это было проблемой?
https://jsfiddle.net/5z7pet1w/

[Алексей]

be_a_man, я нет, но мы не знаем сколько ссылок у ТС и как долго будет отрабатывать данный код.
И смена метода получения объектов не решает данный вопрос, тут скорее Вы правильно указали что ТС не тот атрибут проверяет.

[be_a_man]

Алексей, мы также не знаем знает ли ТС о существовании querySelectorAll, вдруг не знает, заодно погуглит что это))

смена метода получения объектов не решает данный вопрос

да, вы правы, дело привычки, не помню когда посл. раз делал выборку селекторов по тегу, а не по классу. Поправлю ответ. Спасибо за критику