239

I wanted to call a function defined in a first.js file in second.js file. Both files are defined in an HTML file like:

<script type="text/javascript" src="first.js"></script>
<script type="text/javascript" src="second.js"></script>

I want to call fn1() defined in first.js in second.js. From my searches answers were if first.js is defined first it is possible, but from my tests I haven't found any way to do that.

Here is my code:

second.js

document.getElementById("btn").onclick = function() {
    fn1();
}

first.js

function fn1() {
    alert("external fn clicked");
}
13
  • 11
    So call it, there is nothing special as long as it is in global scope. Sep 21, 2014 at 19:22
  • 2
    It should work - stackoverflow.com/questions/3809862/… Sep 21, 2014 at 19:22
  • 2
    That should work. Please edit your question to tell us what happens when you run this using the JavaScript console in Chrome or Firefox/Firebug. (Um, and check that you spelled the function name correctly.)
    – Bob Brown
    Sep 21, 2014 at 19:22
  • 2
    If it doesn't work, I guess you are doing something wrong. Given the information in your question, it's impossible want it is though. Sep 21, 2014 at 19:31
  • 2
    Should the actual code. Sep 21, 2014 at 19:47

17 Answers 17

236

A function cannot be called unless it was defined in the same file or one loaded before the attempt to call it.

A function cannot be called unless it is in the same or greater scope then the one trying to call it.

You declare function fn1 in first.js, and then in second you can just have fn1();

1.js:

function fn1 () {
    alert();
}

2.js:

fn1();

index.html :

<script type="text/javascript" src="1.js"></script>
<script type="text/javascript" src="2.js"></script>
11
  • 7
    So what's the best way to manage all this in a production application? Jul 31, 2016 at 12:03
  • 47
    this doesn't work for me. Gives an error that fn1() is not defined
    – Aayushi
    Jun 3, 2017 at 11:20
  • 14
    This doesn't work for me as well. Is there any work around except using jquery getscript as its suggest a bad practice? Jun 10, 2017 at 11:42
  • 33
    Why is this the accepted answer with 82 votes if it doesn't really work?
    – Daniel
    Feb 26, 2018 at 22:45
  • 43
    If this is not working for you, chances are you defined the function within something else. For instance, I had my function within a $(document).ready(function() {...}) block, and it did not work. I moved it and it worked like a charm. Jul 24, 2018 at 13:07
55

You could consider using the es6 import export syntax. In file 1;

export function f1() {...}

And then in file 2;

import { f1 } from "./file1.js";
f1();

Please note that this only works if you're using <script src="./file2.js" type="module">

You will not need two script tags if you do it this way. You simply need the main script, and you can import all your other stuff there.

3
  • 4
    With Safari, I get cross-origin script load denied by cross-origin resource sharing policy
    – kakyo
    Apr 23, 2020 at 9:54
  • 1
    With older browsers you may not be able to use the es6 import syntax. You could use Babel if it makes it easier for you.
    – tm2josep
    Jun 18, 2020 at 18:14
  • How to (export and) import the whole content of the file1.js ?
    – payne
    Oct 21, 2020 at 16:45
34

1st JS:

function fn(){
   alert("Hello! Uncle Namaste...Chalo Kaaam ki Baat p Aate h...");
}

2nd JS:

$.getscript("url or name of 1st Js File",function(){
fn();
});
5
  • 7
    I think you mean jQuery.getScript() Apr 16, 2018 at 14:53
  • 10
    @Pmpr, the dollar sign is a shorthand for jQuery. Sep 14, 2020 at 10:39
  • 1
    why am I getting an error for the same in the typescript that fn is unknown?
    – Fakipo
    Jun 22, 2021 at 9:52
  • Doesn't work. Unexpected token error
    – parsecer
    Apr 15, 2022 at 4:00
  • 1
    Obviously, this would work only if jQuery is already loaded in the page.
    – Asaf M
    Apr 17, 2022 at 11:19
26

You can make the function a global variable in first.js and have a look at closure and do not put it in document.ready put it outside

you can use ajax too

    $.ajax({
      url: "url to script",
      dataType: "script",
      success: success
    });

same way you can use jquery getScript

$.getScript( "ajax/test.js" )
  .done(function( script, textStatus ) {
    console.log( textStatus );
  })
  .fail(function( jqxhr, settings, exception ) {
    $( "div.log" ).text( "Triggered ajaxError handler." );
});
3
  • 9
    Beginners never try doing anything like this. These are bad practices to be avoided. As long as both are referenced by the web page, yes. You simply call the functions as if they are in the same JS file.
    – Ananda
    May 18, 2016 at 16:30
  • 1
    The only issue is, it doesn't include the header for some reason.
    – Si8
    Mar 31, 2017 at 14:08
  • Cannot use import statement outside a module
    – Alfa Rojo
    Feb 23, 2022 at 17:41
19

declare function in global scope with window

first.js

window.fn1 = function fn1() {
    alert("external fn clicked");
}

second.js

document.getElementById("btn").onclick = function() {
   fn1();
}

include like this

<script type="text/javascript" src="first.js"></script>
<script type="text/javascript" src="second.js"></script>
1
  • Is this recommended? What If I have 1000 functions. What I'm trying to do is to have a single JavaScript file to hold all event listeners. But to make that work I would have to declare each function in global scope as you suggest. Wouldn't this slow things down? Jun 11, 2021 at 13:16
10

use "var" while creating a function, then you can access that from another file. make sure both files are well connected to your project and can access each other.

file_1.js

var firstLetterUppercase = function(str) {
   str = str.toLowerCase().replace(/\b[a-z]/g, function(letter) {
      return letter.toUpperCase();
   });
   return str;
}

accessing this function/variable from file_2.js file

firstLetterUppercase("gobinda");

output => Gobinda

7

It should work like this:

1.js

function fn1() {
  document.getElementById("result").innerHTML += "fn1 gets called";
}

2.js

function clickedTheButton() {
  fn1();
} 

index.html

<html>
  <head>
  </head>
  <body>
    <button onclick="clickedTheButton()">Click me</button>
    <script type="text/javascript" src="1.js"></script>
    <script type="text/javascript" src="2.js"></script>
  </body>
 </html>

output

Output. Button + Result

Try this CodePen snippet: link .

7

Please note this only works if the

<script>

tags are in the body and NOT in the head.

So

<head>
...
<script type="text/javascript" src="first.js"></script>
<script type="text/javascript" src="second.js"></script>
</head>

=> unknown function fn1()

Fails and

<body>
...
<script type="text/javascript" src="first.js"></script>
<script type="text/javascript" src="second.js"></script>
</body>

works.

1
  • OH THAT'S WHY! your answer was really helpful as I couldn't figure out why, although my scripts were in correct order, the functions still were undefined. Why does this happen, though?
    – Artem
    Nov 23, 2022 at 12:00
5

This is actually coming very late, but I thought I should share,

in index.html

<script type="text/javascript" src="1.js"></script>
<script type="text/javascript" src="2.js"></script>

in 1.js

fn1 = function() {
    alert("external fn clicked");
}

in 2.js

fn1()
1
  • Please see the OP's comment under the question: "...I was just experimenting [with] a file located on a wrong location with the same name...".
    – traktor
    Sep 29, 2019 at 5:18
3

Use cache if your server allows it to improve speed.

var extern =(url)=> {           // load extern javascript
    let scr = $.extend({}, {
        dataType: 'script',
        cache: true,
        url: url
    });
    return $.ajax(scr);
}
function ext(file, func) {
    extern(file).done(func);    // calls a function from an extern javascript file
}

And then use it like this:

ext('somefile.js',()=>              
    myFunc(args)
);  

Optionally, make a prototype of it to have it more flexible. So that you don't have to define the file every time, if you call a function or if you want to fetch code from multiple files.

3

first.js

function first() { alert("first"); }

Second.js

var imported = document.createElement("script");
imported.src = "other js/first.js";  //saved in "other js" folder
document.getElementsByTagName("head")[0].appendChild(imported);


function second() { alert("Second");}

index.html

 <HTML>
    <HEAD>
       <SCRIPT SRC="second.js"></SCRIPT>
    </HEAD>
    <BODY>
       <a href="javascript:second()">method in second js</a><br/>
       <a href="javascript:first()">method in firstjs ("included" by the first)</a>
    </BODY>
</HTML>
1
window.onload = function(){
    document.getElementById("btn").onclick = function(){
        fn1();
    }
   // this should work, It calls when all js files loaded, No matter what position you have written
});
1
  • 2
    Although your code snippet might solve the issue, you should describe what’s the purpose of your code (how it solves the problem). Furthermore, you might want to check stackoverflow.com/help/how-to-answer
    – Ahmad F
    Mar 14, 2018 at 10:58
1
// module.js
export function hello() {
  return "Hello";
}

// main.js
import {hello} from 'module'; // or './module'
let val = hello(); // val is "Hello";

reference from https://hype.codes/how-include-js-file-another-js-file

1

My idea is let two JavaScript call function through DOM.

The way to do it is simple ... We just need to define hidden js_ipc html tag. After the callee register click from the hidden js_ipc tag, then The caller can dispatch the click event to trigger callee. And the argument is save in the event that you want to pass.

When we need to use above way ?

Sometime, the two javascript code is very complicated to integrate and so many async code there. And different code use different framework but you still need to have a simple way to integrate them together.

So, in that case, it is not easy to do it.

In my project's implementation, I meet this case and it is very complicated to integrate. And finally I found out that we can let two javascript call each other through DOM.

I demonstrate this way in this git code. you can get it through this way. (Or read it from https://github.com/milochen0418/javascript-ipc-demo)

git clone https://github.com/milochen0418/javascript-ipc-demo
cd javascript-ipc-demo
git checkout 5f75d44530b4145ca2b06105c6aac28b764f066e

Anywhere, Here, I try to explain by the following simple case. I hope that this way can help you to integrate two different javascript code easier than before there is no any JavaScript library to support communication between two javascript file that made by different team.

<html>
<head>
    <link rel="stylesheet" type="text/css" href="css/style.css" />
</head>
<body>
    <div id="js_ipc" style="display:none;"></div>
    <div id="test_btn" class="btn">
        <a><p>click to test</p></a>
    </div>    
</body>
<script src="js/callee.js"></script>
<script src="js/caller.js"></script>
</html>

And the code css/style.css

.btn {
    background-color:grey;
    cursor:pointer;
    display:inline-block;
}

js/caller.js

function caller_add_of_ipc(num1, num2) {
    var e = new Event("click");
    e.arguments = arguments;
    document.getElementById("js_ipc").dispatchEvent(e);
}
document.getElementById("test_btn").addEventListener('click', function(e) {
    console.log("click to invoke caller of IPC");
    caller_add_of_ipc(33, 22);      
});

js/callee.js

document.getElementById("js_ipc").addEventListener('click', (e)=>{
    callee_add_of_ipc(e.arguments);
});    
function callee_add_of_ipc(arguments) {
    let num1 = arguments[0];
    let num2 = arguments[1];
    console.log("This is callee of IPC -- inner-communication process");
    console.log( "num1 + num2 = " + (num1 + num2));
}
1

better late than never

(function (window) {const helper = { fetchApi: function () { return "oke"}
   if (typeof define === 'function' && define.amd) {
    define(function () { return helper; });
   }
   else if (typeof module === 'object' && module.exports) {
    module.exports = helper;
  }
  else {
    window.helper = helper;
  }
}(window))

index html <script src="helper.js"></script> <script src="test.js"></script>

in test.js file helper.fetchApi()

0

I have had same problem. I have had defined functions inside jquery document ready function.

$(document).ready(function() {
   function xyz()
   {
       //some code
   }
});

And this function xyz() I have called in another file. This doesn't working :) You have to defined function above document ready.

1
  • This is downvoted but is the same problem I had. My solution is to take the function out of the $(document).ready() function.
    – Ben
    Nov 26, 2020 at 17:27
-1

TLDR: Load Global Function Files first, Then Load Event Handlers

Whenever you are accessing an element within a JS file or <script> block, it is essential to check to make sure that element exists, i.e., jQuery's $(document).ready() or plain JS's document.addEventListener('DOMContentLoaded', function(event)....

However, the accepted solution does NOT work in the event that you add an event listener for the DOMContentLoaded, which you can easily observe from the comments.

Procedure for Loading Global Function Files First

The solution is as follows:

  • Separate the logic of your JS script files so that each file only contains event listeners or global, independent functions.
  • Load the JS script files with the global, independent functions first.
  • Load the JS script files with event listeners second. Unlike the other previous files, make sure to wrap your code in document.addEventListener('DOMContentLoaded', function(event) {...}). or document.Ready().

Not the answer you're looking for? Browse other questions tagged or ask your own question.