Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Enhanced the cohesion of the Account class by grouping related elements to specific classes [ TLS + Media] #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

soundoss
Copy link

@soundoss soundoss commented Dec 2, 2016

The class Account had 192 methods among which many were not really related or at least could be grouped in different classes. So we've created new classes for instance AccountTLS and AccountMedia in order to group related elements, so that the code will be more readable, maintainable. In addition the cohesion of the newly formed class will be higher.

@aviau
Copy link
Contributor

aviau commented Dec 2, 2016

Thank you for the pull request. However, this GitHub repository is only used as a mirror. Please submit your patches using gerrit: https://gerrit-ring.savoirfairelinux.com

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants