shreef_1990 shreef_1990 - 2 months ago 5
Javascript Question

what is wrong with .this. javascript if else code?

it works if i use the first half only but i need to widen the parameters

//document.querySelectorAll('font[color="black"]');
var fonts = document.querySelectorAll('font[color="black"]');
var searchString = 'Mir';
var searchString2 = 'MirrorCreator';
for (var i = 0; i < fonts.length; i++) {
var font = fonts[i];
if (fonts[i].innerHTML.indexOf(searchString) !== - 1) {
//alert('Match');
var eventLink = 'ComeHere';
var elA = document.createElement('a');
elA.setAttribute('id', eventLink);
elA.setAttribute('name', eventLink);
font.appendChild(elA);
window.location.hash = 'ComeHere';
break;
}
else (fonts[i].innerHTML.indexOf(searchString2) !== - 1) {
//alert('Match');
var eventLink2 = 'ComeHere2';
var elA2 = document.createElement('a');
elA2.setAttribute('id', eventLink2);
elA2.setAttribute('name', eventLink2);
font.appendChild(elA2);
window.location.hash = 'ComeHere2';
break;
}
}

Answer

Here you have wrong syntax:

else (fonts[i].innerHTML.indexOf(searchString2) !== - 1) {

It should be simple

else {

or

else if (fonts[i].innerHTML.indexOf(searchString2) !== - 1) {
Comments