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

2048! #11

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

2048! #11

wants to merge 17 commits into from

Conversation

digivava
Copy link

@digivava digivava commented Jul 9, 2016

Here it is!

So, I uploaded a separate branch with my original attempt, so you can see how I first tried to solve the problem. I had tried creating an entirely separate Board class, and using the "cell" divs to figure out which spots were already taken.

BUT! When I asked Charles for some help with a bug, he said that my way would get really complicated really soon, and then suggested a different way to do it. So I went with that instead, which is what you'll see on the Master branch. But because I struggled to understand his solution idea at first, I ended up having to spend more time asking him for help and working alongside him than I'd have liked to. (I mean Charles is awesome, but I wanted to feel like I was coming up with solutions more independently.)

Soooo even though I don't feel like this is "my solution" enough, I crammed my brain with a lot of new concepts that I'd never even considered, but look like they'll be useful in the future, and I worked hard to understand them.

One of the best parts was when we ran into several weird bugs with Charles' solution and I had to pair-program with him for a long time to figure it out. And for once, I felt like I was able to contribute good ideas about why things weren't working, and it was the first time I felt that I was capable of "keeping up" with his debugging. And we eventually figured things out together.

So YAY PROGRESS AS A PROGRAMMER. And yay Charles. He is amazing.

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.

1 participant