Javascript无法调用未定义的方法 'indexOf'

Question

我需要创建一个对字符串进行排序的函数。字符串中的每个单词都将包含一个数字。数字可以是 1 到 9(不能是 0)。

例如输入:“is2 Thi1s T4est 3a”，函数应返回“Thi1s is2 3a T4est”。

我的代码是:

 function order(words)
 {
 // ... 

    if(words == '')
    {
      return words;
    }

    var all_words = words.split(" ");
    var checked_words = new Array(); 
    var joined_words = "";

    for(i = 1; i <= 9; i++)
    {
      //console.log(checked_words);
      //checked_words[i-1] = all_words;
      for(j = 1;j <= all_words.length; j++)
      {
        if(all_words[i-1].indexOf(i) != -1)
        {
          checked_words.push(all_words[i-1]);

          if(i == (all_words.length))
          {
            joined_words = checked_words.join(" ");
            return joined_words;
          }
        }
      }
    }
  }

问题是它一直显示“TypeError:无法按顺序调用未定义的方法'indexOf'”。请帮忙谢谢!

Answer 1

我一直试图弄清楚第一个循环在做什么，我终于意识到它正在检查字符串中的数字。你不应该这样做，因为它 1) 很奇怪，2) 如果它有效的话，效果不佳。

而是使用一个循环和正则表达式来查找字符串中的数字。

for (var i = 0; i < all_words.length; i++) {

  // take the first match and coerce it to an integer
  var n = +all_words[i].match(/\d+/)[0];
  checked_words[n] = all_words[i];
}

// checked_words has an undefined element at index 0;
// slice takes a copy of the array from index 1
return checked_words.slice(1).join(' ');

DEMO

或者，您可以使用reduce。

var joined_words = str.split(' ').reduce(function (p, c) {
  var n = +c.match(/\d+/)[0];
  p[n] = c;
  return p;
}, []).slice(1).join(' ');

DEMO

您甚至不受数字 0-9 的束缚。只要你过滤掉未定义的元素，你就是黄金......

var str = 'is2 Thi1s T4est 3a Ye12s!';

var out = str.split(' ').reduce(function (p, c) {
  var n = +c.match(/\d+/)[0];
  p[n] = c;
  return p;
}, []).slice(1).filter(function (el) {
  return el !== undefined;
}).join(' ');

DEMO