Das Video hat er runter genommen, aber es wäre das Video:Dürfte ich mal wissen welches Video ? Möchte das auch mal sehen
DevAthlon #3 - Thema für Runde 1
- Ersteller geNAZt
- Erstellt am
- Status
Dies Stammt nun von @CreepPlays
Beispiel: TeamUnkreativ
Da fallen mir schon auf dem ersten Blick kritische Fehler auf (die mir auf den ersten Blick aufgefallen sind):
- Nicht eindeutige Klassennamen (Move z.B., da weiß man nicht, ob es ein Listener ist oder nicht, da verliert man bei vielen Klassen den Überblick)
- Deutsche Klassennamen (man wurde den Konventionen nicht gerecht): AmbossE, Enderauge, ...
- Die Formatierung geht gar nicht: https://github.com/Lordizzel/DevAthlon2/blob/master/Magic/src/de/unkreativ/magic/Enderauge.java
- Sie machen sinnlose Konstruktoren bei den Listenern, die leer sind.
- Ich verstehe nicht, wie manche Listener überhaupt gehen sollten, wenn sie nicht einmal registriert werden (AmbossA, der Konstruktor ist leer und in der Main wird nur der Konstruktor aufgerufen)
- In der Main-Klasse ist "Partikel" groß geschrieben und auf deutsch und ist sogar noch static, was man ja weitestgehend vermeiden und wenn dann Singletons nutzen sollte
- Unnötiger Else-Block in Enderauge.java
- Wieso implementieren sie zusätzlich Listener in Commands, wenn es eigentlich unnötig ist?
Ja, oder sowas:
Das ist einfach Schwachsinn! Sorry, ist aber so
Beispiel: TeamUnkreativ
Da fallen mir schon auf dem ersten Blick kritische Fehler auf (die mir auf den ersten Blick aufgefallen sind):
- Nicht eindeutige Klassennamen (Move z.B., da weiß man nicht, ob es ein Listener ist oder nicht, da verliert man bei vielen Klassen den Überblick)
- Deutsche Klassennamen (man wurde den Konventionen nicht gerecht): AmbossE, Enderauge, ...
- Die Formatierung geht gar nicht: https://github.com/Lordizzel/DevAthlon2/blob/master/Magic/src/de/unkreativ/magic/Enderauge.java
- Sie machen sinnlose Konstruktoren bei den Listenern, die leer sind.
- Ich verstehe nicht, wie manche Listener überhaupt gehen sollten, wenn sie nicht einmal registriert werden (AmbossA, der Konstruktor ist leer und in der Main wird nur der Konstruktor aufgerufen)
- In der Main-Klasse ist "Partikel" groß geschrieben und auf deutsch und ist sogar noch static, was man ja weitestgehend vermeiden und wenn dann Singletons nutzen sollte
- Unnötiger Else-Block in Enderauge.java
- Wieso implementieren sie zusätzlich Listener in Commands, wenn es eigentlich unnötig ist?
Ja, oder sowas:
Code:
import com.avaje.ebeaninternal.server.deploy.BeanDescriptor.EntityType;Das ist einfach Schwachsinn! Sorry, ist aber so
Mir ist es wirklich egal, ob wir weiter sind oder nicht, es geht ums Dabeisein. Aber mir kann keiner, wirklich keiner erzählen, dass die besser waren als wir.
Auf welchen Bewertungsgrundlagen wurde denn bewertet?
OOP, Konventionen, Thread Sicheheit, Design Pattern, Komplexität(KISS?)...
Oder auch das Spiel selber.. Individualität, Umsetzung etc.. welche Gewichtung gibt es?
Wer hat bewertet und wie? Hat @geNAZt auch mitbewertet, und was?
Es fehlt ein wenig Transparenz bei dieser Bewertung..
Haben wohl die meisten schon ihre Kritik zu geschrieben!
Ein wenig fehlt da @BlackyPaw .. Ich kann mich noch an den ersten Devathlon errinern.. Jeder hat damals eine 3 Seitige Bewertung bekommen aus welchem man sehr gut seine Stärken und Schwächen ablesen konnte.. sowas fehlt hier^
Mfg Marcel
~ #TeamTunity - leider ausgeschieden :c
OOP, Konventionen, Thread Sicheheit, Design Pattern, Komplexität(KISS?)...
Oder auch das Spiel selber.. Individualität, Umsetzung etc.. welche Gewichtung gibt es?
Wer hat bewertet und wie? Hat @geNAZt auch mitbewertet, und was?
Es fehlt ein wenig Transparenz bei dieser Bewertung..
Haben wohl die meisten schon ihre Kritik zu geschrieben!
Ein wenig fehlt da @BlackyPaw .. Ich kann mich noch an den ersten Devathlon errinern.. Jeder hat damals eine 3 Seitige Bewertung bekommen aus welchem man sehr gut seine Stärken und Schwächen ablesen konnte.. sowas fehlt hier^
Mfg Marcel
~ #TeamTunity - leider ausgeschieden :c
- Sauberkeit und Lesbarkeit des Codes [Oh..]
- Umgang mit Tools [Ich denke mal z.B. mit Guava oder Maven, keine Ahnung]
- Code-Style allgemein (Konsistenz, Modularisierung, OOP, Ressource-Safety, …) [Das geht nicht, wenn nur 5 Klassen vorhanden sind ...]
- Kontinuität des Spielerlebnisses
- Gesamteindruck (Passen Spielelemente zueinander, …)
- Detailverliebtheit [JavaDocs oder Kleinigkeiten halt]
Zum Beispiel Devinitia habe ich mir mal angesehen und zwischen deren Code und z.B. den Codes von oben sind Welten.
@TheCrealm Konventionen, Thread Sicherheit wie soll man das bewerten wenn man selber keine Ahnung hat? *hust*
Man hätte die Jury prüfen sollen, ob sie so etwas überhaupt bewerten kann, sodass so etwas nicht passieren würde.
Wenn man schon deutsche Klassennamen sieht, dann wäre das bei mir schon ein Minuspunkt.
Das Ergebnis entspricht einfach zum Großteil nicht der Wahrheit, wenn man sich das mal genauer ansieht, welche Projekte die Teams gemacht haben.
Wenn man schon deutsche Klassennamen sieht, dann wäre das bei mir schon ein Minuspunkt.
Das Ergebnis entspricht einfach zum Großteil nicht der Wahrheit, wenn man sich das mal genauer ansieht, welche Projekte die Teams gemacht haben.
Es sind immernoch Schiedsrichter, sie haben vollmacht und gegen ein urteil darf man nichts einwenden. das ist nunmal deren job ^^
Aber sie sind nun mal eine Jury für einen Developer-Contest, das kann man nicht direkt mit einem Schiedsrichter vergleichen.
Die Schiedsrichter hatten ja auch eine Ausbildung und mussten Tests bestehen, damit kein Leichtsinn geschieht.
Die Schiedsrichter hatten ja auch eine Ausbildung und mussten Tests bestehen, damit kein Leichtsinn geschieht.
trotzdem finde ich es nicht wirklich gut sie jetzt für alles zu blamen, vielleicht ein paar falsche entscheidungen aber sie sind nicht die schlechtesten der schlechten und die teufel in person
- Status