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

Comments on your pad array challenge #1

Open
pdebelak opened this issue Mar 29, 2014 · 1 comment
Open

Comments on your pad array challenge #1

pdebelak opened this issue Mar 29, 2014 · 1 comment

Comments

@pdebelak
Copy link

I thought you did a great job with this challenge. Really concise, readable code. I think your refactored solution to pad! is definitely better. To me it doesn't make a lot of sense to use a line to declare a variable and then only use it once. I find the refactored one more readable.

@XanderGlassman
Copy link

Hey Lana, I love what you did here, you have some really clean the code. When I finished writing my code I thought that it was re-factored well. However, there are some really cool tricks here I could have used like using the pad! method to define pad and that neat << technique! Overall a really great job!

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

No branches or pull requests

2 participants