24

我很确定我的语法是错误的，因为脚本只有在字符串匹配“视频”时才有效，如果字符串有“音频”这个词，它会被忽略。此外，由于 href 标记的值为“#”，重定向为“ ../../../index.html" 不起作用。

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>
4

9 回答 9

92

将其转换为正则表达式要短得多。

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

在旁注中，请不要做你想做的事情。要求用户在使用 Internet Explorer 8 时下载 Safari，这对 Internet 以及该用户都是有害的。

至于将域重定向到另一个位置，您应该使用.preventDefault()阻止浏览器跟踪链接：

$("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 */
  }
});

再次，请不要真的这样做。没有人想成为那个人，当你告诉用户下载另一个浏览器时，你就是那个人。

于 2012-05-10T15:28:55.633 回答
9

'Video' || 'Audio' is a logical OR. A non-empty string is implicitly a true value in JavaScript, and thus the short-circuited OR is not evaluated and this collapses to just 'Video'. This is why you see the results you do.

Others have pointed you in correct directions to resolve.

于 2012-05-10T15:30:08.053 回答
4

我认为您可能希望在 indexOf 之外使用 OR (||) 运算符：

if ((srcTag.indexOf('Video') !== -1) || (srcTag.indexOf('Audio') !== -1)) {
  ...
}
于 2012-05-10T15:28:12.560 回答
2

我认为您需要的是 2 个单独的 indexOf，如下所示，

srcTag.indexOf('Video') != -1  || srcTag.indexOf('Audio') != -1
于 2012-05-10T15:27:43.077 回答
1

是的，你需要这样做：

if (srcTag.indexOf('Video') > -1 || srcTag.indexOf('Audio') > -1) {
于 2012-05-10T15:27:54.217 回答
1

简单的递归函数，您可以获得多个匹配项的索引

function getMultipleIndexOf(str, searchQuery, acc = 0, result = []) {
  if (!str || !searchQuery) {
    return { result, acc };
  }
  const foundIndex = str.toLowerCase().indexOf(searchQuery.toLowerCase());
  if (foundIndex < 0) {
    return { result, acc };
  }
  return getMultipleIndexOf(
    str.slice(foundIndex + searchQuery.length),
    searchQuery,
    acc + foundIndex + searchQuery.length,
    [...result, acc + foundIndex]
  );
}

const res = getMultipleIndexOf('long text some how long text!!', 'te');
// { result: [ 5, 24 ], acc: 26 }

只需忽略跟踪先前找到的文本所需的 acc

于 2020-10-16T09:22:55.317 回答
-1

使用正则表达式更快，使用XRegExp更好。

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

执行时间为 0.10595703125ms

于 2017-11-10T15:34:43.083 回答
-2

这也将起作用：

if (srcTag.indexOf('Video') >= -1 || srcTag.indexOf('Audio') >=-1 ) {
于 2013-11-28T09:43:41.360 回答
-3

这对我有用：

if (srcTag.indexOf('Video' | 'Audio' ) >= -1 ) {
于 2017-06-21T10:28:46.717 回答