Wiki-Push: Spiele mit JavaScript

[Felix Riesterer]

Lieber Matthias,

JavaScript/Tutorials/Spiele/Tic-Tac-Toe

Meine Version des Tutorials hat nur noch 23KB Text (60% gekürzt)

sieht gut aus! Wenn ich aber die großen if-Statements lese, frage ich mich, ob das auch leserlicher geht.

// 3 senkrecht
if (fields[0 + i].getAttribute('aria-label') != ''
  && fields[0 + i].getAttribute('aria-label') == fields[3 + i].getAttribute('aria-label')
  && fields[3 + i].getAttribute('aria-label') == fields[6 + i].getAttribute('aria-label')
)

Diese Bedingungen prüfen doch auf den Wert des aria-label-Attributs. Da könnte man doch eine kürzere Schreibweise verwenden:

function mark (node) {
  if (node.hasAttribute('aria-label')) {
    return node.getAttribute('aria-label');
  }
}

if (mark(fields[0 + i])
  && mark(fields[0 + i]) == mark(fields[3 + i])
  && mark(fields[3 + i]) == mark(fields[6 + i])
)

Ob das jetzt unbedingt besser ist, mag eine Frage der Betrachtungsweise und der persönlichen Vorlieben sein.

Liebe Grüße

Felix Riesterer