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

Refactored code to address code smells and improve readability #28

Merged
merged 1 commit into from Jul 8, 2023

Conversation

ochotzas
Copy link
Contributor

@ochotzas ochotzas commented Jul 4, 2023

This commit focuses on refactoring the code to address identified code smells and enhance the overall readability of the codebase. The following specific changes have been made:

  • In the ExecutationSetupView class:
    Fixed indentation and removed unnecessary empty lines.
    Renamed the class to correct a the typo.

  • In the NewBufferView class:
    Reorganized imports for better organization.
    Improved variable naming to enhance code clarity.

  • In the PowerLineView class:
    Added null safety checks to ensure robustness.
    Extracted a separate method for setting the file name, improving code modularity.

Additionally, a few changes were applied to Main.kt and MainView.kt to align with the overall refactoring effort.

These modifications significantly contribute to the maintainability and comprehensibility of the codebase, facilitating future development and maintenance tasks.

@BenHerbst BenHerbst merged commit 555bf9e into BenHerbst:master Jul 8, 2023
1 check passed
@BenHerbst
Copy link
Owner

Thank you for your work

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.

None yet

2 participants