Skip to content

Assignment 2 - Sisi Cheng/Jean Sung#3

Open
SCheng4 wants to merge 8 commits intohmc-cs111-fall2014:masterfrom
SCheng4:master
Open

Assignment 2 - Sisi Cheng/Jean Sung#3
SCheng4 wants to merge 8 commits intohmc-cs111-fall2014:masterfrom
SCheng4:master

Conversation

@SCheng4
Copy link

@SCheng4 SCheng4 commented Sep 14, 2014

No description provided.

Choose a reason for hiding this comment

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

Nice!

Copy link

Choose a reason for hiding this comment

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

Not all the letters are represented in the input sequence. This function won’t correctly decode "qee" into "zoo". I’m not sure if anybody actually handled that problem well, though: I hardcoded the whole map; Prof. Ben hardcoded a few codes which were not all given in the hints at the start of the function. The sample solution doesn’t seem to do either, but I didn’t quite understand what it did.

Copy link
Author

Choose a reason for hiding this comment

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

Just a heads up, I believe you're looking at a wrong pull request, since this is from DSL last year. You may also be sending email notification to a bunch of people who are no longer in DSL or at Mudd.

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.

6 participants