NOVINKA: Kurz kybernetické bezpečnosti s akreditací MŠMT, nyní již od 0 Kč. Staň se žádaným profesionálem. Zjisti více:
Avatar
filipfr
Člen
Avatar
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}

a

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.sestavOb­jektModulu

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
Avatar
Patrik Valkovič
Člen
Avatar
Odpovídá na filipfr
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.
Avatar
filipfr
Člen
Avatar
Odpovídá na Patrik Valkovič
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
Avatar

Člen
Avatar
Odpovídá na filipfr
: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
 
Nahoru Odpovědět
22.11.2018 23:51
Avatar
Peter Mlich
Člen
Avatar
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
Avatar
filipfr
Člen
Avatar
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
Avatar
Peter Mlich
Člen
Avatar
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
 
Nahoru Odpovědět
23.11.2018 13:05
Avatar

Člen
Avatar
Odpovídá na filipfr
: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
Avatar
Cobis
Člen
Avatar
Odpovídá na filipfr
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
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.