[js] functie een variabele functie laten aanroepen

[xilent_xage]

Hoi,

Ik probeer een functie te maken die aan de hand van een variabele bepaalt welke functie er wordt uitgevoerd. Even wat code:

code:

function functieaanroeper ( action ) {

        if ( action && typeof action == 'function' ) { action; }
        else if ( action ) { alert ( action + " not a function!"); }

}

functieaanroeper(alert("test1));

functieaanroeper(function() { alert("test2"); });

De eerste test wordt ge-alert, de tweede niet. Maar ik zou toch wel graag willen dat ie dat doet

Iemand een idee?

[Verwijderd]

Je hebt het helemaal verkeerd begrepen.

De eerste functie wordt aangeroepen vóórdat de functie functieaanroeper wordt aangeroepen. De alert vindt plaats, de functie returnt void (ofwel undefined). De functie functieaanroeper krijgt als argument undefined mee, de if ( action && ... is dus niet waar, en else if ( action ) { ... ook niet. In het eerste geval doet de functie dus niets.

In het tweede geval wordt er wel degelijk een functie aan de functie functieaanroeper doorgegeven. De eerste if evalueert naar true en daarna wordt uitgevoerd: action;. Dat is een pointer naar een functie die je op de stack stopt, maar waar je verder niets mee doet. Er gebeurt dus niets. Als je action (); ervan maakt wordt de functie geëvalueerd en wordt "test2" ge-alert.

Maar denk er dus om: deze functie doet bij je eerste aanroep absoluut niet wat jij verwacht!

[dominic]

Just a thought: Je zou ook eens naar de eval() functie kunnen kijken, deze accepteert een string waarde waar javascript in mag staan.

[Borizz]

dominic schreef op maandag 21 juli 2008 @ 19:17:
Just a thought: Je zou ook eens naar de eval() functie kunnen kijken, deze accepteert een string waarde waar javascript in mag staan.

Waarom wordt dit altijd standaard aangedragen, zeker in JavaScript zijn er genoeg (betere!) alternatieven. Zie ook Don't be eval(), waar geen eval te gebruiken.

[funkwurm]

Ik doe meestal zoiets:

JavaScript:

function doe(dit, arg) {
  window[dit](arg);
}
// ...
doe('alert', 'Hello World!');

Is wat minder evil dan eval().

[Bosmonster]

funkwurm schreef op dinsdag 22 juli 2008 @ 06:48:
Ik doe meestal zoiets:

JavaScript:

function doe(dit, arg) {
  window[dit](arg);
}
// ...
doe('alert', 'Hello World!');

Is wat minder evil dan eval().

Dan geef ik toch de voorkeur aan functies, die zijn een stuk wijder inzetbaar (en behoeven bijvoorbeeld geen globale definitie).

JavaScript:

function doe (f, a) {
    f(a);
}

doe (alert, 'test');

doe (function(t){alert(t)}, 'test2');

[crisp]

En geschikt voor meerdere argumenten ala PHP's call_user_func:

JavaScript:

function call_user_func(f)
{
    func = typeof f == 'string' ? window[f] : f;

    if (typeof func == 'function')
    {
        return func.apply(null, [].slice.call(arguments, 1));
    }

    alert((f && f.toString ? f.toString() : f) + ' is not a function');

    return false;
}

function foo(a, b)
{
    alert('arguments: a=' + a + ', b=' + b);
}

call_user_func('foo', 'aap', 'noot');
call_user_func(foo, 'mus', 'pim');
call_user_func('bar', 'roos', 'vuur');

[Bosmonster]

JavaScript:

[].slice.call(arguments, 1)

Nice, die kende ik nog niet. Scheelt een hoop 'geclone'.

[ Voor 15% gewijzigd door Bosmonster op 22-07-2008 11:52 ]

[crisp]

Bosmonster schreef op dinsdag 22 juli 2008 @ 11:51:

JavaScript:

[].slice.call(arguments, 1)

Nice, die kende ik nog niet. Scheelt een hoop 'geclone'.

http://crisp.tweakblogs.n...owser-array-generics.html

note btw dat je niet zomaar elke array method op 'array-like' objecten kan loslaten. Het werkt toevallig crossbrowser voor het arguments object (wat eigenlijk van meet af aan al een array had moeten zijn imo), maar met name voor interface objecten (NodeLists e.d.) ymmv (needless to say dat vooral IE daar dwarsligt).

[Bosmonster]

crisp schreef op dinsdag 22 juli 2008 @ 11:58:
[...]

http://crisp.tweakblogs.n...owser-array-generics.html

note btw dat je niet zomaar elke array method op 'array-like' objecten kan loslaten. Het werkt toevallig crossbrowser voor het arguments object (wat eigenlijk van meet af aan al een array had moeten zijn imo), maar met name voor interface objecten (NodeLists e.d.) ymmv (needless to say dat vooral IE daar dwarsligt).

Bedoel je dat ik vaker je blog moet gaan lezen?

Heb hem zelfs gebookmarked nu!

/offtopic