Skip to content

Alert itrusion refactoring#134

Open
Abhiram98 wants to merge 5 commits intoLSTS:developfrom
Abhiram98:alert-itrusion-refactoring
Open

Alert itrusion refactoring#134
Abhiram98 wants to merge 5 commits intoLSTS:developfrom
Abhiram98:alert-itrusion-refactoring

Conversation

@Abhiram98
Copy link

@Abhiram98 Abhiram98 commented Aug 4, 2024

Refactor AlertIntrusion to split up long methods and rename variables. These changes make the code more readable and easier to maintain.

List of changes:

  • Rename gg -> graphicsContext
  • Rename g2 -> imageGraphics
  • Rename askForLaterRepaint -> repaintRequest
  • split the paint function into 'detectAndRenderClosestCollision', 'createInfoLabel'
  • Replace for loop with foreach loop
  • Use ternary operator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant