Feedback zu Aufgabe 3, 14.04.2019

[R4Pi3R]

Konzept:

• Es ist wie gesagt ok, dass das Konzept in eine HTML-Seite eingebunden wird. Achte dann aber bitte darauf, dass sich die Breite an die des Fensters anpasst und das Konzept nicht zum Sidescroller wird
• Im UCD fehlt die Umrandung der einzelnen für den User möglichen Aktionen. Außerdem hat das System keinen Namen
• Was ist "obereKarte"? Woher kommt diese Variable? Warum wird sie dem Stapel hinzugefügt?
• Werden der Funktion immer das selbe Array und die selbe Karte übergeben? Falls ja, warum wird nicht direkt darauf zugegriffen? Falls nein, als Parameter sollten ein Kartenarray und eine karte übergeben werden
• Wie funktioniert das sortieren der Karten genau? Mit welcher Funktion / welchen Funktionen soll es durchgeführt werden?
• Wo werden die Eventlistener installiert?
• Wo werden die Events abgehandelt?

Aufgabe:

• So wie der Code momentan geschrieben ist kann nicht garantiert werden, dass die HTML-Elemente, auf denen Listener installiert werden sollen, auch tatsächlich existieren, wenn dies geschehen soll
• Warum initialisierst du position nicht mit einem leeren String ( also einfach "" ), wenn der Wert ohnehin verändert wird?
• Funktioniert sonst wie gefordert, Sortierung nach Farbe und Wert, zusätzlich Abgleich implementiert, ob Karte gespielt werden kann
• Code quasi identisch mit Reinders, Carlotta/ Sanchez, Lisa/ Kübler, Marvin/ Mainberger, Daniel/ Wurz, Kathrin

[Plagiatus]

Schon wieder so eine riesen Gruppe... 😩

[Bastian-C]

"Schon wieder so eine riesen Gruppe... "

Die Gruppenbeschreibung wurde noch nicht abgeändert, wir haben die Gruppe in 2 Untergruppen aufgespalten. Ich habe darauf geachtet, dass jeder Teilnehmer der Gruppe den Code selbstständig aufbauen kann und erklähren kann.

[Bastian-C]

Das Konzept wird überarbeitet.

"So wie der Code momentan geschrieben ist kann nicht garantiert werden, dass die HTML-Elemente, auf denen Listener installiert werden sollen, auch tatsächlich existieren, wenn dies geschehen soll"

Die HTML Elemente, auf denen die Listener installiert werden, sind in der Index HTML vordefiniert. Die Listener werden ausschließlich auf den Elementen mit der ID: "Deck", "HandDisplay" und "Sort" installiert. Es ist mir nicht begreiflich, unter welchen Umständen diese HTML-Elemente nicht existieren könnten, da bei der "generateHand" Funktion lediglich der Inhalt des elementes mit der ID "HandDisplay" geleert wird, das Element selbst jedoch bestehen bleibt.

"Warum initialisierst du position nicht mit einem leeren String ( also einfach "" ), wenn der Wert ohnehin verändert wird?"

Das ist ein überbleibsel von meinem vorherigen Studium (Elektrotechnik: Sensorik) wo ich mir die Grundlagen von C und C++ angeeignet hatte. Da mussten ich in C "Null" als Null-Pointer und "nullptr" in C++ verwenden, wenn ich einen leeren pointer beschreiben wollte. Was ich damit Beschreiben wollte, ist das die Position "null" in der Hand niemals existiert. Ein leerer String würde ebenfalls funktionieren.

Es hat keine funktionelle Relevanz, sondern ist lediglich Gewohnheit.

"Code quasi identisch mit Reinders, Carlotta/ Sanchez, Lisa/ Kübler, Marvin/ Mainberger, Daniel/ Wurz, Kathrin"

In der Tat. Wir haben die Aufgabe als Gruppe erfüllt. Ich habe sicher gestellt, dass jeder Teilnehmer der Gruppe den Code selbstständig aufbauen, erklähren und verteidigen kann. Die Lösung einiger Probleme wurde gemeinsam erarbeitet.

[R4Pi3R]

Die HTML Elemente, auf denen die Listener installiert werden, sind in der Index HTML vordefiniert. Die Listener werden ausschließlich auf den Elementen mit der ID: "Deck", "HandDisplay" und "Sort" installiert. Es ist mir nicht begreiflich, unter welchen Umständen diese HTML-Elemente nicht existieren könnten, da bei der "generateHand" Funktion lediglich der Inhalt des elementes mit der ID "HandDisplay" geleert wird, das Element selbst jedoch bestehen bleibt. Der geht auf mich, entschuldige bitte die zu kurze Anmerkung. Dass ein Element im HTML vordefiniert ist bedeutet nicht, dass das Element bei der ersten Ausführung des Scripts auch tatsächlich schon existiert. Das Script wird im head verlinkt und beide Dateien werden von oben nach unten durchschritten. Dabei kann es vorkommen, dass das Script versucht auf Objekte zuzugreifen, von denen das DOM noch nicht weiß, dass sie auch tatsächlich da sind.

Das ist ein überbleibsel von meinem vorherigen Studium (Elektrotechnik: Sensorik) wo ich mir die Grundlagen von C und C++ angeeignet hatte. Da mussten ich in C "Null" als Null-Pointer und "nullptr" in C++ verwenden, wenn ich einen leeren pointer beschreiben wollte. Was ich damit Beschreiben wollte, ist das die Position "null" in der Hand niemals existiert. Ein leerer String würde ebenfalls funktionieren.

Es hat keine funktionelle Relevanz, sondern ist lediglich Gewohnheit. War tatsächlich weniger Kritik als Interesse, danke für deine Erklärung

In der Tat. Wir haben die Aufgabe als Gruppe erfüllt. Ich habe sicher gestellt, dass jeder Teilnehmer der Gruppe den Code selbstständig aufbauen, erklähren und verteidigen kann. Die Lösung einiger Probleme wurde gemeinsam erarbeitet. Der Kompetenzgewinn muss sicher gestellt werden, daher ist die Verteidigung notwendig. Wenn alle so gut vorbereitet sind sollte das aber auch kein Problem darstellen