greeting
is out of scope. It was defined inside the function so it's not available at the scope you're calling alert(greeting);
. Fixing this is easy:
var greeting;
function myNameWelcome(userName, thought) {
greeting = "Welcome pardner, so your're name is " + userName + ". " + thought;
}
myNameWelcome("Peter", "Shine on you crazy diamond.");
alert(greeting);
or even better:
function myNameWelcome(userName, thought) {
return "Welcome pardner, so your're name is " + userName + ". " + thought;
}
var greeting = myNameWelcome("Peter", "Shine on you crazy diamond.");
alert(greeting);