Lab 5: Project 1 Peer Code Review

Introduction

In this lab, you will compare your solution to Project 1 with 1-3 other students in your lab. On-time attendance is required this week. If you cannot attend your regular lab, you may attend another instead. If (and only if) you are unable to attend any lab at all, please fill out this exemption form.

As part of this lab, TAs will go over part of the solution for Project 1. Do not attend this lab before you have made your final Project 1 submission to Gradescope, otherwise you will get marked for academic dishonesty.

LinkedListDeque Overview

Your TA will start the lab by giving a brief overview of the staff solution to LinkedListDeque.

LinkedListDeque Peer Review

Group up with 1-3 other students and compare solutions. You should form groups with other students in Discord rooms. Don’t be afraid of meeting someone new! Programmers in 61B vary widely in their level of experience and comfort with programming. Our goal here is to help each other get better. Please be nice, and don’t feel bad if your solution is less elegant or even downright ugly. I’ve certainly written incredibly ugly, inelegant code! Why here’s a 1600 line monstrosity I wrote in 1997, and a video demo of it running if you’re curious what it does. knaveos will not be on the midterm.

Avoid the temptation to explain exactly how your implementation works to your partner. Instead focus your discussion on more specific questions. Some suggested questions are listed below:

  1. What was the most annoying bug you had and how did you fix it? Did you use the debugger? Did you fix it by adding special cases? Did you do any change-and-pray (where you make a tiny change and hope the AG approves)?
  2. Did you end up cutting anything out to make your code simpler? If so, what?
  3. Do you have any special cases in your code?
  4. Do you have any private helper methods?
  5. Does your code repeat itself anywhere? Would private helper methods have helped?
  6. Were you able to call or reuse code anywhere?

After discussion, fill out the first half of self_reflection.txt with your own self reflection.

ArrayDeque Overview

Your TA will go over the ArrayDeque solution.

ArrayDeque Peer Review

Now, pair up again and discuss your ArrayDeque solutions as you did for LinkedListDeque. Fill out your self_reflection.txt. We recommend that you talk to different people than you did for ArrayDeque, but it’s OK to stick with your group for ArrayDeque.

Self Reflection and Submission

Make sure you’ve filled out out at least 4 of the questions in the self_reflection.txt document provided in the skeleton. Ask a TA to check your self_reflection.txt and give you the magic word to put in magic_word.txt. Push to Github and submit to Gradescope.