Regardles of age/experience, it's generally a good idea to use the latest (or at least current) info, especially so in programming. Using outdated information can give you a lot of headaches, talking from experience here.
I looked at your code though, to solve the most obvious problems do the following:
- Name the button on stage: bttnRoll
- you have two functions with the same name, take all the commands from the lower function and paste them into the bottom of the top function, you can delete the empty lower function now
- not sure why the "stop()" is the on top of your code, you can probably get rid of it.
let me know if that worked.