Suggestions to optimize Random Quote generator

Suggestions to optimize Random Quote generator
0

#1

Hey all. I got a pretty rudimentary random quote generator here: https://codepen.io/DebugAyem/pen/XgMWJJ
I’m pretty new to using javascript so it was the best I could come up with. I generated a random number and used a switch case to assign the quote to a quote parameter based on the number selected. I’m sure there are other ways to do this and so I was looking for suggestions to make this better and faster (without using API’s because I have no clue how to utilize that right now). Otherwise I’ll just keep hardcoding quotes in there. Any other recommendations are welcome. Cheers!!


#2

It is a pretty simple task so fon’t worry about optimizing for speed. You could however use an array instead of a switch statement.

Store the quotes in the array.
Generate the random number such that it is between 0 and the length of the array minus one.
Retrieve the quote from the array by index.

That should make the code look a bit better.


#3

Oh great Idea! I remember in another post someone talking about using objects. Any clue how that would be done?


#4

Well you could use an object as an associative array. This is because you can iterate through its props with a for…in statemment. But its more work than an array in your case.


#5

After a journey in which I rediscovered myself, scoured a lot of youtube videos, dabbled in some php, reupped my jquery, and googled “what is an api and why can’t I figure out how to use it” I finally cracked it thanks to fcc,codecademy,udemy,stack overflow and a very cool youtube (https://www.youtube.com/watch?v=5xeDQ1-ZfcQ) video.
This is it by the way
https://codepen.io/DebugAyem/pen/zdxLBd

P.S. Now that I get it, it’s actually pretty easy haha just had to cut through all the noise.