Jonathan
/
Babushka
2
0
Fork 0
Code Pull Requests 3 Releases Activity
Labels Milestones

Fight rework premature merge #22

Merged
Jonathan merged 16 commits from feature/fight_rework into develop 2 months ago
Conversation 0 Commits 16 Files Changed 108
Jonathan commented 2 months ago
Owner

A working but not finished version of the new fight system.

Includes:

  • The entire fight logic is based on a state machine outlined on Miro
  • Procedurally generated fight world. Currently only generating two rooms next to each other.
  • Basic fight attack

Known bugs:

  • Dead enemies still attack
  • No background in fight
  • Buttons other than attack do not work
  • Enemy groups do not disappear when beaten
  • probably more...
A working but not finished version of the new fight system. Includes: - The entire fight logic is based on a state machine outlined on [Miro](https://miro.com/app/board/uXjVK8YEprM=/?moveToWidget=3458764640805655262&cot=14) - Procedurally generated fight world. Currently only generating two rooms next to each other. - Basic fight attack Known bugs: - Dead enemies still attack - No background in fight - Buttons other than attack do not work - Enemy groups do not disappear when beaten - probably more...
Jonathan added 14 commits 2 months ago
2929c5dd25 Bootstrap fight system
Jonathan changed title from Fight rework premature merge to WIP: Fight rework premature merge 2 months ago
Jonathan changed title from WIP: Fight rework premature merge to Fight rework premature merge 2 months ago
Jonathan requested review from kziolkowski 2 months ago
kziolkowski was assigned by Jonathan 2 months ago
kziolkowski reviewed 2 months ago
scripts/CSharp/Common/CharacterControls/InteractionArea2D.cs Outdated
[Export] private bool _useSprite = true;
[Export] private CanvasItem _spriteToOutline;
//[Export] private bool _useSprite = true;
[Export(PropertyHint.ArrayType)] private CanvasItem[] _spriteToOutline = [];
kziolkowski commented 2 months ago
Owner

Bitte in Plural umbenennen.

Bitte in Plural umbenennen.
Jonathan marked this conversation as resolved
kziolkowski reviewed 2 months ago
scripts/CSharp/Common/Fight/ActionAnimationController.cs Outdated
#endregion
[Export] private AllFightersVisual _allFightersVisual = null!;
kziolkowski commented 2 months ago
Owner

Nur aus Neugier, weil ich das noch nie gesehen hab: Was macht diese "null!" Zuweisung hier?

Nur aus Neugier, weil ich das noch nie gesehen hab: Was macht diese "null!" Zuweisung hier?
Jonathan marked this conversation as resolved
kziolkowski reviewed 2 months ago
scripts/CSharp/Common/Fight/ActionDetails/MinigameActionDetail.cs Outdated
using System.Collections.Generic;
kziolkowski commented 2 months ago
Owner

Ich weiß nicht, was es mit diesem Unicode-Character hier auf sich hat, aber ich gehe einfach mal davon aus, dass er uns keine Ärger machen wird 👀

Ich weiß nicht, was es mit diesem Unicode-Character hier auf sich hat, aber ich gehe einfach mal davon aus, dass er uns keine Ärger machen wird 👀
Jonathan marked this conversation as resolved
kziolkowski reviewed 2 months ago
scripts/CSharp/Common/Fight/ActionDetails/MinigameActionDetail.cs Outdated
using System.Collections.Generic;
namespace Babushka.scripts.CSharp.Common.Fight.ActionDetails;
kziolkowski commented 2 months ago
Owner

Vielleicht macht es Sinn, das Fightsystem in einen eigenen Namespace-Bereich auszulagern und nicht einfach mit unter "Common" abzulegen?

Vielleicht macht es Sinn, das Fightsystem in einen eigenen Namespace-Bereich auszulagern und nicht einfach mit unter "Common" abzulegen?
Jonathan marked this conversation as resolved
kziolkowski reviewed 2 months ago
scripts/CSharp/Common/Fight/ActionDetails/MinigameActionDetail.cs Outdated
public class MinigameActionDetail : FighterAction.FighterActionDetail
{
// settings
kziolkowski commented 2 months ago
Owner

brauchen wir den Kommentar noch?

brauchen wir den Kommentar noch?
Jonathan marked this conversation as resolved
kziolkowski reviewed 2 months ago
scripts/CSharp/Common/Fight/ActionDetails/TargetSelectActionDetail.cs Outdated
{
public enum VisualRange
{
Single
kziolkowski commented 2 months ago
Owner

Ein Enum mit nur einem Eintrag? Was soll da voraussichtlich noch kommen? Vielleicht

  • Single und Multiple oder vielleicht
  • Single, Double, Triple usw.?
Ein Enum mit nur einem Eintrag? Was soll da voraussichtlich noch kommen? Vielleicht - Single und Multiple oder vielleicht - Single, Double, Triple usw.?
Jonathan marked this conversation as resolved
kziolkowski reviewed 2 months ago
scripts/CSharp/Common/Fight/ActionDetails/TargetSelectActionDetail.cs Outdated
public FightWorld.Fighter GetTarget()
{
return target ?? throw new InvalidOperationException("No target selected");
kziolkowski commented 2 months ago
Owner

Muss das immer ne Exception werfen? Also kann es keinen Fall geben, wo es keine Selection gibt? ISt es dann für die Praxis nicht vielleicht nicer, wenn man nen Default definiert? Oder passiert das auf den höheren Detailebenen?

Muss das immer ne Exception werfen? Also kann es keinen Fall geben, wo es keine Selection gibt? ISt es dann für die Praxis nicht vielleicht nicer, wenn man nen Default definiert? Oder passiert das auf den höheren Detailebenen?
Jonathan commented 2 months ago
Poster
Owner

GetTarget sollte nur im ActionExecute und ActionAnimate state aufgerufen werden. Die states werden nur erreicht, nach dem alle action details completed sind. Das heißt es muss ein target gesetzt worden sein.

GetTarget sollte nur im ActionExecute und ActionAnimate state aufgerufen werden. Die states werden nur erreicht, nach dem alle action details completed sind. Das heißt es muss ein target gesetzt worden sein.
Jonathan marked this conversation as resolved
kziolkowski reviewed 2 months ago
scripts/CSharp/Common/Fight/Actions/BlobAttackAction.cs Outdated
namespace Babushka.scripts.CSharp.Common.Fight.Actions;
public class BlobAttackAction : FighterAction
kziolkowski commented 2 months ago
Owner

Verstehe ich das richtig, dass wir für jeden Gegner(typ) ne neue Klasse anlegen müssen? Falls ja: Wäre es vielleicht auch möglich, das über Config-Ressourcen ganz oder teilwese konfigurierbar zu machen?

Verstehe ich das richtig, dass wir für jeden Gegner(typ) ne neue Klasse anlegen müssen? Falls ja: Wäre es vielleicht auch möglich, das über Config-Ressourcen ganz oder teilwese konfigurierbar zu machen?
Jonathan commented 2 months ago
Poster
Owner

Es braucht nicht jeder gegner type eine klasse, sondern jede action. Es können auch mehrere gegner typen die gleiche action haben.

Das irgendwie cool konfigurierbar zu machen fänd ich auch sinnvoll, aber erst später und nicht für den prototypen

Es braucht nicht jeder gegner type eine klasse, sondern jede action. Es können auch mehrere gegner typen die gleiche action haben. Das irgendwie cool konfigurierbar zu machen fänd ich auch sinnvoll, aber erst später und nicht für den prototypen
Jonathan marked this conversation as resolved
kziolkowski reviewed 2 months ago
scripts/CSharp/Common/Fight/AllFightersVisual.cs Outdated
namespace Babushka.scripts.CSharp.Common.Fight;
public partial class AllFightersVisual : Node
kziolkowski commented 2 months ago
Owner

Diese Klasse managed das Layout der Kampfszene, oder? Das Naming hatte mich etwas verwirrt.

Diese Klasse managed das Layout der Kampfszene, oder? Das Naming hatte mich etwas verwirrt.
Jonathan marked this conversation as resolved
kziolkowski approved these changes 2 months ago
kziolkowski left a comment
Owner

please see comments and resolve merge conflicts.

please see comments and resolve merge conflicts.
Jonathan added 1 commit 2 months ago
Jonathan force-pushed feature/fight_rework from 1edbe7aeb7 to f27f69c15f 2 months ago
Jonathan merged commit e864c62a3a into develop 2 months ago

Reviewers

kziolkowski approved these changes 2 months ago
The pull request has been merged as e864c62a3a.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
kziolkowski
Labels
Apply labels
Clear labels
No items
No Label
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
kziolkowski
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Jonathan/Babushka#22
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'feature/fight_rework'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes