Skip to content

Singleton#251

Open
JaydonFernandes wants to merge 2 commits into
claritylab:masterfrom
JaydonFernandes:singleton
Open

Singleton#251
JaydonFernandes wants to merge 2 commits into
claritylab:masterfrom
JaydonFernandes:singleton

Conversation

@JaydonFernandes

Copy link
Copy Markdown

No description provided.

@seraasch

Copy link
Copy Markdown

Jayson - This project is essentially abandoned. You might consider cloning it.

@jhauswald

Copy link
Copy Markdown
Contributor

Hey @JaydonFernandes thank you for both of your PRs! While @seraasch mentioned this project has been silent for some time, we'd definitely welcome contributions and continued support from the community.

Could you add a more detailed description so we can better understand the goal of the PR and how to test? I'll have to get reacquainted with this code, especially OpenEphyra was a project developed by researchers at CMU a while back.

@JaydonFernandes

Copy link
Copy Markdown
Author

Hey @jhauswald this is actually for one of our university classes. Our goal was to find a project on GitHub and see where we can implement a design pattern. We chose the SentenceExtractor class as there is only a need for a single instance of it in the system and having multiple instances could make the data inconsistent. We saw that many of its fields and methods were static to prevent to against this. However, by making it a singleton we can now ensure that there is only at most one instance for the whole system.

All tests should pass with this new change as it does not change any of the logic or functionality, only makes it more robust.

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.

3 participants