Question

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

例如输入：＆＃34; is2 Thi1s T4est 3a＆＃34;该函数应该返回＆＃34; Thi1s是2 3a T4est＆＃34;。

我的代码是：

 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;
          }
        }
      }
    }
  }

问题是它一直显示＆＃34; TypeError：无法调用方法＆＃39; indexOf＆＃39;未定义的顺序＆＃34;。请帮助谢谢！

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

Javascript无法调用方法＆＃39; indexOf＆＃39;未定义的订单

1 个答案: