-1

このコードは 8 つの div を取り、div に割り当てられた時間値に応じて画面上でそれらを競合させます。私のコードがそれほどアマチュアっぽく見えないようにするには、どうすればこれをより適切にコーディングできますか?

for ループでハード コードされた 8 以外のものを使用する必要があることはわかっていますが、スプライスを使用して配列から項目を削除しているため、timeArray.length は使用できません。

$(document).ready(function() {
var timeArray = new Array(3,4,5,6,7,8,9,10);
var shortestTime = timeArray[7];
var fastestPony = {};
var index;

var pony = {
    name: "pony",
    raceTime: -1,
    selected: ""
};

//change the color of the pony when the user clicks on it
$('.pony').bind('click', function() {
    $('.pony').removeClass('selectedPony');
    $(this).addClass('selectedPony');

    //get the pony that the user selected
    pony.selected = $(this);
});

$('#startButton').click(function() { 
    if (pony.selected == "") {
        alert("Please click the pony you think will win the race.");
    }
    else {
        for (i = 1; i <= 8; i++) {
        //get a random number from the timeArray
            index = Math.floor(Math.random() * timeArray.length);
            pony.raceTime = timeArray[index];

            //pull the random race time number out of the array 
            //so it can't be assigned to another horse
            timeArray.splice(index, 1);

            //get the fastest pony
            if (pony.raceTime < shortestTime) {
                shortestTime = pony.raceTime;
                fastestPony = $('#pony' + i);
            }

            //award the winner after the ponies have reached the finish line
            if (i == 8) {
                fastestPony.addClass('winner').append(' - Winner!');
            }

            //send the horses on their way to race!
            $('#pony' + i).animate({left: '320px'}, pony.raceTime * 1000);
        }
    }
});

//reset the ponies back to the starting line by reloading the page
$('#resetButton').click(function() {
    document.location.reload(true);
});
});
4

1 に答える 1

0

これは本当に好みと細かいところの問題ですが、あなたが尋ねたので...

on(...)ほとんどのサークルでこの方法が好まれていると聞きました。

$('.pony').on('click', function() {

と

$('#startButton').on('click', function() {

もちろん、これは使用している jQuery のバージョンに依存し、完全に主観的なものになる可能性があります。

于 2013-03-06T18:37:42.510 に答える