单击按钮获得随机数

Question

I'm trying to get a random number each time someone clicks a button. The range of the numbers is from one(1) to five(5). If this question has already been asked and answered, please point me to the question and I'll delete this one to avoid duplication.

HTML

<div id="content"></div>
<button id="add">Add Object</button>

JS

$(document).ready(function(){
    var Player = {
        name: "One",
        weapon: Sword = {
            damage: 1 + Math.floor(Math.random() * 5),
            speed: 10
        }
    }
    var player_damage = Player.weapon.damage;

    $("#add").on("click", function(){
        $("#content").append(player_damage + " ");
    })
})

Fiddle

Answer 1

You don't update your number on click. You set it only once.

 var Player = { name: "One", weapon: Sword = { damage: 0, speed: 10 } } $("#add").on("click", function(){ Player.weapon.damage = 1 + Math.floor(Math.random() * 5); $("#content").append(Player.weapon.damage + " "); }) 

 <script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script> <div id="content"></div> <button id="add">Add Object</button> 

Answer 2

Generate the new number inside the click function:

 $("#add").on("click", function(){
            var damage= 1 + Math.floor(Math.random() * 5);
            $("#content").append(damage.toString() + " ");
        })

jsfiddle

Answer 3

I updated it using a JavaScript property on the Sword object. Whenever the damage property is accessed, it generates a new value using a getter:

http://jsfiddle.net/hd7tec5f/3/

Object.defineProperty(Sword, "damage", {
    get: function() {
        return 1 + Math.round(Math.random() * 1));
    }
});

Answer 4

$(document).ready(function(){
   $("#add").on("click", function(){
        var Player = {
            name: "One",
            weapon: Sword = {
                damage: 1 + Math.floor(Math.random() * 5),
                speed: 10
            }
        };

        var player_damage = Player.weapon.damage;                                           
        $("#content").append(player_damage + " ");
    })
});

http://jsfiddle.net/manzapanza/rayn6m4L/