Diskuze: předání callbacku => callback is not a function

filipfr

Člen

filipfr:22.11.2018 13:42

Ahoj, asi trochu plavu ve volání callbacků. mám pole objektů. Každý objekt má property funkce: "názevfce" a nazev: "nějaký string". Chci v cyklu sestavit objekty, každý se sestavuje jinou fcí... funkce(nadpis).

if (!obj.sestaveno) {
            obj.poleFunkci.forEach(polozka => {
               obj.sestavObjektModulu(polozka.funkce, polozka.nazev);
            });
        }

k tomu mám fci

function info(nadpis) {.....return sestavený_objekt}

obj.sestavObjektModulu: function (callback, nadpis) {
        console.log(callback);
        callback(nadpis);
    },

konzole mi vypíše:
info.....to očekávám,
a
pak

Uncaught TypeError: callback is not a function
at Object.sestavObjektModulu

poradíte někdo, co dělám špatně? Už to řeším 2. den.
díky

Odpovědět

22.11.2018 13:42

Patrik Valkovič

Tvůrce

Patrik Valkovič:22.11.2018 15:27

Možná

obj.sestavObjektModulu = function (callback, nadpis) {
}

Nahoru Odpovědět

22.11.2018 15:27

Nikdy neumíme dost na to, abychom se nemohli něco nového naučit.

filipfr

Člen

filipfr:22.11.2018 15:44

Omlouvám se, v tom problém není. Ten obj. v původním kódu není. Je to definováno přímo v tom objektu. Dodal jsem to tam jen aby bylo vidět, že se jedná o metodu objektu obj. Původně je to

var obj = {
  .....................
   sestavObjektModulu: function (callback, nadpis) {
        console.log(callback);
        callback(nadpis);
    },
........................

Nahoru Odpovědět

22.11.2018 15:44

Člen

:22.11.2018 23:51

if (!obj.sestaveno) {
  obj.poleFunkci.forEach(polozka => {
    obj.sestavObjektModulu(
      polozka.funkce, // tu zle voláš funkciu
      polozka.nazev
    )
  })
}

Viac sa k tomu nedá povedať, musel by si sem napísať celú štruktúru objektu obj, tie útržky nestačia, skôr mätú. Cyklíš totiž s forEach property poleFunkci, ale zjavne to nie je pole funkcií ako naznačuje názov, ale pole objektov. Teda aspoň tak ku každej položke následne pristupuješ...

Editováno 22.11.2018 23:53

Nahoru Odpovědět

22.11.2018 23:51

Peter Mlich

Člen

Peter Mlich:23.11.2018 7:58

On ti pise, ze to neni funkce.

Objekty se daji resit temito zpusoby

xxx = {}
xxx.aaa = function () {...}
xxx.bbb = function () {...}
xxx.aaa();
xxx.bbb();
// ---
function yyy()
{
this.aaa = function () {...}
this.bbb = function () {...}
}
xxx = new yyy();
xxx.aaa();
xxx.bbb();

Kdyz tam chces predat parametr, pak

function yyy(eee)
{
this.eee = eee;
}
xxx = new yyy(789);
alert(xxx.eee)
xxx.eee = 123;
alert(xxx.eee)

Kdyz ten objekt chces pouzivat s casovacem, tak takto:

function yyy()
{
var root = this; // root je muj nazev promenne, do ktere ulozim pointer funkce. Ten se tam vytvori pri pouziti new. A pak ten pointer predam do vsech vnitrnich funkci. This, aspon ve starsich prohlizecich, to nebere
this.aaa = function () {...}
this.bbb = function () {root.aaa();}
//this.bbb = function () {this.aaa();} // tohle by slo pouzit, kdybys nemel casovac
}
xxx = new yyy();
xxx.aaa();
xxx.bbb();

Co se tyce callbacku, je to podobne

function yyy()
{
var root = this;
this.aaa = function () {...}
this.bbb = function (zzz) {root[zzz]();}
//this.bbb = function (zzz) {this[zzz]();} // xxx('bbb') // toez jako xxx.bbb()
//this.bbb = function (zzz) {window[zzz]();} // xxx('yyy') // totez jako yyy()
//this.bbb = function (zzz) {zzz();} // xxx(xxx.aaa) // totez jako xxx.aaa()
}
xxx = new yyy();
xxx.aaa();
xxx.bbb();

Podle mne, cey tvuj zapis je hodne divoky. Co se divam na ruzne priklady tady, tak davaji do foreach jeste function.
https://developer.mozilla.org/…rray/forEach

var words = ['one', 'two', 'three', 'four'];
words.forEach(function(word) {
  console.log(word);
  if (word === 'two') {
    words.shift();
  }
});
// one
// two
// four

Foreach je takova novinka, ze to treba podporuje az IE9. A kdo vi, jake kombinace zapisu uz zna. A tez neni na skodu zamyslet se nad prototype. Bez toho take nektere veci, kter bys ocekaval, ze pujdou, nejdou

Nahoru Odpovědět

23.11.2018 7:58

filipfr

Člen

filipfr:23.11.2018 10:16

Já se asi špatně vyjádřil. Prostě dostanu z databáze jméno fce(jako string) a nadpis(taky jako string), resp, pole takových dvojic(neznám jejich počet). Mám připravené funkce, jejichž jména odpovídají těm stringům. Každá fce sestavuje objekt. V podstatě jde o to, aby si každý objekt pamatoval svůj nadpis. A chtěl jsem si je sestavit jednoduše pomocí forEach, jak jsem popsal. Ale prostě mi to nefungovalo. A já nechápu proč. Samozřejmě, že jsem to nějak obešel, ale ten kód je mnohem delší a nepřehlednější.

Nahoru Odpovědět

23.11.2018 10:16

Peter Mlich

Člen

Peter Mlich:23.11.2018 13:05

Co treba nejakou ukazku kodu, co z te db dostanes a ukazku, co chces docilit?
Cili, treba neco jako

data = [
['aaa', 'bbb'],
['ccc', 'bbb']
]
obj[0].funkce = 'aaa'
obj[0].nazev = 'bbb'
 // nebo mozna chces toto ...
function aaa(str) {alert(str);}

obj[0].funkce = window['aaa']
obj[0].funkce('xyz') // to by melo spustit aaa, cili alert(str)

Editováno 23.11.2018 13:05

Nahoru Odpovědět

23.11.2018 13:05

Člen

:23.11.2018 13:18

Ak tomu dobre rozumiem, mal by si tie funkcie teda volať takto:

if (!obj.sestaveno) {
  obj.poleFunkci.forEach(polozka => {
    obj.sestavObjektModulu(
      obj[polozka.funkce],
      polozka.nazev
    )
  })
}

Nahoru Odpovědět

23.11.2018 13:18

Cobis

Člen

Cobis:23.11.2018 13:34

Snad takto:

array.forEach(function(currentValue, index, array), thisValue)

currentValue by mela bát položka

if (!obj.sestaveno) {
            obj.poleFunkci.forEach((currentValue, index, array) => {
               obj.sestavObjektModulu(currentValue);
            });
        }

obj.sestavObjektModulu: function (polozka) {
        console.log(polozka.callback);
        callback(polozka.nadpis);
    }

Nahoru Odpovědět

23.11.2018 13:34

Naučíme tě pracovat na home-office.

Zjistit více...

Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 9 zpráv z 9.