search

pklaschka / RGW-Q11-inf21-16-18-Projekt

A school project by the German inf21-Kurs ;-)
MIT License
4 stars 1 forks source link

Hochdruckreinigung von PlayerMovementController #11

Closed S-Florian closed 7 years ago

S-Florian commented 7 years ago

Unser PlayerMovementController hatte ein paar Schönheitsmacken, wie z.B. der if-Haufen den wir fürs Springen gebraucht haben, sowie wiederholten code. Die PR ist nicht unglaublich bedeutsam, aber solche Sachen stören mich halt 😛 .

S-Florian commented 7 years ago

Klar, eine Sekunde!

"x?a:b"-Verzweigungs-System (ich finde gerade keinen anderen Begriff dafür 😀)

Das ist der ternary operator 😛 .

S-Florian commented 7 years ago

Ich hab den enum nicht kommentiert, den haben wir ja in der 10. Klasse durchgenommen.

pklaschka commented 7 years ago

@S-Florian

Das ist der ternary operator :stuck_out_tongue: .

:astonished: Hast du das jetzt aus dem Stegreif gewusst?!

S-Florian commented 7 years ago

@pklaschka Natürlich, ich kenn meine Operatoren in- und auswendig ;).

Übrigens, ein bisschen off-topic, aber ich hab grad realisiert, dass wir'n bisschen verpeilt sind 😛 . Das ist viel einfacher als 180° Rotation oder Skalierung und wir haben auch keine Probleme mit dem Licht, das ab und zu mal verschwindet. Das ist aber ein fix für eine neue PR, meine ich.

pklaschka commented 7 years ago

@S-Florian Alles klar, dann machen wir das tatsächlich am Besten im neuen PR, VG und danke für das Engagement, Pablo

pklaschka commented 7 years ago

Sorry, ich hatte eben ausversehen "close and comment" geklickt, jetzt ist aber alles gemerged...

pklaschka commented 7 years ago

@S-Florian Die Frage mit dem Flip-X ist dann aber auch, ob die Animationen dann noch funktionieren (die Position der Elemente müsste ja dann getauscht werden)

S-Florian commented 7 years ago

@pklaschka Ach ja, stimmt, da hast du Recht. Und wir müssten es sowieso auf jedem einzelnen Körperteil machen, da jedes seinen eigenen SpriteRenderer hat.

Von daher, wäre vielleicht eine bessere Lösung, das Licht einen Skript zu geben, dem Spieler zu folgen anstatt es ein Unterelement zu machen, da dabei keine Rotation dabei ist. Wir haben ja bereits den PositionFollower, der sollte doch funktionieren?

pklaschka commented 7 years ago

@S-Florian Ich habe das Licht schon an die Kamera "geklemmt", jedoch hat das Material nur einseitig die Beleuchtungsmöglichkeit, ggf. muss ich hier noch den Shader etw. abändern.

S-Florian commented 7 years ago

@pklaschka Okay, jetzt verstehe ich. Ich dachte das Problem wäre, dass sich das Licht hinter den Spieler dreht. Dann ist mit dem Shader basteln wohl eine Lösung 😛 .