Skip to content

Code refactoring. + Synchronized methods getContent(), getContentWith… #453

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Code refactoring. + Synchronized methods getContent(), getContentWith… #453

wants to merge 2 commits into from

Conversation

dev-blogs
Copy link

Code refactoring

  • Synchronized methods getContent(), getContentWithoutUnicode, saveContent()
  • Replaced String by StringBuilder in getContent() and getContentWithoutUnicode() methods
  • Replaced FileInputStream by FileReader in getContent() method
  • Optimazed reading by wrapping FileInputStream in getContentWithoutUnicode() method to BufferedInputStream and FileReader in getContent() method to BufferedFileReader

zheka added 2 commits November 27, 2020 23:08
Code refactoring. + Synchronized methods getContent(), getContentWith…
014b3de 
…outUnicode, saveContent(). + Replaced String by StringBuilder in getContent and getContentWithoutUnicode methods. + Replaced FileInputStream by FileReader in getContent method. + Optimazed reading by wrapping FileInputStream in getContentWithoutUnicode method and FileReader in getContent method to BufferedInputStream and BufferedFileReader methods
Code refactoring. Turned all sync methods to nonsync and added constr…
eda2973 
…uctor because class is threadsafe
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dev-blogs