Hi Ram,
This is a good effort, thanks for trying out the example. Your code has gone a bit wrong in a few places. You are using variables in your string rather than placeholders that could be substituted. This is an ok alternative approach, but you need to make sure that variables you are trying to use are declared before you try to use them in the string.
Also, think about where you are playing other instructions — in this case you are only creating a single silly story, as soon as the app loads, but all the variables are coming up undefined (for reasons touched on above). You want to make it so that a silly story is generated each time the button is pressed, but not before.
You can find our finished version here, for reference:
- Marking guide: https://github.com/mdn/learning-area/blob/master/javascript/introduction-to-js-1/assessment-finished/marking-guide.md 3
- Source code: https://github.com/mdn/learning-area/blob/master/javascript/introduction-to-js-1/assessment-finished/main.js 1
- Live example: https://mdn.github.io/learning-area/javascript/introduction-to-js-1/assessment-finished/