524

What are the possible reasons for document.getElementById, $("#id") or any other DOM method / jQuery selector not finding the elements?

Example problems include:

  • jQuery silently failing to bind an event handler
  • jQuery "getter" methods (.val(), .html(), .text()) returning undefined
  • A standard DOM method returning null resulting in any of several errors:

Uncaught TypeError: Cannot set property '...' of null Uncaught TypeError: Cannot read property '...' of null

The most common forms are:

Uncaught TypeError: Cannot set property 'onclick' of null

Uncaught TypeError: Cannot read property 'addEventListener' of null

Uncaught TypeError: Cannot read property 'style' of null

StudioTime
  • 18,874
  • 30
  • 103
  • 186
Felix Kling
  • 705,106
  • 160
  • 1,004
  • 1,072
  • 38
    Many questions are asked about why a certain DOM element is not found and the reason is often because the JavaScript code is placed before the DOM element. This is intended to be a canonical answer for these type of questions. It's community wiki, so **please feel free to improve it**. – Felix Kling Dec 25 '12 at 08:28

6 Answers6

527

The element you were trying to find wasn’t in the DOM when your script ran.

The position of your DOM-reliant script can have a profound effect upon its behavior. Browsers parse HTML documents from top to bottom. Elements are added to the DOM and scripts are (generally) executed as they're encountered. This means that order matters. Typically, scripts can't find elements which appear later in the markup because those elements have yet to be added to the DOM.

Consider the following markup; script #1 fails to find the <div> while script #2 succeeds:

<script>
  console.log("script #1: %o", document.getElementById("test")); // null
</script>
<div id="test">test div</div>
<script>
  console.log("script #2: %o", document.getElementById("test")); // <div id="test" ...
</script>

So, what should you do? You've got a few options:


Option 1: Move your script

Move your script further down the page, just before the closing body tag. Organized in this fashion, the rest of the document is parsed before your script is executed:

<body>
  <button id="test">click me</button>
  <script>
    document.getElementById("test").addEventListener("click", function() {
      console.log("clicked: %o", this);
    });
  </script>
</body><!-- closing body tag -->

Note: Placing scripts at the bottom is generally considered a best practice.


Option 2: jQuery's ready()

Defer your script until the DOM has been completely parsed, using $(handler):

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<script>
  $(function() {
    $("#test").click(function() {
      console.log("clicked: %o", this);
    });
  });
</script>
<button id="test">click me</button>

Note: You could simply bind to DOMContentLoaded or window.onload but each has its caveats. jQuery's ready() delivers a hybrid solution.


Option 3: Event Delegation

Delegated events have the advantage that they can process events from descendant elements that are added to the document at a later time.

When an element raises an event (provided that it's a bubbling event and nothing stops its propagation), each parent in that element's ancestry receives the event as well. That allows us to attach a handler to an existing element and sample events as they bubble up from its descendants... even those added after the handler is attached. All we have to do is check the event to see whether it was raised by the desired element and, if so, run our code.

jQuery's on() performs that logic for us. We simply provide an event name, a selector for the desired descendant, and an event handler:

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<script>
  $(document).on("click", "#test", function(e) {
    console.log("clicked: %o",  this);
  });
</script>
<button id="test">click me</button>

Note: Typically, this pattern is reserved for elements which didn't exist at load-time or to avoid attaching a large amount of handlers. It's also worth pointing out that while I've attached a handler to document (for demonstrative purposes), you should select the nearest reliable ancestor.


Option 4: The defer attribute

Use the defer attribute of <script>.

[defer, a Boolean attribute,] is set to indicate to a browser that the script is meant to be executed after the document has been parsed, but before firing DOMContentLoaded.

<script src="https://gh-canon.github.io/misc-demos/log-test-click.js" defer></script>
<button id="test">click me</button>

For reference, here's the code from that external script:

document.getElementById("test").addEventListener("click", function(e){
   console.log("clicked: %o", this); 
});

Note: The defer attribute certainly seems like a magic bullet but it's important to be aware of the caveats...
1. defer can only be used for external scripts, i.e.: those having a src attribute.
2. be aware of browser support, i.e.: buggy implementation in IE < 10

canon
  • 37,149
  • 9
  • 67
  • 92
  • 1
    It's now 2020 — is "Placing scripts at the bottom" still "considered a best practice" ? I (nowdays) put all my resources in the `` and use `defer` on scripts (I don't have to support bad-old incompatible browsers) – Stephen P May 05 '20 at 17:53
  • 1
    I'm a big proponent of moving onto modern browsers. That said, this practice doesn't really cost me anything and it just works everywhere. On the other hand, both [`defer`](https://caniuse.com/#feat=script-defer) and [`async`](https://caniuse.com/#feat=script-async) have pretty broad support -even reaching back to some much older browser versions. They have the additional advantage of clearly, explicitly signaling the intended behavior. Just remember that these attributes only work for scripts specifying a `src` attribute, i.e.: external scripts. Weigh the benefits. Maybe use both? Your call. – canon May 05 '20 at 19:39
  • 1
    In my opinion, this answer should also include a reference to https://developer.mozilla.org/en-US/docs/Web/API/Window/DOMContentLoaded_event in approach #2. Essentially that is the vanilla JS solution to this issue, which probably something at least some Googlers would like to know about. – Joeytje50 Mar 09 '21 at 16:32
  • 1
    @Joeytje50 [DOMContentLoaded](https://developer.mozilla.org/en-US/docs/Web/Reference/Events/DOMContentLoaded) is already referenced in a footnote for #2. That said, this answer was originally migrated by a moderator from a jQuery-specific question. So, it still bears some of that flavor, despite some transitional edits. – canon Mar 10 '21 at 00:58
154

Short and simple: Because the elements you are looking for do not exist in the document (yet).


For the remainder of this answer I will use getElementById as example, but the same applies to getElementsByTagName, querySelector and any other DOM method that selects elements.

Possible Reasons

There are two reasons why an element might not exist:

  1. An element with the passed ID really does not exist in the document. You should double check that the ID you pass to getElementById really matches an ID of an existing element in the (generated) HTML and that you have not misspelled the ID (IDs are case-sensitive!).

    Incidentally, in the majority of contemporary browsers, which implement querySelector() and querySelectorAll() methods, CSS-style notation is used to retrieve an element by its id, for example: document.querySelector('#elementID'), as opposed to the method by which an element is retrieved by its id under document.getElementById('elementID'); in the first the # character is essential, in the second it would lead to the element not being retrieved.

  2. The element does not exist at the moment you call getElementById.

The latter case is quite common. Browsers parse and process the HTML from top to bottom. That means that any call to a DOM element which occurs before that DOM element appears in the HTML, will fail.

Consider the following example:

<script>
    var element = document.getElementById('my_element');
</script>

<div id="my_element"></div>

The div appears after the script. At the moment the script is executed, the element does not exist yet and getElementById will return null.

jQuery

The same applies to all selectors with jQuery. jQuery won't find elements if you misspelled your selector or you are trying to select them before they actually exist.

An added twist is when jQuery is not found because you have loaded the script without protocol and are running from file system:

<script src="//somecdn.somewhere.com/jquery.min.js"></script>

this syntax is used to allow the script to load via HTTPS on a page with protocol https:// and to load the HTTP version on a page with protocol http://

It has the unfortunate side effect of attempting and failing to load file://somecdn.somewhere.com...


Solutions

Before you make a call to getElementById (or any DOM method for that matter), make sure the elements you want to access exist, i.e. the DOM is loaded.

This can be ensured by simply putting your JavaScript after the corresponding DOM element

<div id="my_element"></div>

<script>
    var element = document.getElementById('my_element');
</script>

in which case you can also put the code just before the closing body tag (</body>) (all DOM elements will be available at the time the script is executed).

Other solutions include listening to the load [MDN] or DOMContentLoaded [MDN] events. In these cases it does not matter where in the document you place the JavaScript code, you just have to remember to put all DOM processing code in the event handlers.

Example:

window.onload = function() {
    // process DOM elements here
};

// or

// does not work IE 8 and below
document.addEventListener('DOMContentLoaded', function() {
    // process DOM elements here
});

Please see the articles at quirksmode.org for more information regarding event handling and browser differences.

jQuery

First make sure that jQuery is loaded properly. Use the browser's developer tools to find out whether the jQuery file was found and correct the URL if it wasn't (e.g. add the http: or https: scheme at the beginning, adjust the path, etc.)

Listening to the load/DOMContentLoaded events is exactly what jQuery is doing with .ready() [docs]. All your jQuery code that affects DOM element should be inside that event handler.

In fact, the jQuery tutorial explicitly states:

As almost everything we do when using jQuery reads or manipulates the document object model (DOM), we need to make sure that we start adding events etc. as soon as the DOM is ready.

To do this, we register a ready event for the document.

$(document).ready(function() {
   // do stuff when DOM is ready
});

Alternatively you can also use the shorthand syntax:

$(function() {
    // do stuff when DOM is ready
});

Both are equivalent.

Felix Kling
  • 705,106
  • 160
  • 1,004
  • 1,072
  • 1
    Would it be worth amending the last bit about the `ready` event to reflect the "newer" preferred syntax, or is it better to leave it as-is so that it works across older versions? – Tieson T. Dec 25 '12 at 08:56
  • 1
    For jQuery, is it worth also adding in the alternative `$(window).load()` (and possibly syntactic alternatives to the `$(document).ready()`, `$(function(){})`? It seems related, but it *feels* slightly tangential to the point you're making. – David says reinstate Monica Dec 25 '12 at 08:58
  • @David: Good point with `.load`. Regarding syntax alternatives, they could be looked up in the documentation but since answers should be self contained, it might actually be worth adding them. Then again, I'm pretty sure this specific bit about jQuery was answered already and is probably covered in another answers and linking to it might suffice. – Felix Kling Dec 25 '12 at 09:00
  • 1
    @David: I have to revise: Apparently `.load` is deprecated: http://api.jquery.com/load-event/. I don't know if it's for images only or `window` as well. – Felix Kling Dec 25 '12 at 09:03
  • Those are pretty much the reasons I didn't add those parts into the answer (despite the CW); but, as you say the 'answers should be self-contained.' Though if this is intended to become a/the canonical answer on the matter, it might still be worth adding? I'm...I dunno, I feel like I'm trying to push you into something that I'm not entirely sure of myself with these comments. Addenda: wow, I...honestly had no idea. Darn, I'll have to go do some reading, now... – David says reinstate Monica Dec 25 '12 at 09:03
  • 1
    @David: No worries :) Even if you are not sure, it's good that you mentioned it. I will probably add just some simple code snippets to show the usage. Thanks for your input! – Felix Kling Dec 25 '12 at 09:05
  • @FelixKling: I realize this is set up as a wiki, but before I go mucking up a good resource, what's you take on mentioning the quirks involved with jQuery Mobile: http://jquerymobile.com/demos/1.2.0/docs/api/globalconfig.html – Tieson T. Dec 26 '12 at 06:12
  • The "does not exist" phrasing fails to account for elements that exist but are not in the document's node tree, having been either detached or not inserted in the first place. – Nathan Bubna Dec 06 '13 at 00:29
  • @Nathan: Well, from a DOM methods point of view, elements that are not in the document don't exist. – Felix Kling Dec 06 '13 at 01:53
  • @FelixKling document.createDocumentFragment() and node.removeChild(child) do not return "nonexistent" DOM nodes, and don't forget shadow DOM nodes, which *are* in the document, just not the query-able node tree. Point of view does not change the meaning of "exist", and such detached/shadow nodes are increasingly encountered as part of modern JavaScript development. There's no reason to stick with this outdated and potentially confusing phrasing. – Nathan Bubna Dec 06 '13 at 17:21
  • Case two led to a *devastating* debugging session back when I first started doing JavaScript. – Jordan Gray Jan 09 '14 at 16:34
  • The only thing I see missing is an expansion of: "Because the elements you are looking for do not exist." If you are manipulating the DOM after the page has loaded, then you need to make sure the new DOM elements you added are ready too. An example of this is the jQuery `.load()` method, which allows you to supply a "complete" callback function - providing utility similar to `.ready()`. – daalbert Jan 09 '14 at 19:38
  • @daalbert: Do you think changing it to "do not exist in the document" would be better? – Felix Kling Jan 09 '14 at 20:31
  • @FelixKling, I suppose I would say something like: "the elements you are looking for do not exist in the DOM when you try to access them." My point being that when you are dynamically loading content, it is possible some things won't be accessible (yet) when the `window.onload` event is fired. – daalbert Jan 09 '14 at 21:02
16

Reasons why id based selectors don't work

  1. The element/DOM with id specified doesn't exist yet.
  2. The element exists, but it is not registered in DOM [in case of HTML nodes appended dynamically from Ajax responses].
  3. More than one element with the same id is present which is causing a conflict.

Solutions

  1. Try to access the element after its declaration or alternatively use stuff like $(document).ready();

  2. For elements coming from Ajax responses, use the .bind() method of jQuery. Older versions of jQuery had .live() for the same.

  3. Use tools [for example, webdeveloper plugin for browsers] to find duplicate ids and remove them.

Peter Mortensen
  • 28,342
  • 21
  • 95
  • 123
sumit
  • 1,545
  • 14
  • 19
16

If the element you are trying to access is inside an iframe and you try to access it outside the context of the iframe this will also cause it to fail.

If you want to get an element in an iframe you can find out how here.

Community
  • 1
  • 1
George Mulligan
  • 11,530
  • 6
  • 33
  • 49
14

As @FelixKling pointed out, the most likely scenario is that the nodes you are looking for do not exist (yet).

However, modern development practices can often manipulate document elements outside of the document tree either with DocumentFragments or simply detaching/reattaching current elements directly. Such techniques may be used as part of JavaScript templating or to avoid excessive repaint/reflow operations while the elements in question are being heavily altered.

Similarly, the new "Shadow DOM" functionality being rolled out across modern browsers allows elements to be part of the document, but not query-able by document.getElementById and all of its sibling methods (querySelector, etc.). This is done to encapsulate functionality and specifically hide it.

Again, though, it is most likely that the element you are looking for simply is not (yet) in the document, and you should do as Felix suggests. However, you should also be aware that that is increasingly not the only reason that an element might be unfindable (either temporarily or permanently).

Nathan Bubna
  • 6,540
  • 2
  • 33
  • 36
11

If script execution order is not the issue, another possible cause of the problem is that the element is not being selected properly:

  • getElementById requires the passed string to be the ID verbatim, and nothing else. If you prefix the passed string with a #, and the ID does not start with a #, nothing will be selected:

      <div id="foo"></div>
    
      // Error, selected element will be null:
      document.getElementById('#foo')
      // Fix:
      document.getElementById('foo')
    
  • Similarly, for getElementsByClassName, don't prefix the passed string with a .:

      <div class="bar"></div>
    
      // Error, selected element will be undefined:
      document.getElementsByClassName('.bar')[0]
      // Fix:
      document.getElementsByClassName('bar')[0]
    
  • With querySelector, querySelectorAll, and jQuery, to match an element with a particular class name, put a . directly before the class. Similarly, to match an element with a particular ID, put a # directly before the ID:

      <div class="baz"></div>
    
      // Error, selected element will be null:
      document.querySelector('baz')
      $('baz')
      // Fix:
      document.querySelector('.baz')
      $('.baz')
    

    The rules here are, in most cases, identical to those for CSS selectors, and can be seen in detail here.

  • To match an element which has two or more attributes (like two class names, or a class name and a data- attribute), put the selectors for each attribute next to each other in the selector string, without a space separating them (because a space indicates the descendant selector). For example, to select:

      <div class="foo bar"></div>
    

    use the query string .foo.bar. To select

      <div class="foo" data-bar="someData"></div>
    

    use the query string .foo[data-bar="someData"]. To select the <span> below:

      <div class="parent">
        <span data-username="bob"></span>
      </div>
    

    use div.parent > span[data-username="bob"].

  • Capitalization and spelling does matter for all of the above. If the capitalization is different, or the spelling is different, the element will not be selected:

      <div class="result"></div>
    
      // Error, selected element will be null:
      document.querySelector('.results')
      $('.Result')
      // Fix:
      document.querySelector('.result')
      $('.result')
    
  • You also need to make sure the methods have the proper capitalization and spelling. Use one of:

    $(selector)
    document.querySelector
    document.querySelectorAll
    document.getElementsByClassName
    document.getElementsByTagName
    document.getElementById
    

    Any other spelling or capitalization will not work. For example, document.getElementByClassName will throw an error.

  • Make sure you pass a string to these selector methods. If you pass something that isn't a string to querySelector, getElementById, etc, it almost certainly won't work.

  • If the HTML attributes on elements you want to select are surrounded by quotes, they must be plain straight quotes (either single or double); curly quotes like or will not work if you're trying to select by ID, class, or attribute.

CertainPerformance
  • 260,466
  • 31
  • 181
  • 209