Skip to content

Instantly share code, notes, and snippets.

@mcsee
Last active August 21, 2023 19:28
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save mcsee/c34dd227f16b52772f8c4cfbb31841e8 to your computer and use it in GitHub Desktop.
Save mcsee/c34dd227f16b52772f8c4cfbb31841e8 to your computer and use it in GitHub Desktop.
class ExcelSheet {
FileProperties fileProperties;
SecurityProperties securityProperties;
DocumentDatingProperties datingProperties;
RevisionProperties revisionProperties;
LanguageProperties languageProperties;
DocumentContent content;
}
// Object has less attributes
// They are not only grouped for testability
// New objects are more cohesive, more testable,
// less conflicts and more reusable
// FileProperties/SecurityProperties can be reused for other documents
// Rules and preconditions on fileProperties will be moved to this object
// so ExcelSheet constructor will be cleaner
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment