0

我正在尝试编写一个脚本，从对象列表中随机选择三个项目，按一个字段将它们按字母顺序排列，然后显示另一个字段的字母顺序值。到目前为止，我可以让它随机选择项目并显示它们，但它仍然没有排序。这是我的代码：

function random_text() {};
var random_text = new random_text();
var number = 0;
random_text[number++] = {
    show: "Bob",
    sort: "A"
}
random_text[number++] = {
    show: "Casey",
    sort: "B"
}
random_text[number++] = {
    show: "Dan",
    sort: "C"
}
random_text[number++] = {
    show: "Alfred",
    sort: "D"
}

var random_number = Math.floor(Math.random() * number);
var random_number1 = Math.floor(Math.random() * number);
var random_number2 = Math.floor(Math.random() * number);

if (random_number !== random_number1) {
    var name1 = random_text[random_number1].show;
} else {
    var name1 = "";
}

if (random_number !== random_number2 && random_number1 !== random_number2) {
    var name2 = random_text[random_number2].show;
} else {
    var name2 = "";
}

var name3 = random_text[random_number].show;

var objs = [name1 + " " + name2 + " " + name3];

objs.sort(function (a, b) {
    var nameA = a.sort.toLowerCase(),
        nameB = b.sort.toLowerCase()
        if (nameA < nameB) //sort string ascending
    return -1
    if (nameA > nameB) return 1
    return 0 //default return value (no sorting)
})

document.write(objs.sort());

在此先感谢您的帮助。如果有任何方法可以清理代码，我也很感激这方面的提示。

4

1 回答 1

0

见http://jsfiddle.net/Nc2Je/

var random_text = [
    {
        show: "Bob",
        sort: "A"
    },
    {
        show: "Casey",
        sort: "B"
    },
    {
        show: "Dan",
        sort: "C"
    },
    {
        show: "Alfred",
        sort: "D"
    }
],random_numbers = new Array(),
    objs=new Array();
for(var i=0;i<3;i++){
    function random(){
        var n=Math.floor(Math.random() * random_text.length);
        if(random_numbers.indexOf(n)===-1){return n;}
        return random();
    }
    random_numbers.push(random());
    objs.push(random_text[random_numbers[i]])
}
var alphabetized=new Array();
outerloop:
for(var i=0;i<objs.length;i++){
    for(var j=0;j<alphabetized.length;j++){
        if(alphabetized[j].sort>objs[i].sort){
            alphabetized.splice(j,0,objs[i]);
            continue outerloop;
        }
    }
    alphabetized.push(objs[i]);
}
alphabetized.getNames=function () {
    var arr=new Array();
    for(var i=0;i<this.length;i++){
        arr.push(this[i].show);
    }
    return arr;
}

document.write(alphabetized.getNames());

编辑：

你做了一些奇怪的事情：

1-为什么要创建一个空函数，然后用它自己的实例覆盖它？

function random_text() {};
var random_text = new random_text();

2-如果您想要一个像数组一样工作的对象，为什么不使用数组而不是将类似数组的属性设置为random_text？

var number = 0;
random_text[number++] = {
    show: "Bob",
    sort: "A"
}
random_text[number++] = {
    show: "Casey",
    sort: "B"
}
random_text[number++] = {
    show: "Dan",
    sort: "C"
}
random_text[number++] = {
    show: "Alfred",
    sort: "D"
}

3-您应该使用数组random_numbers而不是那些变量：

var random_number = Math.floor(Math.random() * number);
var random_number1 = Math.floor(Math.random() * number);
var random_number2 = Math.floor(Math.random() * number);

4- 而不是那些奇怪的检查，你应该确保你的随机数与以前的随机数不同。用你的支票，可能三个随机数是一样的，所以你只有一个名字。

if (random_number !== random_number1) {
    var name1 = random_text[random_number1].show;
} else {
    var name1 = "";
}

if (random_number !== random_number2 && random_number1 !== random_number2) {
    var name2 = random_text[random_number2].show;
} else {
    var name2 = "";
}

var name3 = random_text[random_number].show;

5- 为什么要创建一个只有一个元素（字符串）的数组？

var objs = [name1 + " " + name2 + " " + name3];

6-我没有使用您的以下代码，因为我从未使用过，所以我编写了自己的排序函数。

objs.sort(function (a, b) {
    var nameA = a.sort.toLowerCase(),
        nameB = b.sort.toLowerCase()
        if (nameA < nameB) //sort string ascending
    return -1
    if (nameA > nameB) return 1
    return 0 //default return value (no sorting)
})

7- 对对象进行排序后，如果要编写它们，则应不document.write(objs)使用sort().

编辑2：

alphabetized.getNames返回一个数组。

这样你就可以做到

var names=alphabetized.getNames();
names.length;
names[0];
...

如果你想要一个字符串，你可以做alphabetized.getNames().join(" ");

于 2012-08-30T23:12:48.613 回答