javascript basic objects / functions

Question

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/

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 Kling, Commented Oct 13, 2011 at 23:47

Nery Jr · Accepted Answer · 2011-10-13 23:40:39Z

5

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.

edited Oct 13, 2011 at 23:40

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

Keith.Abramo · Accepted Answer · 2011-10-13 23:41:32Z

3

      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.

edited Oct 13, 2011 at 23:41

answered Oct 13, 2011 at 23:31

Keith.Abramo

6,9652 gold badges34 silver badges46 bronze badges

1 Comment

Keith.Abramo Over a year ago

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.

Blender · Accepted Answer · 2011-10-13 23:35:08Z

2

Get rid of the var deceleration (why?):

mine = {
    start: {
        modeBox: function() {
          alert();
        }
    }
};

Demo: http://jsfiddle.net/YqP93/3/

answered Oct 13, 2011 at 23:35

Blender

300k55 gold badges462 silver badges511 bronze badges

Comments

Eric H · Accepted Answer · 2011-10-14 22:45:32Z

-1

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>

edited Oct 14, 2011 at 22:45

answered Oct 13, 2011 at 23:32

Eric H

1,6763 gold badges16 silver badges18 bronze badges

1 Comment

Nery Jr Over a year ago

Answer is very wrong and still got OK. This undermines the reliability of the StackOverflow!

Collectives™ on Stack Overflow

javascript basic objects / functions

4 Answers 4

Comments

1 Comment

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

1 Comment

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related