Simple JQuery Array

Question

I'm trying to get a random generator of text and images which should be super simple but I cannot for the life of me figure out why it's not working out. Any help is greatly appreciated!

HTML Scroll Demo #1

<body>
    <div id="container">
        <p>Where would you live and what would you drive?</p>
        <p id="text"></p>
    </div>
</body>

CSS

#container {
width: 960px;
margin-left: auto;
margin-right: auto;
border: gray solid;
padding: auto;
background-image: url(../images/house.jpg) no-repeat;
}

#text {
font-size: 40px;
text-align: left;
}

p {
width: 100px;
color: black;
font-family: arial;
font-size: 18px;
text-align: left;
}

JQUERY

$(document).ready(function(){


  var car = ["limo", "golf cart", "pig"];
  var images = ["house.jpg", "dumpster.jpg", "mansion.jpg"];

  var x = images[Math.Floor(Math.random() * images.length)];

    document.getElementById("text").innerHTML=car[x];
  $('html').css({'background-image':'url(images/' + images [x] + ')'});
    });

Thanks in advance!

Do you see any error in your console? What is the value of x? — hamed
– hamed, Commented Mar 25, 2015 at 20:05
Uncaught TypeError: undefined is not a function on script.js line 7 which is the var x=images[math.floor(math.random() *images.length)]; — oosmoos
– oosmoos, Commented Mar 25, 2015 at 20:06
Did you try Blazemonger's suggestion? you should have var x = Math.Floor(Math.random() * images.length), not x=images[math.floor(math.random() *images.length)]; — hamed
– hamed, Commented Mar 25, 2015 at 20:09

Blazemonger · Accepted Answer · 2015-03-26 13:28:39Z

4

You appear to expect x to be a number, not an image:

var x = Math.floor(Math.random() * images.length); // not Floor

edited Mar 26, 2015 at 13:28

answered Mar 25, 2015 at 19:49

Blazemonger

93.3k28 gold badges147 silver badges181 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

oosmoos Over a year ago

thank you for your response. I did change and it's still not behaving accurately. :(

oosmoos Over a year ago

Oh my god thank you so much. I can't believe that was the problem. You're the best!!

Collectives™ on Stack Overflow

Simple JQuery Array

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related