Carsten Gips

Results 289 comments of Carsten Gips

then all tests regarding headings are currently wrong and this is the intended behaviour: ``` $ pandoc -t asciidoc # Level 1 ## Level 2 ### Level 3 ^D ==...

> > [@AMatutat](https://github.com/AMatutat) [@Flamtky](https://github.com/Flamtky) Was denkt ihr? Ich weiss, dass das grad nicht oben auf eurer Prioritätenliste steht, aber der Mix aus Optional und klassischem OO kratzt ein wenig. >...

> Ich verstehe den Sinn von Optionals, um NullPointerExceptions vorzubeugen, `Optional` ist keine Vorbeugung von NullPointerExceptions. Es dient einzig und allein dazu, den Fall "ich habe keine Ergebnisse und das...

> Damit aus sowas > > PositionComponent pc = > entity > .fetch(PositionComponent.class) > .orElseThrow(() -> MissingComponentException.build(entity, PositionComponent.class)); > pc.position(Game.startTile()); > pc.viewDirection(PositionComponent.Direction.DOWN); Das ist auch ganz schlechter Code (imho). Wenn...

> > Ja, aber nur wenn die Entität die gewünschte Component auch sicher hat. Und dann brauchst Du da kein Optional > > Das ist ja schön und gut, aber...

@viddie @AMatutat @Flamtky Hier sind verschiedene Aspekte vermischt, die alle relevant sind und die in irgendeiner Form adressiert werden sollten, da sie immer wieder aufpoppen. 1. **Explizites Auspacken von `Optionals`...

> > Idee von [@Flamtky](https://github.com/Flamtky) eine `Entity#applyOn(Class component, Consumer onPresent, Consumer orElse)` mit ggf. default implementierung der `orElse` mit Logging > > Vorschlag [@cagix](https://github.com/cagix) > > Java `assert` Keyword verwenden...

genau. vararg würde dir das überladen sparen. aber dann kriegst du das mit den individuellen typen nicht hin, dann hast du einen gemeinsamen obertyp für alle. und ja, die "kunden"...

@Flamtky so richtig glücklich bin ich irgendwie noch nicht damit, aber es wäre ein schritt in die gesuchte richtung - man fragt nach components (klassen) und kriegt ein tupel mit...

> > > Das ist doch jetzt einfach die Optional/Stream-Api ausgelagert. > > > > > > In gewisser Weise schon. > > > Warum sollte ich `entity.applyIfPresent(......)` machen anstelle...