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

combine game high&standard #53

Merged
merged 4 commits into from Nov 20, 2017
Merged

Conversation

adm2656
Copy link
Contributor

@adm2656 adm2656 commented Nov 17, 2017

combine game high and standard, also fix the problem when the team
sucks and the highest Plus/Minus is 0

PS. I'm just a college student, if my code has any problem feel free to tell me :)

combine game high and standard, also fix the problem when the team
sucks and the highest Plus/Minus is 0
`${field_goals_made}-${field_goals_attempted}`,
`${three_pointers_made}-${three_pointers_attempted}`,
`${free_throws_made}-${free_throws_attempted}`,
checkOverStandard(plus_minus, 15),
Copy link
Owner

Choose a reason for hiding this comment

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

This is the only difference place between two cases, right?
I don't think this is necessary, cuz it's still the highest in the team even though it's 0.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I got it, I will remove that part of code and pull the request again.
Btw thanks for the reply.

Copy link
Owner

Choose a reason for hiding this comment

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

no problem.

Copy link
Owner

@homerchen19 homerchen19 left a comment

Choose a reason for hiding this comment

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

Thanks!

@homerchen19 homerchen19 merged commit a81d36f into homerchen19:master Nov 20, 2017
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.

None yet

2 participants