OO bgColorChanger - next try: alles in einem Objekt - kl. opt.

[jobo]

Hallo,

Man kann viel, einen wirklichen Grund dafür gibt es nicht. Diesen Code finde ich ziemlich unlesbar und unverständlich. Eine klassische OOP-Lösung, die mit tatsächlichen Objekten arbeitet, würde ich hier vorziehen.

[code lang=javascript]function createColorHandler(elem, prop, red, redRange, green, greenRange, blue, blueRange) {

createColorHandler ist ein ziemlich unpassender Name. Zurückgegeben wird ja eine Step-Funktion für eine Farb-Animation. Wieso nicht gleich die Animation als ganze wegkapseln?

Ist das jetzt ernst gemeint?

function createColorObj(v, r) {
      var value = v;
      var max = v;
      var range = r;
      var dir = 1;
      return function() {
        if (value <= max - range || value >= max) {
          dir *= -1;
        }
        value += dir;
        return value;
      };
    }

Hier wird kein Objekt erzeugt, sondern eine Closure. Kein Problem an sich, aber createColorObj ist kein passender Name, eher createNextColorFunc oder so.

Das habe ich schon mal geändert bei mir.

Vielleicht sollte jobo erst einmal versuchen, einfachen, klassischen OO-Code zu schreiben

naja, so hatte ich es ja versucht: https://forum.selfhtml.org/?t=209455&m=1425788, bzw. versucht, einen Kommentar für meinen Versuch zu erhaschen.

, verständliche APIs zu entwerfen und dann stärker funktional zu programmieren. Denn diesen Code finde ich alles andere als elegant.

Frustierend, denn ich dachte auf Anhieb, er wäre es. Ich dachte auch, gut strukturierter Code würde sich gerade dadurch auszeichen, dass eben der externe Zugriff und auf die Dinge möglich ist, auf die zugegriffen werden soll. So macht es Crockford in seinem Memoizer ja auch, wie hier verlinkt: https://forum.selfhtml.org/?t=209455&m=1425815

Gruß

jobo