Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor OrbitFinder #804

Open
5 tasks
thkruz opened this issue Nov 10, 2023 · 0 comments
Open
5 tasks

Refactor OrbitFinder #804

thkruz opened this issue Nov 10, 2023 · 0 comments
Assignees
Milestone

Comments

@thkruz
Copy link
Owner

thkruz commented Nov 10, 2023

User Story

  • As a developer of the Keep Track project,
  • I want to refactor the OrbitFinder class to reduce its complexity and improve logical flow,
  • So that the codebase becomes more maintainable, and methods within the class are more encapsulated and less inter-dependent.

Acceptance Criteria

  • Given the current state of the OrbitFinder class with high complexity and interdependent methods,
  • When I refactor the class,
  • Then the class should exhibit lower complexity, have a more logical flow, and its methods should be more encapsulated and less inter-dependent.

Tasks

  • Analyze the current OrbitFinder class to identify areas of high complexity and inter-dependency.
  • Design a refactoring plan that outlines the steps to reduce complexity and improve logical flow.
  • Refactor methods to be private where appropriate, ensuring encapsulation and reducing unnecessary exposure of class internals.
  • Implement changes to reduce the inter-dependency of methods within the class.
  • Conduct code reviews and testing to ensure the refactored class maintains its functionality and adheres to project standards.

Commentary

The refactoring of the OrbitFinder class is crucial for the long-term maintainability of the Keep Track codebase. This task should focus on improving code readability and reducing the complexity without altering the external behavior of the class. Collaboration and continuous integration practices should be followed to ensure a smooth transition and effective refactoring process.

@thkruz thkruz added this to the Backlog milestone Nov 10, 2023
@thkruz thkruz self-assigned this Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant