Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Added a simple way to convert between strings and integer #3

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

Merged
BolajiAyodeji merged 4 commits into BolajiAyodeji:master from 1baga:master
Feb 9, 2019

Conversation

@1baga
Copy link
Contributor

@1baga 1baga commented Feb 9, 2019
edited
Loading

Added a simple way to convert an Integer to a string and vice versa. Useful when you are making quick conversions to be put in one line of code. Also added the modulus math operation.

screenshot from 2019年02月09日 21-46-46

Copy link
Owner

@BolajiAyodeji BolajiAyodeji left a comment
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!
You need to run some fixes first.

  • Your Integer to String conversion doesn't seem to work, I get 34 instead of "34"
  • You should use the camel case style to name your variables
    age_toString should be ageToString
    length_toInt should be lengthToInt

Copy link
Owner

@BolajiAyodeji BolajiAyodeji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!
Change the variable name from modulus to findModulus

Copy link
Owner

Looks great!
Change the variable name from modulus to findModulus

You haven't made this change @1baga

1baga reacted with eyes emoji

Copy link
Contributor Author

1baga commented Feb 9, 2019

Looks great!
Change the variable name from modulus to findModulus

You haven't made this change @1baga

done

Copy link
Owner

@BolajiAyodeji BolajiAyodeji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!
Thank you.

Copy link
Owner

@all-contributors please add @1baga for code

Copy link
Contributor

@BolajiAyodeji

I've put up a pull request to add @1baga! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BolajiAyodeji BolajiAyodeji BolajiAyodeji approved these changes

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /