Undefined error in JavaScript

Question

I'm getting data from XML. I can successfully pick up a price from the XML but there is a unexpected error called undefined that shows up when I use the function given below;

<html>
  <head>
    <script type="text/javascript">
      function myXml(origin, destination) {
        var x=xmlDoc.getElementsByTagName("flights");

        for(i=0;i<x.length;i++) {
          if(x[i].getAttribute('FrTLAs')==origin && x[i].getAttribute('destination')==destination) {
            document.write(x[i].getAttribute('price'))
          }
        }
      }
    </script>
  </head>
  <body>
    <script type="text/javascript">
      document.write(myXml('SYD','Bali'));
    </script>
  </body>
</html>

if xml is not defined so how come my code pick a price from xml. result on browser is 636undefined — Jitender
– Jitender, Commented Apr 14, 2012 at 10:44
Not related to your problem, but better declare i as a variable in the local function scope: for(var i=0;.. — Niko
– Niko, Commented Apr 14, 2012 at 10:53

Engineer · Accepted Answer · 2012-04-14 10:48:49Z

3

myXml('SYD','Bali') call returns undefined, as you do not return anything in function body. So

document.write(myXml('SYD','Bali'));

will print "undefined" . Just replace above code with this:

myXml('SYD','Bali');

answered Apr 14, 2012 at 10:48

Engineer

48.9k12 gold badges90 silver badges93 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

hitesh israni · Accepted Answer · 2012-04-14 10:52:19Z

1

Engineer is correct, or better return the value from your myXml function.

so, document.write(undefined) wont occur and you may not get the above error.

answered Apr 14, 2012 at 10:52

hitesh israni

1,7524 gold badges26 silver badges49 bronze badges

Collectives™ on Stack Overflow

Undefined error in JavaScript

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related