i tried to pop an alert when clicked on a div , didn't really succeed , i messed it a little to practice with objects.
here is the the code :
<!DOCTYPE html>
<html>
<head>
<script type="text/javascript">
var mine = {
var start = {
modeBox : function(){
alert();
}
}
}
</script>
</head>
<body>
<div id="askMode" onclick="mine.start.modeBox();">My mine</div>
</body>
</html>
the problem : not alerting
why i don't get alert when i click on the div? live example : http://jsfiddle.net/YqP93/
asked Oct 13, 2011 at 23:24
homerun
20.9k15 gold badges49 silver badges71 bronze badges
-
Have a look at the console and you will see that you get a syntax error. Learn about the JavaScript basics and especially how to debug JavaScript code.Felix Kling– Felix Kling2011年10月13日 23:47:12 +00:00Commented Oct 13, 2011 at 23:47
4 Answers 4
The following code tested in Firefox and Chrome:
<!DOCTYPE html>
<html>
<head>
<script type="text/javascript">
var mine = {
start: {
modeBox : function(){
alert('Test');
}
}
}
</script>
</head>
<body>
<div id="askMode" onclick="mine.start.modeBox();">My mine</div>
</body>
</html>
NOTE: the code you posted in jsfiddle is not working, but you can copy and paste the code above that works in browsers mentioned.
answered Oct 13, 2011 at 23:30
Nery Jr
4,0271 gold badge28 silver badges24 bronze badges
Sign up to request clarification or add additional context in comments.
Comments
this.mine = {
start: {
modeBox : function(){
alert();
}
}
you are trying to declare a var in an object literal. I don't think this is valid js for one but more importantly you can not get at vars declared in that scope. You have to assign the property to the object. You had it right with the modeBox.
answered Oct 13, 2011 at 23:31
Keith.Abramo
6,9652 gold badges34 silver badges46 bronze badges
1 Comment
Keith.Abramo
It appears the "var mine" is also out of scope when trying to access it from an onclick event. You need to do this.mine to bind it to the global scope. I have updated my answer to reflect this.
Get rid of the var deceleration (why?):
mine = {
start: {
modeBox: function() {
alert();
}
}
};
answered Oct 13, 2011 at 23:35
Blender
300k55 gold badges463 silver badges513 bronze badges
Comments
try this
<!DOCTYPE html>
<html>
<head>
<script type="text/javascript">
var mine = {
start: {
modeBox: function(){
alert();
}
}
}
</script>
</head>
<body>
<div id="askMode" onclick="mine.start.modeBox();">My mine</div>
</body>
</html>
answered Oct 13, 2011 at 23:32
Eric H
1,6763 gold badges16 silver badges18 bronze badges
1 Comment
Nery Jr
Answer is very wrong and still got OK. This undermines the reliability of the StackOverflow!
lang-js