Wiele dopasowań ciągów z indexOf()

Question

Jestem pewien, że moja składnia jest zła, ponieważ skrypt działa tylko wtedy, gdy ciąg znaków pasuje do "wideo", jeśli ciąg ma słowo "dźwięk", jest ignorowany. wartość "#" przekierowania dla "../../../index.html" robi praca.

js

var ua = navigator.userAgent.toLowerCase(); 
var isIE8 = /MSIE 8.0/i.test(ua); 
if (isIE8) { 
    $('a').click(function() { 
     var srcTag = $(this).find('img').attr('src'); 
     if (srcTag.indexOf('Video' || 'Audio') > -1) { 
      if (confirm('Download Safari? \n\n http://apple.com/safari/download/')) { 
      window.location = 'http://apple.com/safari/download/'; 
      } else { window.location = '../../../index.html';} 
     } else { 
      alert('no match'); 
     } 
    }); 
} 

html

<a href="#"><img src="Video/000_Movies/assets/005_CCC_Jesus_Story_80x60.jpg" />test1</a> 
<a href="#"><img src="Audio/000_Movies/assets/006_GSP_Gods_Story_80x60.jpg" />test2</a> 
<a href="#"><img src="Media/000_Movies/assets/002_God_Man_80x60.jpg" />test3</a> 

Answer 1

O wiele krótsze jest przekształcenie tego w wyrażenie regularne.

if (srcTag.match(/(video|audio)/)) { 
    /* Found */ 
} else { 
    /* Not Found */ 
} 

Na marginesie, proszę nie rób tego, co próbujesz zrobić. Prośba użytkowników o pobranie Safari podczas korzystania z przeglądarki Internet Explorer 8 jest szkodliwa dla Internetu, a także dla tego użytkownika.

chodzi o przekierowanie domeny do innej lokalizacji, należy użyć .preventDefault() zachować przeglądarkę następujący link:

$("a.videoDownload").on("click", function(e){ 
    e.preventDefault(); 
    if (this.getElementsByTagName("img")[0].src.match(/(video|audo)/)) { 
    window.location = confirm('Download Safari?') 
     ? "http://apple.com/safari/download" 
     : "../../../index.html" ; 
    } else { 
    /* No match */ 
    } 
}); 

Ponownie proszę nie faktycznie to zrobić. Nikt nie chce być tym facetem, a kiedy powiesz użytkownikom, aby pobrać inną przeglądarkę, jesteś ten facet.

Answer 2

myślę, czego potrzeba 2 oddzielnych indexOf jak poniżej,

srcTag.indexOf('Video') != -1 || srcTag.indexOf('Audio') != -1 

Answer 3

Tak, trzeba by było coś takiego to zrobić:

if (srcTag.indexOf('Video') > -1 || srcTag.indexOf('Audio') > -1) { 

Answer 4

myślę, że pewnie chcą OR (||) operatora poza indexOf jak tak:

if ((srcTag.indexOf('Video') !== -1) || (srcTag.indexOf('Audio') !== -1)) { 
    ... 
} 

Answer 5

'Video' || 'Audio' jest logicznym OR. Niepusty łańcuch jest niejawnie prawdziwą wartością w kodzie JavaScript, a zatem zwarta OR nie jest oceniana, a ta zapada się tylko na 'Video'. Dlatego widzisz wyniki, które robisz.

Inni wskazali ci właściwy kierunek, aby rozwiązać.

Answer 6

Będzie to również działa:

if (srcTag.indexOf('Video') >= -1 || srcTag.indexOf('Audio') >=-1) { 

Answer 7

Ten pracował dla mnie:

if (srcTag.indexOf('Video' | 'Audio') >= -1) { 

Answer 8

Jego dość szybciej regex i coraz lepiej z XRegExp.

var sourceString = 'hello world, i am web developer'; 
if (XRegExp.test(sourceString, /(hello|web)/)) { 
    // yes, `hello` or `web` is found in `sourceString` 
} 

Czas działania wynosi 0.10595703125ms