createElement

Question

Mam taki kod: 

core.e = function(tag, param1, param2) {
  const element = document.createElement(tag);

  const setAttrs = attrs => {
    Object.keys(attrs).forEach(attr => {
      element.setAttribute(attr, attrs[attr]);
    });
  };

  const setChilds = childs =>{
    if(typeof childs === 'string') childs = [new Text(childs)];

    childs.forEach(child => {
      element.appendChild(child);
    });
  };

  if(arguments.length === 2) {
    setChilds(param1);
  } else {
    setAttrs(param1);
    setChilds(param2);
  }

  return element;
};

Coś jest źle? 

Comments

Nie ma takiego słowa jak childs -> one child, two children

---

No tak, to jest błąd .

---

A co jeszcze jest źle (bo uwierzę, że wszystko poza sprawami językowymi zrobiłem dobrze )

---

core.e = function(tag, param1, param2) {

Dlaczego to nie jest arrow function?

---

A co ty zagadki zadajesz? Nie napisałeś co ten kod ma robić i jakie komunikaty błedów dostajesz, tudzież jaki jest spodziewany wynik a jaki faktyczny, dla jakich inputów nie działa zgodnie z zamierzeniami. Próbowałeś to debuggować (chociażby prymitywnym console.logiem)? Cy może działa, a pytasz co można zrefaktorować?

---

@Krzycho92 Pewnie dlatego, że arrow nie ma pseudotablicy arguments (co jest git, nie jest do niczego potrzebna).

---

@kap pseudo-tablicy arguments można się pozbyć i wtedy ten fragment:

if(arguments.length === 2) {

zamienić na:

if ( tag && param1 ) {

---

@Krzycho92 Wiem że się można pozbyć, napisałem przecież "nie jest do niczego potrzebna".
Btw - te dwie wersje to nie są odpowiedniki - sprawdzasz tam zupełnie co innego - czy dwa pierwsze parametry są truthy.

---

Cy może działa, a pytasz co można zrefaktorować?

 Debugowałem i wsztstko działa, chyba chodziło mi o to refaktorowanie...

Arrow function nie użyłem bo nie ma tablicy arguments

Answer 1

const element = document.createElement(tag);

tag w ' '

Comments

? Nie do końca zrozumiałem.

---

cudzysłowach

---

Dalej do końca nie rozumiem. Tag ma być stringiem, przykład:

$.core('h1', {style: 'background-color: red;'}, 'SomeH1');