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

New code will be easier to understand and is well written #2

Open
wants to merge 1 commit into
base: master
from

Conversation

@JAI-2002
Copy link

@JAI-2002 JAI-2002 commented Sep 30, 2020

No description provided.

@neilbalch
Copy link
Owner

@neilbalch neilbalch commented Sep 30, 2020

Ookay. 😃 Well:

  1. The code needs to match with what's described in the written tutorial
  2. I'm not exactly sure how this is an improvement in the eyes of a novice. Sure, it's more concise and more like what an experienced programmer would produce, but I think the compressed lines make it harder to understand the flow of what's happening for novices.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.