819

My code is

function getID( swfID ){
     if(navigator.appName.indexOf("Microsoft") != -1){
          me = window[swfID];
     }else{
          me = document[swfID];
     }
}

function js_to_as( str ){
     me.onChange(str);
}

However, sometimes my onChange does not load. Firebug errors with

me.onChange is not a function

I want to degrade gracefully because this is not the most important feature in my program. typeof gives the same error.

Any suggestions on how to make sure that it exists and then only execute onChange?

(None of the methods below except try catch one work)

1

34 Answers 34

1632

Try something like this:

if (typeof me.onChange !== "undefined") { 
    // safe to use the function
}

or better yet (as per UpTheCreek upvoted comment)

if (typeof me.onChange === "function") { 
    // safe to use the function
}
Sign up to request clarification or add additional context in comments.

7 Comments

@UpTheCreek, it would be a bit dangerous as a general solution since older versions of IE treat certain functions as objects, e.g. typeof window.alert === 'object'.
Why not just use if (me.onChange) { // do something } ?
@BornToCode because then me.onChange could be anything evaluating to true, not necessarily a function (e.g. it could be a boolean, a string, etc). For example see jsfiddle.net/j5KAF/1
@UpTheCreek and @Noyo: I missed your comments and ran into the issue. I just turned the test into: ((typeof me.onChange === "function") || (typeof me.onChange === "object")) Works fine down to ie8 in my case.
Just to point out the obvious typeof me.onChange === "function" fails if that global me is undefined.
|
206

Modern JavaScript / TypeScript

me.onChange?.(str)

Use Optional Chaining - available from JavaScript ES2020 and TypeScript 3.7

  • if me.onChange is a function, it is called.

  • if me.onChange is undefined or null, nothing happens.

  • if me.onChange is defined but is not a function, a TypeError is thrown.

5 Comments

This can't check if a bare function exists though. If me is undefined, this will throw.
@duhaime me being undefined is a separate concern (the function just happens to be on a me object in the question) - but, if you did need to check first if me is defined you can also do that with optional chaining: me?.onChange?.(str)
If you run me?.onChange?.(str) in your dev console it'll bomb. Though if you know the global context that can work window?.$me?.onChange?.(str)
Ah, interesting one! Yeah I guess that'd be a ReferenceError if me is not even declared (as opposed to declared but undefined). Looking at the question that's indeed the case there if that's the entire code, I was assuming it was just a snippet with me implied to be declared somewhere else. Nice catch!
A cautionary tale about using this: blog.jim-nielsen.com/2022/a-web-for-all
170

I had this problem. if (obj && typeof obj === 'function') { ... } kept throwing a reference error if obj happened to be undefined, so in the end I did the following:

if (typeof obj !== 'undefined' && typeof obj === 'function') { ... }

However, a colleague pointed out to me that checking if it's !== 'undefined' and then === 'function' is redundant, thus:

Simpler:

if (typeof obj === 'function') { ... }

Much cleaner and works great.

3 Comments

Anyone has idea why the first code snippet throws ReferenceError? That seems illogical to me.
@saidfagan See the definition of ReferenceError developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/…
Any motive why typeof obj == 'function' would be insufficient? ;-) Reminder: strict equality test operator only makes sense when the static operand is empty or null or 0 or subject to any cast amalgame like this.
29

How about:

if('functionName' in Obj){
    //code
}

e.g.

'appendChild' in document // returns true
// or
var color1 = new String("green");
"length" in color1 // returns true
"indexOf" in color1 // returns true
"blablabla" in color1 // returns false

or as for your case:

if('onChange' in me){
    //code
}

See MDN docs.

3 Comments

but this solution is not working everytime :( for example if I want to know it there is lastIndexOf() in string (we know, that is, but theoretically) typeof String().lastIndexOf === "function" it's true, but 'lastIndexOf' in String is false.
'lastIndexOf' in new String returns true
new String() is not a string - it is string object. For example, the same is true about boolean object, new Boolean(false) is not boolean primitive. Expression: new Boolean(false)?'yes':'no' evaluates to yes
22

If you're using eval to convert a string to function, and you want to check if this eval'd method exists, you'll want to use typeof and your function string inside an eval:

var functionString = "nonexsitantFunction"
eval("typeof " + functionString) // returns "undefined" or "function"

Don't reverse this and try a typeof on eval. If you do a ReferenceError will be thrown:

var functionString = "nonexsitantFunction"
typeof(eval(functionString)) // returns ReferenceError: [function] is not defined

1 Comment

You can do this without eval. Example: var a = 'alert'; window[a]('it works');
15

Try typeof -- Look for 'undefined' to say it doesn't exist, 'function' for a function. JSFiddle for this code

function thisishere() {
    return false;
}
alert("thisishere() is a " + typeof thisishere);
alert("thisisnthere() is " + typeof thisisnthere);

Or as an if:

if (typeof thisishere === 'function') {
    // function exists
}

Or with a return value, on a single line:

var exists = (typeof thisishere === 'function') ? "Value if true" : "Value if false";
var exists = (typeof thisishere === 'function') // Returns true or false

Comments

14

Didn't see this suggested: me.onChange && me.onChange(str);

Basically if me.onChange is undefined (which it will be if it hasn't been initiated) then it won't execute the latter part. If me.onChange is a function, it will execute me.onChange(str).

You can even go further and do:

me && me.onChange && me.onChange(str);

in case me is async as well.

Comments

13

Put double exclamation mark i.e !! before the function name that you want to check. If it exists, it will return true.

function abc(){
}
!!window.abc; // return true
!!window.abcd; // return false

1 Comment

simple clean solution
12

For me the easiest way :

function func_exists(fname)
{
  return (typeof window[fname] === 'function');
}

1 Comment

this is the correct answer, because it's a string litteral .. I would say it better like this if ((typeof window["function_name"] !== 'undefined') && ((typeof window["function_name"] === 'function'))) { return true; } else { return false; } but it's a minor adjustment
9
function function_exists(function_name)
{
    return eval('typeof ' + function_name) === 'function';
}
alert(function_exists('test'));
alert(function_exists('function_exists'));

OR

function function_exists(func_name) {
  //  discuss at: http://phpjs.org/functions/function_exists/
  // original by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
  // improved by: Steve Clay
  // improved by: Legaev Andrey
  // improved by: Brett Zamir (http://brett-zamir.me)
  //   example 1: function_exists('isFinite');
  //   returns 1: true

  if (typeof func_name === 'string') {
    func_name = this.window[func_name];
  }
  return typeof func_name === 'function';
}

2 Comments

Please, some explanation.
In two cases, simply call function_exists with the name of function parameter to check existence, returns bool.
7
//Simple function that will tell if the function is defined or not
function is_function(func) {
    return typeof window[func] !== 'undefined' && $.isFunction(window[func]);
}

//usage

if (is_function("myFunction") {
        alert("myFunction defined");
    } else {
        alert("myFunction not defined");
    }

3 Comments

your solution is not clear, as it containt jQUery, which is odd at this moment i think, so it throws errors when simple testing.
@tazotodua it is self explanatory function. if you want to use without jquery. just remove the && part. that is an extra condition to check and to avoid errors.
Does the condition at the left side of the && take care of something that the jQuery's isFunction at the right side of the && doesn't ?
6
function js_to_as( str ){
     if (me && me.onChange)
         me.onChange(str);
}

Comments

6

I'll go 1 step further to make sure the property is indeed a function

function js_to_as( str ){
     if (me && me.onChange && typeof me.onChange === 'function') {
         me.onChange(str);
     }
}

Comments

6

I had the case where the name of the function varied according to a variable (var 'x' in this case) added to the functions name. This works:

if ( typeof window['afunction_'+x] === 'function' ) { window['afunction_'+x](); } 

Comments

6

Here is a working and simple solution for checking existence of a function and triggering that function dynamically by another function;

Trigger function

function runDynamicFunction(functionname){ 

    if (typeof window[functionname] == "function") { //check availability

        window[functionname]("this is from the function it"); // run function and pass a parameter to it
    }
}

and you can now generate the function dynamically maybe using php like this

function runThis_func(my_Parameter){

    alert(my_Parameter +" triggerd");
}

now you can call the function using dynamically generated event

<?php

$name_frm_somware ="runThis_func";

echo "<input type='button' value='Button' onclick='runDynamicFunction(\"".$name_frm_somware."\");'>";

?>

the exact HTML code you need is

<input type="button" value="Button" onclick="runDynamicFunction('runThis_func');">

Comments

6

In a few words: catch the exception.

I am really surprised nobody answered or commented about Exception Catch on this post yet.

Detail: Here goes an example where I try to match a function which is prefixed by mask_ and suffixed by the form field "name". When JavaScript does not find the function, it should throw an ReferenceError which you can handle as you wish on the catch section.

function inputMask(input) {
  try {
    let maskedInput = eval("mask_"+input.name);

    if(typeof maskedInput === "undefined")
        return input.value;
    else
        return eval("mask_"+input.name)(input);

  } catch(e) {
    if (e instanceof ReferenceError) {
      return input.value;
    }
  }
}

Comments

5

I like using this method:

function isFunction(functionToCheck) {
  var getType = {};
  return functionToCheck && getType.toString.call(functionToCheck) === '[object Function]';
}

Usage:

if ( isFunction(me.onChange) ) {
    me.onChange(str); // call the function with params
}

Comments

5

The Underscore.js library defines it in the isFunction method as this (which comments suggest may cater for some browser bugs)

typeof obj == 'function' || false

http://underscorejs.org/docs/underscore.html#section-143

Comments

5

If you're checking for a function that is a jQuery plugin, you need to use $.fn.myfunction

if (typeof $.fn.mask === 'function') {
    $('.zip').mask('00000');
}

Comments

4

With no conditions

me.onChange=function(){};

function getID( swfID ){
     if(navigator.appName.indexOf("Microsoft") != -1){
          me = window[swfID];
     }else{
          me = document[swfID];
     }
}

function js_to_as( str ){
     me.onChange(str);
}

Comments

4

I would suspect that me is not getting correctly assigned onload.

Moving the get_ID call into the onclick event should take care of it.

Obviously you can further trap as previously mentioned:

function js_to_as( str) {
  var me = get_ID('jsExample');
  if (me && me.onChange) {
    me.onChange(str);
  }
}

Comments

4

I always check like this:

if(!myFunction){return false;}

just place it before any code that uses this function

Comments

4

This simple jQuery code should do the trick:

if (jQuery.isFunction(functionName)) {
    functionName();
}

Comments

4

I have tried the accepted answer; however:

console.log(typeof me.onChange);

returns 'undefined'. I've noticed that the specification states an event called 'onchange' instead of 'onChange' (notice the camelCase).

Changing the original accepted answer to the following worked for me:

if (typeof me.onchange === "function") { 
  // safe to use the function
}

Comments

4

I would suggest using:

function hasMethod(subject, methodName) {
  return subject != null && typeof subject[methodName] == "function";
}

The first check subject != null filters out nullish values (null and undefined) which don't have any properties. Without this check subject[methodName] could throw an error:

TypeError: (undefined|null) has no properties

Checking for only a truthy value isn't enough, since 0 and "" are both falsy but do have properties.

After validating that subject is not nullish you can safely access the property and check if it matches typeof subject[methodName] == "function".


Applying this to your code you can now do:

if (hasMethod(me, "onChange")) {
  me.onChange(str);
}

Comments

3

I have also been looking for an elegant solution to this problem. After much reflection, I found this approach best.

const func = me.onChange || (str => {}); func(str);

2 Comments

If you're going for minimalizm, you could also do this: (me.onChange||(_=>_))() Will execute the function if it exists, otherwise will do nothing.
It is better me.onChange && me.onChange() to prevent define a void function.
2

To illustrate the preceding answers, here a quick JSFiddle snippet :

function test () {
console.log()

}

console.log(typeof test) // >> "function"

// implicit test, in javascript if an entity exist it returns implcitly true unless the element value is false as :
// var test = false
if(test){ console.log(true)}
else{console.log(false)}

// test by the typeof method
if( typeof test === "function"){ console.log(true)}
else{console.log(false)}


// confirm that the test is effective : 
// - entity with false value
var test2 = false
if(test2){ console.log(true)}
else{console.log(false)}

// confirm that the test is effective :
// - typeof entity
if( typeof test ==="foo"){ console.log(true)}
else{console.log(false)}

/* Expected :
function
true 
true 
false
false
*/

Comments

2

I prefer it using lodash library as below (looks much cleaner):

if (_.has(me, "onChange")) {
   // your desired code here
}

// or generic one like

if (_.has(this, "some property or function name")) {
   // your desired code here
}

Comments

1
    function sum(nb1,nb2){

       return nb1+nb2;
    }

    try{

      if(sum() != undefined){/*test if the function is defined before call it*/

        sum(3,5);               /*once the function is exist you can call it */

      }

    }catch(e){

      console.log("function not defined");/*the function is not defined or does not exists*/
    }

Comments

1

And then there is this...

( document.exitPointerLock || Function )();

4 Comments

hello Master, good one, but what do you think about catching the Exception? check my answer
Hi thanks for the note. Your solution is admirable. I try to avoid the 'try-catch' paradigm whenever possible or easier, or maybe it's been implied/imposed by a 3rd party aid. 'Eval' is also frowned upon for various reasons one being like try-catch another wrapper so time delays. There are alternatives to eval that might 'throw' too. I think 'new Function(parms, body)' would maybe throw you an error, but I guess the next question for me is speed comparisons, other then just best code size reduction (implies faster to me). [Hmm this looks new. I've not tried it jsben.ch]
mmm rly interesting point about calling new Function instead. feel free to submit your changes suggestions to my answer. thanks alot
Ya I mean even in bash using 'source myScript.sh' will let you 'export' via aliases etc. it's like classes and libraries have always been an inherent property of any and all code. You should have a js interpreter check code if it's not been sanctioned for sure. I built one in a single function, which is another fun challenge to help understand the simplest axiomatic nature of code, and even a run-time is not beyond anyone's ability.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.