Question

83

Multiple string matches with indexOf()

rated 0 times [ 87] [ 4] / answers: 1 / hits: 71774 / 12 Years ago, thu, may 10, 2012, 12:00:00

Im pretty sure my syntax this wrong because the script only works if the string matches Video, if the string has the word Audio it is ignored. Also since the href tags have a value of # the redirect for ../../../index.html doesnt work.

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? nn 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>

Answers

Only authorized users can answer the question. Please sign in first, or register a free account.

deanna

Add To Favorites

Follow

Total Points: 84

Total Questions: 86

Total Answers: 107

Location: Cyprus

Member since Wed, Dec 8, 2021

3 Years ago

deanna questions

1 CastError: Cast to ObjectId failed for value XXXXXXXXXX (type string) at path _id for model XXXX

Thu, Apr 21, 22, 00:00, 2 Years ago

1 Next.js behavior on back button pressed

Mon, Sep 6, 21, 00:00, 3 Years ago

1 React-Router + Typescript error while wrapping component with Router No overload matches this call

Sat, Feb 13, 21, 00:00, 3 Years ago

1 Use the webpack asset module with a custom output filename

Sun, Jan 17, 21, 00:00, 3 Years ago

1 React js random color

Sun, Nov 15, 20, 00:00, 4 Years ago

View All

answered 12 Years ago stephonkeandrer · Accepted Answer

It's far shorter to turn this into a regular expression.

if ( srcTag.match( /(video|audio)/ ) ) {

  /* Found */

} else {

  /* Not Found */

}

On a side note, please don't do what you're attempting to do. Asking users to download Safari when they're using Internet Explorer 8 is doing a disservice to the Internet, as well as to that user.

As for redirecting the domain to another location, you should use .preventDefault() to keep the browser from following the 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 */

  }

});

Again, please don't actually do this. Nobody wants to be that guy, and when you tell users to download another browser, you're being that guy.