nilsreich / tictac

https://tictactoe-psi.now.sh
0 stars 0 forks source link

verst #1

Open nilsreich opened 4 years ago

nilsreich commented 4 years ago

Ich glaube wir hatten im prinzip die selben ansätze, nur bei der umsetzung sind wir anders vorgegeangen

Was ich mich frage ist folgendes: Du nutzt "count" offensichtlich als variable, mit der du den Spieler definierst und gibst ihr als wert eine zahl. Wieso nennst du die variable nicht "player" und gibst ihr jeweils den wert "kreuz" oder "kreis"? Wäre das nicht einfacher zu lesen?

So wie ich das verstanden habe, nutzt du sumh, sum... als variable, in der festgestellt wird, ob diese reihen nur mit kreuz oder kreis gefüllt sind? Allerdings überprüfst du alle reihen in dem spielfeld bei jeder neuen ausgewählten Zelle. Wieso? Warum nicht einfach nur die Reihe, die die ausgewählt zelle auch beinhaltet?

wieso resetest du nich einfach, in dem du neu lädst?

nilsreich commented 4 years ago
  1. die Variable 'count' zählt die klicks um bei jeder gerade Anzahl den Spieler zu wechseln, aber ja, ein einfaches togglen ist vielleicht besser zu lesen. Änderst du es?

  2. das überprüfen ob einer gewonnen hat, finde ich wirklich nicht so leicht. Ich habe zwar deins gelesen, aber nicht verstanden. Spieler A füllt das Array "clicked" mit 1, SpielerB mit -1. sumh berechnet die horizontale Summen, sumv und sumd dementsprechend die Vertikale und Diagonale. Aber glücklich damit bin ich nicht... Wenn du einen "gut lesbaren" Vorschlag hast, schaue ich ihn mir gerne an!

  3. Ich glaube bei einem Refresh der Seite wird alles neu gerendert und neu gezeichnet. So nur die Elemente die sich ändern