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

Implemented string interpolation for increased code readability #113

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

Conversation

paulritzman
Copy link

Converted strings using String.Format or string concatenation to instead use string interpolation. This was done in an attempt to improve readability.

Within the GetResults method in ResponsivenessTimer.cs, resultString was changed to use StringBuilder as the "+=" operation was being called each iteration of the loop.

Fixed minor spelling errors for variable names, and added code comments.

…s by converting text to use string interpolation.
…mponents - improving memory usage; Implemented string interpolation to reduce number of String.Concat() invocations; Added measureSet parameter comment and corrected a spelling error
… string "noValidExtensionMessage" to store text due to the message being multiple lines, reducing need to change text in multiple places in the future.
@msftclas
Copy link

msftclas commented Aug 2, 2018

CLA assistant check
All CLA requirements met.

…ileLogging method in Logger.cs - due to readability
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.

2 participants